Skip to content

Commit

Permalink
fix(reporter): replace colons in the output path
Browse files Browse the repository at this point in the history
change browser name used to generate output path, remove instances of `:` to make it compatible with windows

BREAKING CHANGE: the output folder names change, they no longer contain `:`
  • Loading branch information
itsthatianguy authored and dignifiedquire committed Aug 27, 2016
1 parent 1a876d5 commit 3b2bffa
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion lib/reporter.js
Expand Up @@ -266,7 +266,8 @@ var CoverageReporter = function (rootConfig, helper, logger, emitter) {

var mainDir = reporterConfig.dir || config.dir
var subDir = reporterConfig.subdir || config.subdir
var simpleOutputDir = generateOutputDir(browser.name, mainDir, subDir)
var browserName = browser.name.replace(':', '')
var simpleOutputDir = generateOutputDir(browserName, mainDir, subDir)
var resolvedOutputDir = path.resolve(basePath, simpleOutputDir)

var outputDir = helper.normalizeWinPath(resolvedOutputDir)
Expand Down

0 comments on commit 3b2bffa

Please sign in to comment.