=BG= karma reports can be specified using both absolute paths and project local installations

Author: bguiz

lib/test/karma.js

@@ -16,7 +16,68 @@ function quote(value) {
   return '"' + value + '"';
 }
 
+var defaultReporterName = 'angularity-karma-reporter';
 var filesAppendRegex = /\/\*+\s*ANGULARITY_FILE_LIST\s*\*+\// ;
+var reportersAppendRegex = /\/\*+\s*ANGULARITY_REPORTER_LIST\s*\*+\// ;
+var pluginsAppendRegex = /\/\*+\s*ANGULARITY_PLUGIN_LIST\s*\*+\// ;
+var karmaReporterMatchNameRegex = /^karma-(.+)-reporter$/ ;
+
+/**
+ * Determine the `require` path for a plugin.
+ *
+ * @param {string} reporterName Either a package name,
+ *   *OR* an absolute path, for a karma reporter.
+ * @return {string} In karma config file, `plugins` will be inline `require`d
+ *   using this return value.
+ */
+function getKarmaReporterPluginPath(reporterName) {
+  if (typeof reporterName !== 'string') {
+    throw 'Get Karma Reporter Plugin Path: Reporter name is unspecified';
+  }
+
+  if (reporterName === defaultReporterName) {
+    return defaultReporterName;
+  }
+  else {
+    var reporterPath = (path.dirname(reporterName) === '.') ?
+      path.resolve('node_modules', 'karma-'+reporterName+'-reporter') :
+      reporterName;
+    try {
+      require(reporterPath);
+    }
+    catch (ex) {
+      throw 'Get Karma Reporter Plugin Path: Attempt to require reporter from path '+reporterPath+' with no success.';
+    }
+    return reporterPath;
+  }
+}
+
+/**
+ * Determine the registered reporter name.
+ *
+ * @param {string} reporterName Either the registered reporter name,
+ *   *OR* an absolute path, for a karma reporter.
+ * @return {string} The registered reporter name for use in
+ *   the `reporters` section of the karma config file.
+ */
+function getKarmaReporterName(reporterName) {
+  if (typeof reporterName !== 'string') {
+    throw 'Get Karma Reporter Name: Reporter name is unspecified';
+  }
+
+  var name;
+  if (path.dirname(reporterName) === '.') {
+    name = reporterName;
+  }
+  else {
+    name = path.basename(reporterName);
+  }
+  var match = karmaReporterMatchNameRegex.exec(name);
+  if (!!match && match.length === 2) {
+    name = match[1];
+  }
+  return name
+}
 
 /**
  * Create a through2 object stream.
@@ -27,13 +88,15 @@ var filesAppendRegex = /\/\*+\s*ANGULARITY_FILE_LIST\s*\*+\// ;
  * The new karma config file is added to the stream,
  * All input `*.js` files are filtered out of the stream
  *
- * @param {string} configFileName The project local karma config file to augment
+ * @param {Array.<string>} [reporters] The name of the karma reporter to use
+ * @param {string} [configFileName] The project local karma config file to augment
  *   Defaults to "karma.conf.js"
  * @return {stream.Through} The output of this stream is expected to contain
  *   just one file: the augmented karma config file.
  */
