fix: all browsers should have their own output folder when not combined

Closes #39
mattlewis92 · Mar 6, 2018 · 4aad40b · 4aad40b
1 parent fccc472
commit 4aad40b
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/src/reporter.js b/src/reporter.js
@@ -60,12 +60,16 @@ function CoverageIstanbulReporter(baseReporterDecorator, logger, config) {
   }
 
   function createReport(browserOrBrowsers, results) {
-    if (!coverageConfig.combineBrowserReports && coverageConfig.dir) {
-      coverageConfig.dir = coverageConfig.dir.replace(BROWSER_PLACEHOLDER, browserOrBrowsers.name);
-    }
+    const reportConfigOverride = (!coverageConfig.combineBrowserReports && coverageConfig.dir) ?
+      {dir: coverageConfig.dir.replace(BROWSER_PLACEHOLDER, browserOrBrowsers.name)} :
+      {};
 
     const reportConfig = istanbul.config.loadObject({
-      reporting: coverageConfig
+      reporting: Object.assign(
+        {},
+        coverageConfig,
+        reportConfigOverride
+      )
     });
     const reportTypes = reportConfig.reporting.config.reports;