[BUGFIX beta][FEAT] Adds a second dist build which targets IE

This PR adds a second dist to the build which targets IE. This allows
the primary bundles to drop ES5 compilation from babel, and allows us to
choose which bundles we want to use based on the consuming application's
targets.

In the long run, we want to actually enable consuming apps to build the
Ember source code themselves. This is a temporary in-between measure to
address a bug in native classes, in which users who do _not_ have IE as
a build target cannot `.extend` from native classes due to conflicts
in transpilation vs actual native class syntax.

.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

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() {

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';
     },

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));

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 });
+};

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);
 }
 

broccoli/strip-for-prod.js

@@ -0,0 +1,13 @@
+'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);
+};

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;

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);