Merge pull request #864 from mikepenz/featue/require_passed_tests

Offer configuration `require_passed_tests`
mikepenz · May 23, 2023 · 16f9a74 · 16f9a74
2 parents 62e3580 + 6fe2c81
commit 16f9a74
Show file tree

Hide file tree

Showing 5 changed files with 17 additions and 5 deletions.
diff --git a/README.md b/README.md
@@ -82,6 +82,7 @@ jobs:
 | `commit`          | Optional. The commit SHA to update the status. This is useful when you run it with `workflow_run`.                                                                 |
 | `fail_on_failure` | Optional. Fail the build in case of a test failure.                                                                                                                |
 | `require_tests`   | Optional. Fail if no test are found.                                                                                                                               |
+| `require_passed_tests`   | Optional. Fail if no passed test are found. (This is stricter than `require_tests`, which accepts skipped tests).                                           |
 | `include_passed`   | Optional. By default the action will skip passed items for the annotations. Enable this flag to include them.                                                                                                                               |
 | `check_retries`         | Optional. If a testcase is retried, ignore the original failure.                                                                                             |
 | `check_title_template`  | Optional. Template to configure the title format. Placeholders: {{FILE_NAME}}, {{SUITE_NAME}}, {{TEST_NAME}}.                                                |

diff --git a/action.yml b/action.yml
@@ -49,6 +49,10 @@ inputs:
     description: 'Fail if no test are found.'
     required: false
     default: 'false'
+  require_passed_tests:
+    description: 'Fail if no passed test are found.'
+    required: false
+    default: 'false'
   include_passed:
     description: 'Include passed tests in the report'
     required: false

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
@@ -19,6 +19,7 @@ export async function run(): Promise<void> {
     const commit = core.getInput('commit')
     const failOnFailure = core.getInput('fail_on_failure') === 'true'
     const requireTests = core.getInput('require_tests') === 'true'
+    const requirePassedTests = core.getInput('require_passed_tests') === 'true'
     const includePassed = core.getInput('include_passed') === 'true'
     const checkRetries = core.getInput('check_retries') === 'true'
     const annotateNotice = core.getInput('annotate_notice') === 'true'
@@ -87,10 +88,12 @@ 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) {
+    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