From 8efcec67ccef8d37a474427dd8cf8d9d39ffcf72 Mon Sep 17 00:00:00 2001 From: Vojta Jina Date: Fri, 28 Jun 2013 16:35:05 -0700 Subject: chore: clean up angularFiles.js --- angularFiles.js | 59 +++++++++++++++++---------------------------------- karma-jqlite.conf.js | 4 ++-- karma-jquery.conf.js | 4 ++-- karma-modules.conf.js | 2 +- 4 files changed, 24 insertions(+), 45 deletions(-) diff --git a/angularFiles.js b/angularFiles.js index 865d7040..e2622188 100644 --- a/angularFiles.js +++ b/angularFiles.js @@ -108,7 +108,7 @@ angularFiles = { 'test/ngMock/*.js' ], - 'jstd': [ + 'karma': [ 'lib/jquery/jquery.js', 'test/jquery_remove.js', '@angularSrc', @@ -120,18 +120,18 @@ angularFiles = { 'example/personalLog/test/*.js' ], - 'jstdExclude': [ + 'karmaExclude': [ 'test/jquery_alias.js', 'src/angular-bootstrap.js', 'src/ngScenario/angular-bootstrap.js' ], - 'jstdScenario': [ + 'karmaScenario': [ 'build/angular-scenario.js', 'build/docs/docs-scenario.js' ], - "jstdModules": [ + "karmaModules": [ 'build/angular.js', 'src/ngMock/angular-mocks.js', 'src/ngCookies/cookies.js', @@ -148,21 +148,7 @@ angularFiles = { 'test/ngSanitize/filter/*.js' ], - 'jstdPerf': [ - '@angularSrc', - '@angularSrcModules', - 'src/ngMock/angular-mocks.js', - 'perf/data/*.js', - 'perf/testUtils.js', - 'perf/*.js' - ], - - 'jstdPerfExclude': [ - 'src/ng/angular-bootstrap.js', - 'src/ngScenario/angular-bootstrap.js' - ], - - 'jstdJquery': [ + 'karmaJquery': [ 'lib/jquery/jquery.js', 'test/jquery_alias.js', '@angularSrc', @@ -175,7 +161,7 @@ angularFiles = { 'example/personalLog/test/*.js' ], - 'jstdJqueryExclude': [ + 'karmaJqueryExclude': [ 'src/angular-bootstrap.js', 'src/ngScenario/angular-bootstrap.js', 'test/jquery_remove.js' @@ -183,29 +169,22 @@ angularFiles = { }; if (exports) { - exports.files = angularFiles - exports.mergeFiles = function mergeFiles() { + exports.files = angularFiles; + exports.mergeFilesFor = function() { var files = []; - [].splice.call(arguments, 0).forEach(function(file) { - if (file.match(/karma/)) { - files.push(file); - } else { - angularFiles[file].forEach(function(f) { - // replace @ref - var match = f.match(/^\@(.*)/); - if (match) { - var deps = angularFiles[match[1]]; - files = files.concat(deps); - } else { - if (!/jstd|jasmine/.test(f)) { //TODO(i): remove once we don't have jstd/jasmine in repo - files.push(f); - } - } - }); - } + Array.prototype.slice.call(arguments, 0).forEach(function(filegroup) { + angularFiles[filegroup].forEach(function(file) { + // replace @ref + var match = file.match(/^\@(.*)/); + if (match) { + files = files.concat(angularFiles[match[1]]); + } else { + files.push(file); + } + }); }); return files; - } + }; } diff --git a/karma-jqlite.conf.js b/karma-jqlite.conf.js index 9284f491..d3a297a8 100644 --- a/karma-jqlite.conf.js +++ b/karma-jqlite.conf.js @@ -5,8 +5,8 @@ module.exports = function(config) { sharedConfig(config); config.set({ - files: angularFiles.mergeFiles('jstd'), - exclude: angularFiles.files.jstdExclude, + files: angularFiles.mergeFilesFor('karma'), + exclude: angularFiles.mergeFilesFor('karmaExclude'), junitReporter: { outputFile: 'test_out/jqlite.xml', diff --git a/karma-jquery.conf.js b/karma-jquery.conf.js index 28f2fae2..236824f7 100644 --- a/karma-jquery.conf.js +++ b/karma-jquery.conf.js @@ -5,8 +5,8 @@ module.exports = function(config) { sharedConfig(config); config.set({ - files: angularFiles.mergeFiles('jstdJquery'), - exclude: angularFiles.files.jstdJqueryExclude, + files: angularFiles.mergeFilesFor('karmaJquery'), + exclude: angularFiles.mergeFilesFor('karmaJqueryExclude'), junitReporter: { outputFile: 'test_out/jquery.xml', diff --git a/karma-modules.conf.js b/karma-modules.conf.js index f4c28eb3..38c22b81 100644 --- a/karma-modules.conf.js +++ b/karma-modules.conf.js @@ -5,7 +5,7 @@ module.exports = function(config) { sharedConfig(config); config.set({ - files: angularFiles.mergeFiles('jstdModules', 'angularSrcModules'), + files: angularFiles.mergeFilesFor('karmaModules', 'angularSrcModules'), junitReporter: { outputFile: 'test_out/modules.xml', -- cgit v1.2.3