Breaking test for #1068

stealjs · Aug 30, 2018 · 6255900 · 6255900
1 parent 1397fe6
commit 6255900
Show file tree

Hide file tree

Showing 5 changed files with 29 additions and 3 deletions.
diff --git a/test/tree_shaking_test.js b/test/tree_shaking_test.js
@@ -126,6 +126,14 @@ describe("Tree-shaking", function(){
 				assert.equal(connect.didFail, true, "marked as failed");
 			});
 		});
+
+		describe.only("ES modules imported by CommonJS modules", function() {
+			it("Tree-shakes their ES module dependencies", function() {
+				let mod = app.dep5dep5AnotherESModule;
+				assert.equal(typeof mod.mainThing, "function", "Kept the used export");
+				assert.equal(typeof mod.another, "undefined", "Removed the unused export");
+			});
+		});
 	});
 
 	describe("treeShaking: false", function(){

diff --git a/test/treeshake/basics/main.js b/test/treeshake/basics/main.js
@@ -53,7 +53,8 @@ export default function(){
 		steal.import("dep4/other"),
 		steal.import("dep4/and-another"),
 		steal.import("~/from-exports"),
-		shouldFail
+		shouldFail,
+		steal.import("dep5/another-es-module")
 	]);
 
 	return p
@@ -64,7 +65,8 @@ export default function(){
 		dep4Other,
 		dep4AndAnother,
 		fromExports,
-		canConnect
+		canConnect,
+		dep5dep5AnotherESModule
 	]) => {
 		return {
 			anon,
@@ -75,7 +77,8 @@ export default function(){
 			dep4AndAnother,
 			fromExports,
 			dep5,
-			canConnect
+			canConnect,
+			dep5dep5AnotherESModule
 		};
 	});
 };
diff --git a/test/treeshake/basics/node_modules/dep5/another-es-module.js b/test/treeshake/basics/node_modules/dep5/another-es-module.js
diff --git a/test/treeshake/basics/node_modules/dep5/es-module.js b/test/treeshake/basics/node_modules/dep5/es-module.js
diff --git a/test/treeshake/basics/node_modules/dep5/main.js b/test/treeshake/basics/node_modules/dep5/main.js