Merge pull request #1145 from stealjs/es2015

Fix build of ES2015 code with forceES5 set to false

Author: m-mujica

package.json

@@ -39,7 +39,7 @@
     "through2": "^3.0.0",
     "tmp": "0.0.33",
     "traceur": "0.0.111",
-    "transpile": "^2.7.0",
+    "transpile": "^2.7.2",
     "urix": "^0.1.0",
     "winston": "^2.2.0",
     "ws": "^5.1.1",
@@ -105,7 +105,7 @@
     "test:exports-worker": "node test/exports_worker/exports.js && testee test/exports_worker/worker.html --browsers firefox --reporter Spec"
   },
   "engines": {
-    "node": "4.x - 10.x"
+    "node": "8.x - 10.x"
   },
   "system": {
     "npmDependencies": [

test/es2015_literals/config.js

@@ -0,0 +1,4 @@
+steal.config({
+	forceES5: false,
+	treeShaking: false
+});

test/es2015_literals/main.js

@@ -0,0 +1,4 @@
+let name = "John";
+let time = "today";
+
+export default `Hello ${name}, how are you ${time}?`;

test/multibuild_test.js

@@ -2330,4 +2330,37 @@ describe("multi build", function(){
 				);
 			});
 	});
+
+	it("should build ES2015 when data.loader.forceES5 is false", function() {
+		var config = {
+			config: path.join(__dirname, "es2015_literals", "config.js"),
+			main: "main"
+		};
+
+		var options = {
+			minify: false,
+			quiet: true,
+			sourceMaps: false
+		};
+
+		return asap(rmdir)(path.join(__dirname, "es2015_literals", "dist"))
+			.then(function() {
+				return multiBuild(config, options);
+			})
+			.then(function() {
+				var main = path.join(
+					__dirname,
+					"es2015_literals",
+					"dist",
+					"bundles",
+					"main.js"
+				);
+				var actualJS = fs.readFileSync(main, "utf8");
+
+				assert(
+					actualJS.includes("`Hello ${ name }, how are you ${ time }?`"),
+					"source should include ES2015 string literals"
+				);
+			});
+	});
 });