fix: Setting default value for config in runner and stopper

Default config to empty Object if undefined.
karma-runner · Feb 21, 2016 · 414db89 · 414db89
1 parent f10fd81
commit 414db89
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/lib/runner.js b/lib/runner.js
@@ -31,6 +31,10 @@ var parseExitCode = function (buffer, defaultCode, failOnEmptyTestSuite) {
 
 // TODO(vojta): read config file (port, host, urlRoot)
 exports.run = function (config, done) {
+  config = config || {}
+
+  logger.setupFromConfig(config)
+
   done = helper.isFunction(done) ? done : process.exit
   config = cfg.parseConfig(config.configFile, config)
 
@@ -71,6 +75,7 @@ exports.run = function (config, done) {
     removedFiles: config.removedFiles,
     changedFiles: config.changedFiles,
     addedFiles: config.addedFiles,
-    refresh: config.refresh
+    refresh: config.refresh,
+    colors: config.colors
   }))
 }
diff --git a/lib/stopper.js b/lib/stopper.js
@@ -5,6 +5,7 @@ var logger = require('./logger')
 var helper = require('./helper')
 
 exports.stop = function (config, done) {
+  config = config || {}
   logger.setupFromConfig(config)
   done = helper.isFunction(done) ? done : process.exit
   var log = logger.create('stopper')