wip karma config for webstorm support. More work is needed to get files and the baseUrl to work in all three scenarios.

Author: Chris Decoster

lib/test/karma.js

@@ -9,18 +9,8 @@ var gulp            = require('gulp'),
     childProcess    = require('child_process');
 
 var yargs           = require('../util/yargs'),
-    streams         = require('../config/streams'),
     platform        = require('../config/platform');
 
-/**
- * Wrap the given value in quotation marks
- * @param {*} value The value to wrap
- * @returns {string} The string representation of the value in quotation marks
- */
-function quote(value) {
-  return '"' + value + '"';
-}
-
 var defaultReporterName = 'karma-angularity-reporter';
 
 var filesAppendRegex = /\/\*+\s*ANGULARITY_FILE_LIST\s*\*+\// ;
@@ -95,7 +85,7 @@ function getKarmaReporterName(reporterName) {
   if (!!match && match.length === 2) {
     name = match[1];
   }
-  return name
+  return name;
 }
 
 /**
@@ -118,25 +108,33 @@ function karmaCreateConfig(reporters, configFileName) {
   configFileName = configFileName || 'karma.conf.js';
   var files = [];
 
+  files.push('**/!(app|app-build|app-release|app-test)/*.js');
+
+  files.push({
+    'pattern': '**/*.map',
+    'included': false
+  });
+
+  files.push({
+    'pattern': '**/*.spec.js',
+    'included': false
+  });
+
+  // add the bundles
   function transformFn(file, encoding, transformDone) {
     if (!file || !file.path) {
       throw 'Files must have paths';
     }
     if (path.extname(file.path) === '.js') {
       files.push(file.path);
     }
-    else {
-      //non-javascript files, such as source maps, should be included but not tested
-      files.push({
-        pattern: file.path,
-        included: false,
-      });
-    }
     transformDone();
   }
 
   function flushFn(flushDone) {
+    /*jshint validthis:true */
     var stream = this;
+    constructFile();
 
     function constructFile() {
       var filesAppend = JSON.stringify(files, null, '  ');
@@ -160,28 +158,7 @@ function karmaCreateConfig(reporters, configFileName) {
         .on('end', function() {
           flushDone();
         });
-      }
-
-    function specTransformFn(file, encoding, specFileDone) {
-      files.push({
-        pattern: file.path,
-        included: false,
-      });
-      specFileDone();
-    }
-
-    function specFlushFn(specFlushDone) {
-      //delay construction of file until all *.spec.js files have been
-      //added to the files array
-      constructFile();
-      specFlushDone();
     }
-
-    //obtain a list of *.spec.js files (file contents skipped)
-    //and add them to the list of files, with `included: false`,
-    //prior to writing to file
-    streams.jsSpec({ read: false })
-      .pipe(through.obj(specTransformFn, specFlushFn));
   }
 
   return through.obj(transformFn, flushFn);
@@ -193,10 +170,9 @@ function karmaCreateConfig(reporters, configFileName) {
  * Runs karma in a child process, to avoid `process.exit()` called by karma.
  *
  * @param {Array.<string>} [reporters] The name of the karma reporter to use
- * @param {number} [bannerWidth] The width of banner comment, zero or omitted for none
  * @returns {stream.Through} A through stream intended to have a gulp stream piped through it
  */
-function karmaRun(reporters, bannerWidth) {
+function karmaRun(reporters) {
   var options = {
     configFile: undefined
   };
@@ -222,12 +198,12 @@ function karmaRun(reporters, bannerWidth) {
         SUPPORTS_COLOR: true,
       }),
     });
-    karmaBackground.on('close', function(exitCode) {
+    karmaBackground.on('close', function() {
       done();
       return;
     });
   });
-};
+}
 
 var yargsOptionDefiniton = {
   key: 'karmareporter',