fix: update to work with Karma 0.11

Since Karma executes immediately, we don't have all the browsers in advance, the reporter has to listen on browser_start instead.
karma-runner · Aug 25, 2013 · b744d6f · b744d6f
1 parent 16bd46f
commit b744d6f
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/lib/reporter.js b/lib/reporter.js
@@ -63,11 +63,12 @@ var CoverageReporter = function(rootConfig, emitter, helper, logger) {
     }
   }
 
-  this.onRunStart = function(browsers) {
-    collectors = {};
-    browsers.forEach(function(browser) {
-      collectors[browser.id] = new istanbul.Collector();
-    });
+  this.onRunStart = function() {
+    collectors = Object.create(null);
+  };
+
+  this.onBrowserStart = function(browser) {
+    collectors[browser.id] = new istanbul.Collector();
   };
 
   this.onBrowserComplete = function(browser, result) {

diff --git a/test/reporter.spec.coffee b/test/reporter.spec.coffee
@@ -118,7 +118,8 @@ describe 'reporter', ->
       fakeOpera = makeBrowser 'bbb', 'Opera/9.80 Mac OS X Version/12.00', browsers, emitter
       browsers.add fakeChrome
       browsers.add fakeOpera
-      reporter.onRunStart browsers
+      reporter.onRunStart()
+      browsers.forEach (b) -> reporter.onBrowserStart b
       mockFs.writeFile.reset()
       mockMkdir.reset()