From f79f50884815a02036fba386d6d435320377a2e3 Mon Sep 17 00:00:00 2001 From: Nathan Cahill Date: Thu, 9 Nov 2017 10:02:05 -0700 Subject: [PATCH] add case for sequences with imports --- test/form/samples/sequence-expression/_expected/amd.js | 5 +++++ test/form/samples/sequence-expression/_expected/cjs.js | 5 +++++ test/form/samples/sequence-expression/_expected/es.js | 5 +++++ test/form/samples/sequence-expression/_expected/iife.js | 5 +++++ test/form/samples/sequence-expression/_expected/umd.js | 5 +++++ test/form/samples/sequence-expression/foo.js | 3 +++ test/form/samples/sequence-expression/main.js | 3 +++ 7 files changed, 31 insertions(+) create mode 100644 test/form/samples/sequence-expression/foo.js diff --git a/test/form/samples/sequence-expression/_expected/amd.js b/test/form/samples/sequence-expression/_expected/amd.js index 976cfc810df..1d43ed77362 100644 --- a/test/form/samples/sequence-expression/_expected/amd.js +++ b/test/form/samples/sequence-expression/_expected/amd.js @@ -1,5 +1,8 @@ define(function () { 'use strict'; + function foo$1 () { + console.log( 'foo' ); + } // should remove expressions without side-effect, multiple effects var a = (foo(), foo(), 2); // without white-space, effect at the end @@ -10,5 +13,7 @@ define(function () { 'use strict'; console.log(d); // should infer value + // should keep f import + var e = (foo$1()); }); diff --git a/test/form/samples/sequence-expression/_expected/cjs.js b/test/form/samples/sequence-expression/_expected/cjs.js index b26a5f65a3c..56352bc41e9 100644 --- a/test/form/samples/sequence-expression/_expected/cjs.js +++ b/test/form/samples/sequence-expression/_expected/cjs.js @@ -1,5 +1,8 @@ 'use strict'; +function foo$1 () { + console.log( 'foo' ); +} // should remove expressions without side-effect, multiple effects var a = (foo(), foo(), 2); // without white-space, effect at the end @@ -10,3 +13,5 @@ var d = (2); console.log(d); // should infer value +// should keep f import +var e = (foo$1()); diff --git a/test/form/samples/sequence-expression/_expected/es.js b/test/form/samples/sequence-expression/_expected/es.js index 66136f6070a..d4183f505d0 100644 --- a/test/form/samples/sequence-expression/_expected/es.js +++ b/test/form/samples/sequence-expression/_expected/es.js @@ -1,3 +1,6 @@ +function foo$1 () { + console.log( 'foo' ); +} // should remove expressions without side-effect, multiple effects var a = (foo(), foo(), 2); // without white-space, effect at the end @@ -8,3 +11,5 @@ var d = (2); console.log(d); // should infer value +// should keep f import +var e = (foo$1()); diff --git a/test/form/samples/sequence-expression/_expected/iife.js b/test/form/samples/sequence-expression/_expected/iife.js index af169b5ee86..896b8cd1958 100644 --- a/test/form/samples/sequence-expression/_expected/iife.js +++ b/test/form/samples/sequence-expression/_expected/iife.js @@ -1,6 +1,9 @@ (function () { 'use strict'; + function foo$1 () { + console.log( 'foo' ); + } // should remove expressions without side-effect, multiple effects var a = (foo(), foo(), 2); // without white-space, effect at the end @@ -11,5 +14,7 @@ console.log(d); // should infer value + // should keep f import + var e = (foo$1()); }()); diff --git a/test/form/samples/sequence-expression/_expected/umd.js b/test/form/samples/sequence-expression/_expected/umd.js index 57f9a02664c..ab7a58185a4 100644 --- a/test/form/samples/sequence-expression/_expected/umd.js +++ b/test/form/samples/sequence-expression/_expected/umd.js @@ -4,6 +4,9 @@ (factory()); }(this, (function () { 'use strict'; + function foo$1 () { + console.log( 'foo' ); + } // should remove expressions without side-effect, multiple effects var a = (foo(), foo(), 2); // without white-space, effect at the end @@ -14,5 +17,7 @@ console.log(d); // should infer value + // should keep f import + var e = (foo$1()); }))); diff --git a/test/form/samples/sequence-expression/foo.js b/test/form/samples/sequence-expression/foo.js new file mode 100644 index 00000000000..80f89059b70 --- /dev/null +++ b/test/form/samples/sequence-expression/foo.js @@ -0,0 +1,3 @@ +export function foo () { + console.log( 'foo' ); +} diff --git a/test/form/samples/sequence-expression/main.js b/test/form/samples/sequence-expression/main.js index ffb3fefc939..3a5d2086ea6 100644 --- a/test/form/samples/sequence-expression/main.js +++ b/test/form/samples/sequence-expression/main.js @@ -1,3 +1,4 @@ +import * as f from './foo'; // should remove expressions without side-effect, multiple effects var a = (0, foo(), 1, foo(), 2); // without white-space, effect at the end @@ -15,3 +16,5 @@ if ((1, 2) !== 2) { console.log( 'effect' ); } +// should keep f import +var e = (0, f.foo());