Log messages "Written ... report" after report writing, not before.

gslowikowski · gslowikowski · commit 873471a90475 · 2017-08-14T21:39:01.000+02:00
diff --git a/src/main/scala/scoverage/ScoverageSbtPlugin.scala b/src/main/scala/scoverage/ScoverageSbtPlugin.scala
@@ -174,25 +174,25 @@ object ScoverageSbtPlugin extends AutoPlugin {
     reportDir.mkdirs()
 
     if (coverageOutputCobertura) {
-      log.info(s"Written Cobertura report [${coberturaDir.getAbsolutePath}/cobertura.xml]")
       new CoberturaXmlWriter(compileSourceDirectories, coberturaDir).write(coverage)
+      log.info(s"Written Cobertura report [${coberturaDir.getAbsolutePath}/cobertura.xml]")
     }
 
     if (coverageOutputXML) {
-      log.info(s"Written XML coverage report [${reportDir.getAbsolutePath}/scoverage.xml]")
       new ScoverageXmlWriter(compileSourceDirectories, reportDir, false).write(coverage)
       if (coverageDebug) {
         new ScoverageXmlWriter(compileSourceDirectories, reportDir, true).write(coverage)
       }
+      log.info(s"Written XML coverage report [${reportDir.getAbsolutePath}/scoverage.xml]")
     }
 
     if (coverageOutputHTML) {
-      log.info(s"Written HTML coverage report [${reportDir.getAbsolutePath}/index.html]")
       new ScoverageHtmlWriter(compileSourceDirectories, reportDir, coverageSourceEncoding).write(coverage)
+      log.info(s"Written HTML coverage report [${reportDir.getAbsolutePath}/index.html]")
     }
     if (coverageOutputTeamCity) {
-      log.info("Writing coverage report to teamcity")
       reportToTeamcity(coverage, coverageOutputHTML, reportDir, crossTarget, log)
+      log.info("Written coverage report to TeamCity")
     }
 
     log.info(s"Statement coverage.: ${coverage.statementCoverageFormatted}%")

Original file line number	Diff line number	Diff line change
`@@ -174,25 +174,25 @@ object ScoverageSbtPlugin extends AutoPlugin {`
`174`	`174`	`reportDir.mkdirs()`
`175`	`175`
`176`	`176`	`if (coverageOutputCobertura) {`
`177`		`- log.info(s"Written Cobertura report [${coberturaDir.getAbsolutePath}/cobertura.xml]")`
`178`	`177`	`new CoberturaXmlWriter(compileSourceDirectories, coberturaDir).write(coverage)`
	`178`	`+ log.info(s"Written Cobertura report [${coberturaDir.getAbsolutePath}/cobertura.xml]")`
`179`	`179`	`}`
`180`	`180`
`181`	`181`	`if (coverageOutputXML) {`
`182`		`- log.info(s"Written XML coverage report [${reportDir.getAbsolutePath}/scoverage.xml]")`
`183`	`182`	`new ScoverageXmlWriter(compileSourceDirectories, reportDir, false).write(coverage)`
`184`	`183`	`if (coverageDebug) {`
`185`	`184`	`new ScoverageXmlWriter(compileSourceDirectories, reportDir, true).write(coverage)`
`186`	`185`	`}`
	`186`	`+ log.info(s"Written XML coverage report [${reportDir.getAbsolutePath}/scoverage.xml]")`
`187`	`187`	`}`
`188`	`188`
`189`	`189`	`if (coverageOutputHTML) {`
`190`		`- log.info(s"Written HTML coverage report [${reportDir.getAbsolutePath}/index.html]")`
`191`	`190`	`new ScoverageHtmlWriter(compileSourceDirectories, reportDir, coverageSourceEncoding).write(coverage)`
	`191`	`+ log.info(s"Written HTML coverage report [${reportDir.getAbsolutePath}/index.html]")`
`192`	`192`	`}`
`193`	`193`	`if (coverageOutputTeamCity) {`
`194`		`- log.info("Writing coverage report to teamcity")`
`195`	`194`	`reportToTeamcity(coverage, coverageOutputHTML, reportDir, crossTarget, log)`
	`195`	`+ log.info("Written coverage report to TeamCity")`
`196`	`196`	`}`
`197`	`197`
`198`	`198`	`log.info(s"Statement coverage.: ${coverage.statementCoverageFormatted}%")`