Merge pull request #797 from mikepenz/fix/794

Fix condition resulting in an early abort for multiple `report_paths`
mikepenz · Feb 16, 2023 · 204f987 · 204f987
2 parents f89327e + a466fdb
commit 204f987
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 17 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/main.ts b/src/main.ts
@@ -79,15 +79,6 @@ export async function run(): Promise<void> {
       mergedResult.skipped += testResult.skipped
       mergedResult.failed += testResult.failed
       mergedResult.passed += testResult.passed
-
-      const foundResults = testResult.totalCount > 0 || testResult.skipped > 0
-      if (!foundResults) {
-        if (requireTests) {
-          core.setFailed(`❌ No test results found for ${checkName}`)
-        }
-        return
-      }
-
       testResults.push(testResult)
     }
 
@@ -96,6 +87,11 @@ export async function run(): Promise<void> {
     core.setOutput('skipped', mergedResult.skipped)
     core.setOutput('failed', mergedResult.failed)
 
+    const foundResults = mergedResult.totalCount > 0 || mergedResult.skipped > 0
+    if (!foundResults && requireTests) {
+      core.setFailed(`❌ No test results found for ${checkName}`)
+    }
+
     const pullRequest = github.context.payload.pull_request
     const link = (pullRequest && pullRequest.html_url) || github.context.ref
     const conclusion: 'success' | 'failure' =