Merge pull request #693 from stealjs/clean-css-rebase

Prevent clean-css to rebase urls

lib/assign_default_options.js

@@ -12,6 +12,7 @@ module.exports = function(config, options){
 		bundleSteal: false,
 		uglifyOptions: {},
 		cleanCSSOptions: {
+			rebase: false,
 			inline: ["none"]
 		},
 		removeDevelopmentCode: true,

test/css_image_urls/foo.js

@@ -0,0 +1 @@
+require("./main.css");

test/css_image_urls/main.css

@@ -0,0 +1 @@
+.topbanner { background-image: url("../../../topbanner.png"); }

test/css_image_urls/main.js

@@ -0,0 +1 @@
+require("./main.css");

test/css_image_urls/stealconfig.js

@@ -0,0 +1,8 @@
+steal.config({
+	ext: {
+		"css": "steal-css"
+	},
+	paths: {
+		"steal-css": "../steal-css.js"
+	}
+});

test/multibuild_test.js

@@ -6,6 +6,7 @@ var multiBuild = require("../lib/build/multi");
 var rmdir = require("rimraf");
 var path = require("path");
 var testHelpers = require("./helpers");
+var _escapeRegExp = require("lodash/escapeRegExp");
 
 var find = testHelpers.find;
 var open = testHelpers.open;
@@ -2147,6 +2148,7 @@ describe("multi build", function(){
 				return multiBuild({
 					main: "bundle",
 					config: path.join(__dirname, "bundle", "stealconfig.js")
+
 				}, {
 					quiet: true
 				});
@@ -2166,4 +2168,27 @@ describe("multi build", function(){
 					"node name comment is removed when the bundle is minified");
 			});
 	});
+
+	it("clean-css should not rebase urls by default", function() {
+		var base = path.join(__dirname, "css_image_urls");
+
+		return asap(rmdir)(path.join(base, "dist"))
+			.then(function() {
+				return multiBuild({
+					main: "main",
+					config: path.join(base, "stealconfig.js")
+				}, {
+					quiet: true
+				});
+			})
+			.then(function() {
+				return asap(fs.readFile)(
+					path.join(base, "dist", "bundles", "main.css")
+				);
+			})
+			.then(function(source) {
+				var rx = new RegExp(_escapeRegExp("url(../../../../../topbanner.png)"));
+				assert.ok(rx.test(source), "image url should be relative to 'dist'");
+			});
+	});
 });