Merge pull request #124 from piecyk/fix-browserify

fix: when using browserify dont create source code caching
karma-runner · Apr 23, 2015 · 26b453e · 26b453e
2 parents d37cf0b + 50030df
commit 26b453e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lib/reporter.js b/lib/reporter.js
@@ -133,10 +133,10 @@ var CoverageReporter = function(rootConfig, helper, logger) {
           var outputDir = helper.normalizeWinPath(path.resolve(basePath, generateOutputDir(browser.name,
                                                                                            reporterConfig.dir || config.dir,
                                                                                            reporterConfig.subdir || config.subdir)));
-
+          var _ = helper._;
           var options = helper.merge({}, reporterConfig, {
             dir : outputDir,
-            sourceStore : new SourceCacheStore({
+            sourceStore : _.isEmpty(sourceCache) ? null : new SourceCacheStore({
               sourceCache: sourceCache
             })
           });

diff --git a/test/reporter.spec.coffee b/test/reporter.spec.coffee
@@ -34,6 +34,7 @@ describe 'reporter', ->
   mockReportCreate = sinon.stub().returns writeReport: mockWriteReport
   mockMkdir = sinon.spy()
   mockHelper =
+    _: helper._
     isDefined: (v) -> helper.isDefined v
     merge: (v...) -> helper.merge v...
     mkdirIfNotExists: mockMkdir