-function karmaCreateConfig(configFileName) {
-  configFileName = 'karma.conf.js';
+function karmaCreateConfig(reporters, configFileName) {
+  reporters = reporters || [];
+  configFileName = configFileName || 'karma.conf.js';
   var files = [];
 
   function transformFn(file, encoding, done) {
@@ -47,13 +110,20 @@ function karmaCreateConfig(configFileName) {
   function flushFn(done) {
     var stream = this;
     var filesAppend = JSON.stringify(files, null, '  ');
+    var reportersAppend = JSON.stringify(reporters.map(getKarmaReporterName), null, '  ');
+    var pluginsAppend = '[\n' + reporters.map(function(reporter) {
+      return 'require("'+getKarmaReporterPluginPath(reporter)+'")';
+    }).join(',\n  ') + '\n]';
 
     //aggregate and append to karma.conf.js in the project root folder
     gulp
       .src(path.resolve(configFileName))
       .on('data', function(karmaConfigFile) {
         var contents = karmaConfigFile.contents.toString();
         contents = contents.replace(filesAppendRegex, filesAppend);
+        contents = contents.replace(reportersAppendRegex, reportersAppend);
+        contents = contents.replace(reportersAppendRegex, reportersAppend);
+        contents = contents.replace(pluginsAppendRegex, pluginsAppend);
         karmaConfigFile.contents = new Buffer(contents);
         stream.push(karmaConfigFile);
       })
@@ -70,13 +140,15 @@ function karmaCreateConfig(configFileName) {
  * No output. Ends when the Karma process ends.
  * 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 strconcateam that performs the operation of a gulp stream
  */
-function karmaRun(bannerWidth) {
+function karmaRun(reporters, bannerWidth) {
   var options = {
     configFile: undefined
   };
+  reporters = reporters || [];
 
   return through.obj(function transformFn(file, encoding, done) {
     options.configFile = file.path;
@@ -87,23 +159,32 @@ function karmaRun(bannerWidth) {
     var data    = querystring.escape(JSON.stringify(options));
     var command = [ 'node', quote(appPath), data ].join(' ');
 
-    //TODO @bguiz replace reporter function with a standard karma reporter,
-    //and extract it to own module
-    //perhaps extend the spec reporter to do what is being done here, instead of post processing its output here
-    //check if there is a webstorm/ teamcity reporter
     childProcess.exec(command, { cwd: process.cwd() }, function reporter(stderr, stdout) {
+      if (reporters.length > 0) {
+        if (stdout) {
+          process.stdout.write(stdout);
+        }
+        if (stderr) {
+          process.stderr.write(stderr);
+        }
+        done();
+        return;
+      }
+
+      //TODO @bguiz replace reporter function with a standard karma reporter,
+      //and extract it to own module
+
+      //default reporter by parsing the stdout and stderr of karma
       var report;
       if (stdout) {
-        console.log(stdout);
         report = stdout
           .replace(/^\s+/gm, '')                                  // remove leading whitespace
           .replace(/^(LOG.*\n$)/gm, options.silent ? '' : '$1')   // remove logging
           .replace(/at\s+null\..*/gm, '')                         // remove file reference
           .replace(/\n\n/gm, '\n')                                // consolidate consecutive line breaks
           .replace(/^\n|\n$/g, '');                               // remove leading and trailing line breaks overall
-      } else if (stderr) {
-        console.log(stderr);
-
+      }
+      else if (stderr) {
         var analysis = /$Error\:\s*(.*)$/mg.exec(stderr);
         report = analysis ? analysis[1] : stderr;
       }

lib/util/jshint-reporter.js

@@ -35,7 +35,7 @@ var yargsOptionDefiniton = {
 var jsHintReporter;
 function getJsHintReporter(reporterName) {
   if (typeof reporterName !== 'string') {
-    throw 'Reporter name is unspecified';
+    throw 'Get JsHint Reporter: Reporter name is unspecified';
   }
   if (jsHintReporter) {
     return jsHintReporter; //cached copy

tasks/javascript.js

@@ -113,6 +113,7 @@ gulp.task('javascript:lint', function () {
 
 // karma unit tests in local library only
 gulp.task('javascript:unit', function () {
+  var reporters = [];
   return combined.create()
     .append(
       streams
@@ -130,9 +131,9 @@ gulp.task('javascript:unit', function () {
         .pipe(gulp.dest(streams.TEST))
         .pipe(gulpFilter('*.js'))
     )
-    .pipe(karma.createConfig())
+    .pipe(karma.createConfig(reporters))
     .pipe(gulp.dest(streams.TEST))
-    .pipe(karma.run(80));
+    .pipe(karma.run(reporters, 80));
 });
 
 // give a single optimised javascript file in the build directory with source map for each

templates/angularity/karma.conf.js

@@ -19,9 +19,13 @@ module.exports = function(config) {
     // list of files to exclude
     exclude: [],
 
+    // register any plugins which  are not siblings of karma in angularity global installation
+    // and thus need to be registered manually
+    plugins: config.plugins.concat(/*ANGULARITY_PLUGIN_LIST*/),
+
     // use dots reporter, as travis terminal does not support escaping sequences
     // possible values: 'dots', 'progress', 'junit', 'teamcity'
-    reporter: [],
+    reporters: [].concat(/*ANGULARITY_REPORTER_LIST*/),
 
     // web server port
     port: <%= port %>,