Merge pull request #17188 from pzuraq/bugfix/ie-dual-build

[BUGFIX beta][FEAT] Adds a second dist build which targets IE
emberjs · Nov 19, 2018 · 620f97c · 620f97c
2 parents 7be1bdf + 02af025
commit 620f97c
Show file tree

Hide file tree

Showing 18 changed files with 331 additions and 451 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -78,7 +78,9 @@ jobs:
 
     - stage: additional tests
 
-      env: TEST_SUITE=browserstack
+      env:
+      - TEST_SUITE=browserstack
+      - EMBER_ENV=production
     - env:
       - TEST_SUITE=built-tests
       - EMBER_ENV=production

diff --git a/bin/run-tests.js b/bin/run-tests.js
@@ -88,6 +88,7 @@ function generateBuiltTests() {
   // Container isn't publicly available.
   // ember-testing and @ember/debug are stripped from prod/min.
   var common = 'skipPackage=container,ember-testing,@ember/debug';
+
   testFunctions.push(function() {
     return run(common + '&nolint=true');
   });
@@ -100,6 +101,18 @@ function generateBuiltTests() {
   testFunctions.push(function() {
     return run(common + '&enableoptionalfeatures=true&dist=prod&prod=true');
   });
+  testFunctions.push(function() {
+    return run(common + '&ie=true&nolint=true');
+  });
+  testFunctions.push(function() {
+    return run(common + '&ie=true&dist=min&prod=true');
+  });
+  testFunctions.push(function() {
+    return run(common + '&ie=true&dist=prod&prod=true');
+  });
+  testFunctions.push(function() {
+    return run(common + '&ie=true&enableoptionalfeatures=true&dist=prod&prod=true');
+  });
 }
 
 function generateOldJQueryTests() {

diff --git a/broccoli/babel-helpers.js b/broccoli/babel-helpers.js
@@ -2,16 +2,9 @@
 
 const Funnel = require('broccoli-funnel');
 
-module.exports = function(env) {
-  let file;
-  if (env === 'debug') {
-    file = 'external-helpers-dev.js';
-  } else if (env === 'prod') {
-    file = 'external-helpers-prod.js';
-  }
-
+module.exports = function() {
   return new Funnel('packages/external-helpers/lib', {
-    files: [file],
+    files: ['external-helpers.js'],
     getDestinationPath() {
       return 'ember-babel.js';
     },

diff --git a/broccoli/bootstrap-modules.js b/broccoli/bootstrap-modules.js
@@ -1,10 +1,7 @@
 'use strict';
 
 const WriteFile = require('broccoli-file-creator');
-
-function defaultExport(moduleExport) {
-  return `(function (m) { if (typeof module === "object" && module.exports) { module.exports = m } }(requireModule('${moduleExport}').default));\n`;
-}
+const { stripIndent } = require('common-tags');
 
 function sideeffects(moduleExport) {
   return `requireModule('${moduleExport}')`;
@@ -14,11 +11,26 @@ function umd(moduleExport) {
   return `(function (m) { if (typeof module === "object" && module.exports) { module.exports = m } }(requireModule('${moduleExport}')));\n`;
 }
 
-module.exports = function bootstrapModule(moduleExport, type = 'sideeffects') {
+function testing() {
+  return stripIndent`
+    var testing = requireModule('ember-testing');
+    Ember.Test = testing.Test;
+    Ember.Test.Adapter = testing.Adapter;
+    Ember.Test.QUnitAdapter = testing.QUnitAdapter;
+    Ember.setupForTesting = testing.setupForTesting;
+  `;
+}
+
+function empty() {
+  return '';
+}
+
+module.exports = function bootstrapModule(type, moduleExport) {
   let moduleType = {
-    default: defaultExport,
-    umd,
+    empty,
     sideeffects,
+    testing,
+    umd,
   };
 
   return new WriteFile('bootstrap', moduleType[type](moduleExport));

diff --git a/broccoli/debug-macros.js b/broccoli/debug-macros.js
@@ -0,0 +1,43 @@
+'use strict';
+
+const Babel = require('broccoli-babel-transpiler');
+const FEATURES = require('./features');
+
+module.exports = function debugMacros(tree, environment) {
+  let isDebug = environment !== 'production';
+
+  let plugins = [
+    [
+      'debug-macros',
+      {
+        debugTools: {
+          source: '@ember/debug',
+          assertPredicateIndex: 1,
+          isDebug,
+        },
+        externalizeHelpers: {
+          module: true,
+        },
+        flags: [
+          { source: '@glimmer/env', flags: { DEBUG: isDebug } },
+          {
+            source: '@ember/canary-features',
+            flags: Object.assign(
+              // explicit list of additional exports within @ember/canary-features
+              // without adding this (with a null value) an error is thrown during
+              // the feature replacement process (e.g. XYZ is not a supported flag)
+              {
+                FEATURES: null,
+                DEFAULT_FEATURES: null,
+                isEnabled: null,
+              },
+              FEATURES
+            ),
+          },
+        ],
+      },
+    ],
+  ];
+
+  return new Babel(tree, { plugins });
+};
diff --git a/broccoli/deprecated-features.js b/broccoli/deprecated-features.js
@@ -93,10 +93,6 @@ function svelte(infile, outfile) {
     ],
   });
 
-  code = babel.transform(code, {
-    plugins: ['minify-dead-code-elimination'],
-  }).code;
-
   fs.writeFileSync(outfile, code);
 }
 

diff --git a/broccoli/strip-for-prod.js b/broccoli/strip-for-prod.js
@@ -0,0 +1,11 @@
+'use strict';
+
+const Babel = require('broccoli-babel-transpiler');
+
+module.exports = function stripForProd(tree) {
+  let options = {
+    plugins: [['filter-imports', { imports: { 'ember-babel': ['_classCallCheck'] } }]],
+  };
+
+  return new Babel(tree, options);
+};
diff --git a/broccoli/to-es5.js b/broccoli/to-es5.js
@@ -2,57 +2,13 @@
 
 const Babel = require('broccoli-babel-transpiler');
 const injectBabelHelpers = require('./transforms/inject-babel-helpers');
-const injectNodeGlobals = require('./transforms/inject-node-globals');
-const enifed = require('./transforms/transform-define');
-const FEATURES = require('./features');
-const stripClassCallCheck = require('./transforms/strip-class-call-check');
-const resolveModuleSource = require('amd-name-resolver').moduleResolve;
 
-module.exports = function toES5(tree, _options) {
-  let options = Object.assign(
-    {
-      environment: 'developement',
-    },
-    _options
-  );
+module.exports = function toES6(tree, _options) {
+  let options = Object.assign({}, _options);
 
-  let isDebug = options.environment !== 'production';
-
-  options.moduleIds = true;
-  options.resolveModuleSource = resolveModuleSource;
   options.sourceMap = true;
   options.plugins = [
     injectBabelHelpers,
-    [
-      'debug-macros',
-      {
-        debugTools: {
-          source: '@ember/debug',
-          assertPredicateIndex: 1,
-          isDebug,
-        },
-        externalizeHelpers: {
-          module: true,
-        },
-        flags: [
-          { source: '@glimmer/env', flags: { DEBUG: isDebug } },
-          {
-            source: '@ember/canary-features',
-            flags: Object.assign(
-              // explicit list of additional exports within @ember/canary-features
-              // without adding this (with a null value) an error is thrown during
-              // the feature replacement process (e.g. XYZ is not a supported flag)
-              {
-                FEATURES: null,
-                DEFAULT_FEATURES: null,
-                isEnabled: null,
-              },
-              FEATURES
-            ),
-          },
-        ],
-      },
-    ],
     ['transform-es2015-template-literals', { loose: true }],
     ['transform-es2015-literals'],
     ['transform-es2015-arrow-functions'],
@@ -65,43 +21,12 @@ module.exports = function toES5(tree, _options) {
     ['check-es2015-constants'],
     ['transform-es2015-classes', { loose: true }],
     ['transform-object-assign'],
-    injectNodeGlobals,
-    ['transform-es2015-modules-amd', { noInterop: true, strict: true }],
-    enifed,
   ];
 
-  if (options.transformDefine) {
-    options.plugins = [enifed];
-    delete options.transformDefine;
-  }
-
-  if (options.transformModules === false) {
-    options.plugins.pop();
-    options.plugins.pop();
-    delete options.moduleIds;
-    delete options.resolveModuleSource;
-    delete options.transformModules;
-  }
-
   if (options.inlineHelpers) {
     options.plugins.shift();
     delete options.inlineHelpers;
   }
 
-  delete options.environment;
-
   return new Babel(tree, options);
 };
-
-function stripForProd(tree) {
-  let options = {
-    plugins: [
-      [stripClassCallCheck, { source: 'ember-babel' }],
-      ['minify-dead-code-elimination', { optimizeRawSize: true }],
-    ],
-  };
-
-  return new Babel(tree, options);
-}
-
-module.exports.stripForProd = stripForProd;
diff --git a/broccoli/to-named-amd.js b/broccoli/to-named-amd.js
@@ -3,18 +3,28 @@ const resolveModuleSource = require('amd-name-resolver').moduleResolve;
 const enifed = require('./transforms/transform-define');
 const injectNodeGlobals = require('./transforms/inject-node-globals');
 
-module.exports = function processModulesOnly(tree, annotation) {
+module.exports = function processModulesOnly(tree, strict = false) {
+  let transformOptions = { noInterop: true };
+
+  // These options need to be exclusive for some reason, even the key existing
+  // on the options hash causes issues.
+  if (strict) {
+    transformOptions.strict = true;
+  } else {
+    transformOptions.loose = true;
+  }
+
   let options = {
+    sourceMap: true,
     plugins: [
       // ensures `@glimmer/compiler` requiring `crypto` works properly
       // in both browser and node-land
       injectNodeGlobals,
-      ['transform-es2015-modules-amd', { loose: true, noInterop: true }],
+      ['transform-es2015-modules-amd', transformOptions],
       enifed,
     ],
     moduleIds: true,
     resolveModuleSource,
-    annotation,
   };
 
   return new Babel(tree, options);

diff --git a/broccoli/transforms/strip-class-call-check.js b/broccoli/transforms/strip-class-call-check.js