diff --git a/reporter/src/main/scala/scoverage/reporter/CoberturaXmlWriter.scala b/reporter/src/main/scala/scoverage/reporter/CoberturaXmlWriter.scala index a743a139..e71ec786 100644 --- a/reporter/src/main/scala/scoverage/reporter/CoberturaXmlWriter.scala +++ b/reporter/src/main/scala/scoverage/reporter/CoberturaXmlWriter.scala @@ -26,7 +26,7 @@ class CoberturaXmlWriter( val file = new File(outputDir, "cobertura.xml") IOUtils.writeToFile( file, - "\n\n" + + "\n\n" + new PrettyPrinter(120, 4).format(xml(coverage)), sourceEncoding ) diff --git a/reporter/src/test/resources/scoverage/reporter/cobertura.sample.xml b/reporter/src/test/resources/scoverage/reporter/cobertura.sample.xml index 0ceecde6..797ca895 100644 --- a/reporter/src/test/resources/scoverage/reporter/cobertura.sample.xml +++ b/reporter/src/test/resources/scoverage/reporter/cobertura.sample.xml @@ -1,5 +1,5 @@ - + diff --git a/reporter/src/test/scala/scoverage/reporter/CoberturaXmlWriterTest.scala b/reporter/src/test/scala/scoverage/reporter/CoberturaXmlWriterTest.scala index d27582a2..d9dc1ef0 100644 --- a/reporter/src/test/scala/scoverage/reporter/CoberturaXmlWriterTest.scala +++ b/reporter/src/test/scala/scoverage/reporter/CoberturaXmlWriterTest.scala @@ -112,7 +112,7 @@ class CoberturaXmlWriterTest extends FunSuite { ) } - test("cobertura output validates".ignore) { + test("cobertura output validates") { val dir = tempDir() @@ -298,8 +298,6 @@ class CoberturaXmlWriterTest extends FunSuite { writer.write(coverage) val domFactory = DocumentBuilderFactory.newInstance() - // TODO set validating here is no longer working as of 2.13.10. Not fully sure why but I don't use - // cobertura, so if someone wants to fix this, please do. I'll move forward with this not to block the release. domFactory.setValidating(true) val builder = domFactory.newDocumentBuilder() builder.setErrorHandler(new ErrorHandler() {