Skip to content

Commit 6d87ebf

Browse files
author
Thibault Wittemberg
committed
converter: fix a classes container overwrite
1 parent 8d3965c commit 6d87ebf

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

Sources/Core/Converters/CoberturaXmlConverter.swift

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,9 @@ public extension Xccov.Converters.CoberturaXml {
6262
var currentPackageElement: XMLElement!
6363
var isNewPackage = false
6464

65+
var currentClassesElement = XMLElement()
66+
67+
6568
allFiles.forEach { fileCoverageReport in
6669
// Define file path relative to source!
6770
let filePath = fileCoverageReport.path.replacingOccurrences(of: currentDirectoryPath + "/", with: "")
@@ -72,11 +75,10 @@ public extension Xccov.Converters.CoberturaXml {
7275

7376
if isNewPackage {
7477
currentPackageElement = XMLElement(name: "package")
78+
currentClassesElement = XMLElement()
7579
packagesElement.addChild(currentPackageElement)
7680
}
7781

78-
var currentClassesElement = XMLElement()
79-
8082
currentPackage = packageName
8183
if isNewPackage {
8284
currentPackageElement.addAttribute(XMLNode.nodeAttribute(withName: "name", stringValue: packageName))
@@ -95,7 +97,6 @@ public extension Xccov.Converters.CoberturaXml {
9597
classElement.addAttribute(XMLNode.nodeAttribute(withName: "branch-rate", stringValue: "1.0"))
9698
classElement.addAttribute(XMLNode.nodeAttribute(withName: "complexity", stringValue: "0.0"))
9799
currentClassesElement.addChild(classElement)
98-
// currentPackageElement.addChild(classElement)
99100

100101
let linesElement = XMLElement(name: "lines")
101102
classElement.addChild(linesElement)

0 commit comments

Comments
 (0)