Skip to content

Commit a2dec5e

Browse files
authored
fix: Some more small fixes on relative paths in the writers (#403)
1 parent 4fe478c commit a2dec5e

File tree

3 files changed

+5
-3
lines changed

3 files changed

+5
-3
lines changed

scalac-scoverage-plugin/src/main/scala/scoverage/report/BaseReportWriter.scala

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,9 @@ class BaseReportWriter(
1010

1111
// Source paths in canonical form WITH trailing file separator
1212
private val formattedSourcePaths: Seq[String] =
13-
sourceDirectories filter (_.isDirectory) map (_.getCanonicalPath + File.separator)
13+
sourceDirectories
14+
.filter(_.isDirectory)
15+
.map(_.getCanonicalPath + File.separator)
1416

1517
/** Converts absolute path to relative one if any of the source directories is it's parent.
1618
* If there is no parent directory, the path is returned unchanged (absolute).

scalac-scoverage-plugin/src/main/scala/scoverage/report/CoberturaXmlWriter.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ class CoberturaXmlWriter(
4747

4848
def klass(klass: MeasuredClass): Node = {
4949
<class name={klass.fullClassName}
50-
filename={relativeSource(klass.source).replace(File.separator, "/")}
50+
filename={klass.source}
5151
line-rate={twoFractionDigits(klass.statementCoverage)}
5252
branch-rate={twoFractionDigits(klass.branchCoverage)}
5353
complexity="0">

scalac-scoverage-plugin/src/main/scala/scoverage/report/ScoverageXmlWriter.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ class ScoverageXmlWriter(
9191

9292
private def klass(klass: MeasuredClass): Node = {
9393
<class name={klass.fullClassName}
94-
filename={relativeSource(klass.source)}
94+
filename={klass.source}
9595
statement-count={klass.statementCount.toString}
9696
statements-invoked={klass.invokedStatementCount.toString}
9797
statement-rate={klass.statementCoverageFormatted}

0 commit comments

Comments
 (0)