From 0fbe6dde8c17332fcb421f37c8f22f3b0d31a2a4 Mon Sep 17 00:00:00 2001 From: Paul Gschwendtner Date: Mon, 7 Jan 2019 05:26:01 +0100 Subject: [PATCH] build: do not load tslib and moment twice in tests (#14715) * SystemJS is responsible for loading `tslib` and `moment` as registered AMD modules. This means that we don't need to include them in Karma as well. Currently the browsers fetches Moment and tslib twice and also runs it without having any necessary effect. --- test/karma.conf.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/karma.conf.js b/test/karma.conf.js index 024881036c7..816937b3f7c 100644 --- a/test/karma.conf.js +++ b/test/karma.conf.js @@ -15,7 +15,7 @@ module.exports = config => { ], files: [ {pattern: 'node_modules/core-js/client/core.min.js', included: true, watched: false}, - {pattern: 'node_modules/tslib/tslib.js', included: true, watched: false}, + {pattern: 'node_modules/tslib/tslib.js', included: false, watched: false}, {pattern: 'node_modules/systemjs/dist/system.js', included: true, watched: false}, {pattern: 'node_modules/zone.js/dist/zone.min.js', included: true, watched: false}, {pattern: 'node_modules/zone.js/dist/proxy.min.js', included: true, watched: false}, @@ -24,7 +24,7 @@ module.exports = config => { {pattern: 'node_modules/zone.js/dist/async-test.js', included: true, watched: false}, {pattern: 'node_modules/zone.js/dist/fake-async-test.js', included: true, watched: false}, {pattern: 'node_modules/hammerjs/hammer.min.js', included: true, watched: false}, - {pattern: 'node_modules/moment/min/moment-with-locales.min.js', included: true, watched: false}, + {pattern: 'node_modules/moment/min/moment-with-locales.min.js', included: false, watched: false}, // Include all Angular dependencies {pattern: 'node_modules/@angular/**/*', included: false, watched: false},