chore(plugins): refactor logging from plugins

Make it easier to swap out for a new way of printing results, if desired.

Author: juliemr

Diff for: lib/plugins.js

@@ -1,6 +1,7 @@
 var q = require('q'),
     helper = require('./util'),
-    ConfigParser = require('./configParser');
+    ConfigParser = require('./configParser'),
+    log = require('./logger');
 
 var logPrefix = '[plugins]';
 
@@ -11,7 +12,7 @@ var logPrefix = '[plugins]';
  */
 var log_ = function() {
   var args = [logPrefix].concat([].slice.call(arguments));
-  console.log.apply(console, args);
+  log.puts.apply(log, args);
 };
 
 /**
@@ -42,16 +43,36 @@ var Plugins = function(config) {
   });
 };
 
-var green = '\x1b[32m';
-var red = '\x1b[31m';
-var normalColor = '\x1b[39m';
+var noop = function() {};
 
-var printResult = function(message, pass) {
-  console.log(pass ? green : red,
-      '\t', pass ? 'Pass: ' : 'Fail: ', message, normalColor);
-};
+Plugins.printPluginResults = function(specResults) {
+  var green = '\x1b[32m';
+  var red = '\x1b[31m';
+  var normalColor = '\x1b[39m';
 
-var noop = function() {};
+  var printResult = function(message, pass) {
+    log.puts(pass ? green : red,
+        '\t', pass ? 'Pass: ' : 'Fail: ', message, normalColor);
+  };
+
+  for (var j = 0; j < specResults.length; j++) {
+    var specResult = specResults[j];
+    var passed = specResult.assertions.map(function(x) {
+      return x.passed;
+    }).reduce(function(x, y) {
+      return x && y;
+    }, true);
+
+    printResult(specResult.description, passed);
+    if (!passed) {
+      for (var k = 0; k < specResult.assertions.length; k++) {
+        if (!specResult.assertions[k].passed) {
+          log.puts('\t\t' + specResult.assertions[k].errorMsg);
+        }
+      }
+    }
+  }
+}
 
 function pluginFunFactory(funName) {
   return function() {
@@ -81,25 +102,8 @@ function pluginFunFactory(funName) {
 
           // Output test results
           if (pluginResult.specResults) {
-            console.log('Plugin: ' + names[i] + ' (' + funName + ')');
-            for (var j = 0; j < pluginResult.specResults.length; j++) {
-              var specResult = pluginResult.specResults[j];
-              var passed = specResult.assertions.map(function(x) {
-                return x.passed;
-              }).reduce(function(x, y) {
-                return x && y;
-              }, true);
-
-
-              printResult(specResult.description, passed);
-              if (!passed) {
-                for (var k = 0; k < specResult.assertions.length; k++) {
-                  if (!specResult.assertions[k].passed) {
-                    console.log('\t\t' + specResult.assertions[k].errorMsg);
-                  }
-                }
-              }
-            }
+            log.puts('Plugin: ' + names[i] + ' (' + funName + ')');
+            Plugins.printPluginResults(pluginResult.specResults);
           }
 
           // Join objects