🩹 [Refactor]: Implement XML report merging in workflow script to consolidate code coverage data into a single report

MariusStorhaug · MariusStorhaug · commit d197e574720a · 2025-03-02T23:25:28.000+01:00
diff --git a/.github/workflows/workflow.yml b/.github/workflows/workflow.yml
@@ -206,14 +206,62 @@ jobs:
             $runId = $env:GITHUB_RUN_ID
             gh run download $runId --repo $repo --pattern *CodeCoverage --dir .
             $files = Get-ChildItem -Path . -Recurse -File | Select-Object -ExpandProperty FullName | Sort-Object
-            $files | Out-String
-            $files | ForEach-Object {
-              $fileName = $_.Name
-              LogGroup " - $fileName" {
-                Get-Content -Path $_
-              }
+            # Load the first XML as the base report
+            [xml]$mergedReport = Get-Content -Path $files[0]
+
+            # Function to merge counters
+            function Merge-Counters($baseNode, $newNode) {
+                foreach ($newCounter in $newNode.counter) {
+                    $baseCounter = $baseNode.counter | Where-Object { $_.type -eq $newCounter.type }
+                    if ($baseCounter) {
+                        $baseCounter.missed = [int]$baseCounter.missed + [int]$newCounter.missed
+                        $baseCounter.covered = [int]$baseCounter.covered + [int]$newCounter.covered
+                    } else {
+                        # Import new counter if it doesn't exist
+                        $importedCounter = $mergedReport.ImportNode($newCounter, $true)
+                        $baseNode.AppendChild($importedCounter) | Out-Null
+                    }
+                }
             }
 
+            # Loop through remaining reports to merge coverage data
+            foreach ($reportPath in $files[1..($files.Count - 1)]) {
+                [xml]$currentReport = Get-Content -Path $reportPath
+
+                # Merge the top-level counters
+                Merge-Counters -baseNode $mergedReport.report -newNode $currentReport.report
+
+                # Merge packages and classes
+                foreach ($package in $currentReport.report.package) {
+                    $basePackage = $mergedReport.report.package | Where-Object { $_.name -eq $package.name }
+
+                    if ($basePackage) {
+                        # Merge counters at package level
+                        Merge-Counters -baseNode $basePackage -newNode $package
+
+                        foreach ($class in $package.class) {
+                            $baseClass = $basePackage.class | Where-Object { $_.name -eq $class.name }
+                            if ($baseClass) {
+                                # Merge counters at class level
+                                Merge-Counters -baseNode $baseClass -newNode $class
+                            } else {
+                                # Import new class
+                                $importedClass = $mergedReport.ImportNode($class, $true)
+                                $basePackage.AppendChild($importedClass) | Out-Null
+                            }
+                        }
+                    } else {
+                        # Import entire new package
+                        $importedPackage = $mergedReport.ImportNode($package, $true)
+                        $mergedReport.report.AppendChild($importedPackage) | Out-Null
+                    }
+                }
+            }
+
+            # Output the combined report
+            $mergedReport.Save("merged-jacoco-report.xml")
+
+
   # TestModuleStatus:
   #   name: Test module status
   #   if: ${{ contains(fromJson('["success", "skipped"]'), needs.TestModule-pwsh-ubuntu-latest.result) && contains(fromJson('["success", "skipped"]'), needs.TestModule-pwsh-macos-latest.result) && contains(fromJson('["success", "skipped"]'), needs.TestModule-pwsh-windows-latest.result) && !cancelled() }}
