fix: Run tests on both the build as well as the dist folder (#514)

.travis.yml

@@ -22,6 +22,7 @@ before_script:
   - ./scripts/sauce/sauce_connect_block.sh
 
 script:
-  - node_modules/.bin/karma start karma-sauce-jasmine.conf.js --single-run
-  - node_modules/.bin/karma start karma-sauce-mocha.conf.js --single-run
+  - node_modules/.bin/gulp build
+  - node_modules/.bin/karma start karma-dist-sauce-jasmine.conf.js --single-run
+  - node_modules/.bin/karma start karma-build-sauce-mocha.conf.js --single-run
   - node_modules/.bin/gulp test/node

karma.conf.js → karma-base.conf.js

@@ -15,10 +15,7 @@ module.exports = function (config) {
       'node_modules/whatwg-fetch/fetch.js',
       {pattern: 'test/assets/**/*.*', watched: true, served: true, included: false},
       {pattern: 'build/**/*.js.map', watched: true, served: true, included: false},
-      {pattern: 'build/**/*.js', watched: true, served: true, included: false},
-      'build/test/wtf_mock.js',
-      'build/lib/zone.js',
-      'build/test/main.js'
+      {pattern: 'build/**/*.js', watched: true, served: true, included: false}
     ],
 
     plugins: [

karma-build-jasmine.conf.js

@@ -0,0 +1,7 @@
+
+module.exports = function (config) {
+  require('./karma-build.conf.js')(config);
+
+  config.plugins.push(require('karma-jasmine'));
+  config.frameworks.push('jasmine');
+};

karma-build-mocha.conf.js

@@ -0,0 +1,10 @@
+
+module.exports = function (config) {
+  require('./karma-build.conf.js')(config);
+
+  config.plugins.push(require('karma-mocha'));
+  config.frameworks.push('mocha');
+  config.client.mocha = {
+    timeout: 5000 // copied timeout for Jasmine in WebSocket.spec (otherwise Mochas default timeout at 2 sec is to low for the tests)
+  };
+};

karma-sauce-mocha.conf.js → karma-build-sauce-mocha.conf.js

@@ -7,7 +7,6 @@
  */
 
 module.exports = function (config) {
-  require('./karma.conf')(config);
-  require('./karma-mocha.conf')(config);
+  require('./karma-dist-mocha.conf.js')(config);
   require('./sauce.conf')(config);
 };

karma-build.conf.js

@@ -0,0 +1,14 @@
+/**
+ * @license
+ * Copyright Google Inc. All Rights Reserved.
+ *
+ * Use of this source code is governed by an MIT-style license that can be
+ * found in the LICENSE file at https://angular.io/license
+ */
+
+module.exports = function (config) {
+  require('./karma-base.conf.js')(config);
+  config.files.push('build/test/wtf_mock.js');
+  config.files.push('build/lib/zone.js');
+  config.files.push('build/test/main.js');
+};

karma-jasmine.conf.js → karma-dist-jasmine.conf.js

@@ -1,6 +1,6 @@
 
 module.exports = function (config) {
-  require('./karma.conf')(config);
+  require('./karma-dist.conf.js')(config);
 
   config.plugins.push(require('karma-jasmine'));
   config.frameworks.push('jasmine');

karma-mocha.conf.js → karma-dist-mocha.conf.js

@@ -1,6 +1,6 @@
 
 module.exports = function (config) {
-  require('./karma.conf')(config);
+  require('./karma-dist.conf.js')(config);
 
   config.plugins.push(require('karma-mocha'));
   config.frameworks.push('mocha');

karma-sauce-jasmine.conf.js → karma-dist-sauce-jasmine.conf.js

@@ -7,7 +7,6 @@
  */
 
 module.exports = function (config) {
-  require('./karma.conf')(config);
-  require('./karma-jasmine.conf')(config);
+  require('./karma-dist-jasmine.conf.js')(config);
   require('./sauce.conf')(config);
 };

karma-dist.conf.js

@@ -0,0 +1,21 @@
+/**
+ * @license
+ * Copyright Google Inc. All Rights Reserved.
+ *
+ * Use of this source code is governed by an MIT-style license that can be
+ * found in the LICENSE file at https://angular.io/license
+ */
+
+module.exports = function (config) {
+  require('./karma-base.conf.js')(config);
+  config.files.push('build/test/wtf_mock.js');
+  config.files.push('dist/zone.js');
+  config.files.push('dist/async-test.js');
+  config.files.push('dist/fake-async-test.js');
+  config.files.push('dist/long-stack-trace-zone.js');
+  config.files.push('dist/proxy.js');
+  config.files.push('dist/sync-test.js');
+  config.files.push('dist/task-tracking.js');
+  config.files.push('dist/wtf.js');
+  config.files.push('build/test/main.js');
+};

package.json

@@ -19,9 +19,10 @@
     "ws-server": "node ./test/ws-server.js",
     "tsc": "tsc",
     "tsc:w": "tsc -w",
-    "test": "npm run tsc && concurrently \"npm run tsc:w\" \"npm run ws-server\" \"karma start karma-jasmine.conf.js\"",
+    "test": "npm run tsc && concurrently \"npm run tsc:w\" \"npm run ws-server\" \"karma start karma-build-jasmine.conf.js\"",
+    "test-dist": "concurrently \"npm run tsc:w\" \"npm run ws-server\" \"karma start karma-dist-jasmine.conf.js\"",
     "test-node": "gulp test/node",
-    "test-mocha": "npm run tsc && concurrently \"npm run tsc:w\" \"npm run ws-server\" \"karma start karma-mocha.conf.js\"",
+    "test-mocha": "npm run tsc && concurrently \"npm run tsc:w\" \"npm run ws-server\" \"karma start karma-build-mocha.conf.js\"",
     "serve": "python -m SimpleHTTPServer 8000"
   },
   "repository": {

test/main.ts

@@ -15,9 +15,17 @@ declare var __karma__: {
 __karma__.loaded = function() {};
 (window as any).global = window;
 
-System.config({defaultJSExtensions: true});
+System.config({ defaultJSExtensions: true });
+var browserPatchedPromise = null;
+if (window[Zone['__symbol__']('setTimeout')]) {
+  browserPatchedPromise = Promise.resolve('browserPatched');
+} else {
+  // this means that Zone has not patched the browser yet, which means we must be running in
+  // build mode and need to load the browser patch.
+  browserPatchedPromise = System.import('/base/build/test/browser-zone-setup');
+}
 
-System.import('/base/build/test/browser-zone-setup').then(() => {
+browserPatchedPromise.then(() => {
   let testFrameworkPatch = typeof(window as any).Mocha !== 'undefined' ?
       '/base/build/test/test-env-setup-mocha' :
       '/base/build/test/test-env-setup-jasmine';