Merge pull request #736 from stealjs/issue/727-extra-space

Issue/727 extra space

lib/build/clean.js

@@ -7,7 +7,7 @@ module.exports = function(original, options) {
 	}
 
 	removeTags.forEach(function(tag) {
-		result = result.replace(new RegExp('(\\s?)\/\/!' + tag + '-start((.|\n)*?)\/\/!' + tag + '-end', 'gim'), '');
+		result = result.replace(new RegExp('(\\s?)\/\/!(\\s?)' + tag + '-start((.|\n)*?)\/\/!(\\s?)' + tag + '-end', 'gim'), '');
 	});
 
 	return result;

test/clean_test.js

@@ -0,0 +1,15 @@
+var assert = require("assert");
+var clean = require("../lib/build/clean");
+
+describe("build/clean", function() {
+	var source = "123\n//!steal-remove-start\n456\n//!steal-remove-end\n789";
+	var source2 = "123\n//! steal-remove-start\n456\n//! steal-remove-end\n789";
+
+	it("removes blocks surrounded with steal-remove-start and -end", function() {
+		assert.equal(clean(source, {}), "123\n789");
+	});
+
+	it("allows an extra space before steal-remove-start for standard linting", function() {
+		assert.equal(clean(source2, {}), "123\n789");
+	});
+});

test/test.js

@@ -7,6 +7,7 @@ if (semver.lt(process.version, "8.0.0")) {
 
 // Unit tests
 require("./clean_address_test");
+require("./clean_test");
 require("../lib/bundle/bundle_test");
 require("./cli/cmd_build_test");
 require("./cli/cmd_bundle_test");