- add the ability to ungroup test reports

  - Relates to #1297

Author: mikepenz

.github/workflows/build.yml

@@ -57,6 +57,16 @@ jobs:
           comment: true
           annotate_only: ${{ github.event_name == 'workflow_dispatch' }}
 
+      - name: Test Ungrouped test import
+        uses: ./
+        with:
+          check_name: Example Ungrouped Test Report
+          report_paths: test_results/multiple/*.xml
+          include_passed: true
+          detailed_summary: false
+          group_reports: false
+          comment: true
+
       - name: Test JUnit flaky test import
         uses: ./
         with:

action.yml

@@ -15,6 +15,10 @@ inputs:
     description: 'Xml report paths in glob format'
     required: false
     default: '**/junit-reports/TEST-*.xml'
+  group_reports:
+    description: 'Defines if reports are grouped into a combined test result. This '
+    required: false
+    default: 'true'
   test_files_prefix:
     description: 'Prefix to add to test file paths from report files when annotating'
     required: false

dist/index.js

@@ -17,6 +17,7 @@ export async function run(): Promise<void> {
       return
     }
 
+    const groupReports = core.getInput('group_reports') === 'true'
     const annotateOnly = core.getInput('annotate_only') === 'true'
     const updateCheck = core.getInput('update_check') === 'true'
     const checkAnnotations = core.getInput('check_annotations') === 'true'
@@ -103,7 +104,26 @@ export async function run(): Promise<void> {
       mergedResult.failed += testResult.failed
       mergedResult.passed += testResult.passed
       mergedResult.retried += testResult.retried
-      testResults.push(testResult)
+
+      if (groupReports) {
+        testResults.push(testResult)
+      } else {
+        for (const actualTestResult of testResult.testResults) {
+          const failedCount = actualTestResult.annotations.filter(a => a.annotation_level === 'failure').length
+          testResults.push({
+            checkName: actualTestResult.name,
+            summary: testResult.summary,
+            totalCount: actualTestResult.totalCount,
+            skipped: actualTestResult.skippedCount,
+            failed: failedCount,
+            passed: actualTestResult.totalCount - failedCount - actualTestResult.skippedCount,
+            retried: actualTestResult.retriedCount,
+            foundFiles: 1,
+            globalAnnotations: actualTestResult.annotations,
+            testResults: actualTestResult.testResults
+          })
+        }
+      }
     }
 
     core.setOutput('total', mergedResult.totalCount)

dist/index.js.map

@@ -222,7 +222,7 @@ export async function parseFile(
     return undefined
   }
 
-  return await parseSuite(
+  const testResult = await parseSuite(
     testsuite,
     suiteRegex, // no-op
     '',
@@ -240,6 +240,12 @@ export async function parseFile(
     globalAnnotations,
     resolveIgnoreClassname
   )
+
+  if (testResult !== undefined && !testResult.name) {
+    testResult.name = pathHelper.basename(file)
+  }
+
+  return testResult
 }
 
 function templateVar(varName: string): string {

src/main.ts

@@ -0,0 +1,7 @@
+<testsuites failures="0" tests="8" time="1967986371">
+    <testsuite>
+        <testcase name="Command Test: apt-get upgrade" time="999638317"></testcase>
+        <testcase name="File Existence Test: /home/app/app" time="0"></testcase>
+        <testcase name="Metadata Test" time="0"></testcase>
+    </testsuite>
+</testsuites>

src/testParser.ts

@@ -0,0 +1,7 @@
+<testsuites failures="0" tests="8" time="1967986371">
+    <testsuite name = "test">
+        <testcase name="Command Test: apt-get upgrade" time="999638317"></testcase>
+        <testcase name="File Existence Test: /home/app/app" time="0"></testcase>
+        <testcase name="Metadata Test" time="0"></testcase>
+    </testsuite>
+</testsuites>

test_results/multiple/test_10.xml

@@ -0,0 +1,7 @@
+<testsuites failures="0" tests="8" time="1967986371">
+    <testsuite>
+        <testcase name="Command Test: apt-get upgrade" time="999638317"></testcase>
+        <testcase name="File Existence Test: /home/app/app" time="0"></testcase>
+        <testcase name="Metadata Test" time="0"></testcase>
+    </testsuite>
+</testsuites>