From 59436f7b452db5ad63f2d38b9c054b9a87fc26af Mon Sep 17 00:00:00 2001 From: Victor Woeltjen Date: Thu, 28 Jul 2016 10:45:38 -0700 Subject: [PATCH] [Tests] Load all sources (#1098) Load all source files when running Karma's test suite; this ensures that code coverage metrics are not invalidated by missing files. Fixes #1090. --- test-main.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/test-main.js b/test-main.js index 1615bf3193..c329407f93 100644 --- a/test-main.js +++ b/test-main.js @@ -23,13 +23,14 @@ /*global require,window*/ var allTestFiles = []; var TEST_REGEXP = /(Spec)\.js$/; +var SRC_REGEXP = /^\/base\/(src|platform).*\.js$/; var pathToModule = function(path) { return path.replace(/^\/base\//, '').replace(/\.js$/, ''); }; Object.keys(window.__karma__.files).forEach(function(file) { - if (TEST_REGEXP.test(file)) { + if (TEST_REGEXP.test(file) || SRC_REGEXP.test(file)) { // Normalize paths to RequireJS module names. allTestFiles.push(pathToModule(file)); } @@ -38,6 +39,11 @@ Object.keys(window.__karma__.files).forEach(function(file) { // Force es6-promise to load. allTestFiles.unshift('es6-promise'); +// Drop legacyRegistry, since it is at a different path by RequireJS config +allTestFiles = allTestFiles.filter(function (file) { + return file.indexOf('legacyRegistry') === -1; +}); + requirejs.config({ // Karma serves files from the basePath defined in karma.conf.js baseUrl: '/base',