Merge remote-tracking branch 'upstream/main'

# Conflicts: # dist/index.js # dist/index.js.map # src/main.ts
bitmovin-engineering · Nov 16, 2023 · 31d6e9e · 31d6e9e
2 parents 3118ebf + 0831a82
commit 31d6e9e
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 67 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
@@ -99,6 +99,14 @@ export async function run(): Promise<void> {
       return // end if we failed due to no passed tests, but configured to require passed tests
     }
 
+    if (!(mergedResult.totalCount > 0 || mergedResult.skipped > 0) && requireTests) {
+      core.setFailed(`❌ No test results found for ${checkName}`)
+      return // end if we failed due to no tests, but configured to require tests
+    } else if (!(mergedResult.passed > 0) && requirePassedTests) {
+      core.setFailed(`❌ No passed test results found for ${checkName}`)
+      return // end if we failed due to no passed tests, but configured to require passed tests
+    }
+
     const pullRequest = github.context.payload.pull_request
     const link = (pullRequest && pullRequest.html_url) || github.context.ref
     const conclusion: 'success' | 'failure' = mergedResult.failed <= 0 ? 'success' : 'failure'