Merge pull request #14 from mikepenz/feature/require_tests

Fail on missing test cases

README.md

@@ -37,6 +37,7 @@ Based on action for [Surefire Reports by ScaCap](https://github.com/ScaCap/actio
 | `check_name`      | Optional. Check name to use when creating a check run. The default is `Test Report`.                                                                               |
 | `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..                                                                 |
 
 ### Example usage
 

action.yml

@@ -25,6 +25,10 @@ inputs:
     description: 'Fail the build in case a test failure occurred.'
     required: false
     default: 'false'
+  require_tests:
+    description: 'Fail if no test are found.'
+    required: false
+    default: 'false'
 runs:
   using: 'node12'
   main: 'dist/index.js'

src/main.ts

@@ -16,6 +16,7 @@ export async function run(): Promise<void> {
     const checkName = core.getInput('check_name')
     const commit = core.getInput('commit')
     const failOnFailure = core.getInput('fail_on_failure') === 'true'
+    const requireTests = core.getInput('require_tests') === 'true'
 
     core.endGroup()
     core.startGroup(`📦 Process test results`)
@@ -25,7 +26,12 @@ export async function run(): Promise<void> {
     const title = foundResults
       ? `${testResult.count} tests run, ${testResult.skipped} skipped, ${testResult.annotations.length} failed.`
       : 'No test results found!'
-    core.info(`Result: ${title}`)
+    core.info(`ℹ️ ${title}`)
+
+    if (!foundResults && requireTests) {
+      core.setFailed('❌ No test results found')
+      return
+    }
 
     const pullRequest = github.context.payload.pull_request
     const link = (pullRequest && pullRequest.html_url) || github.context.ref
@@ -37,7 +43,7 @@ export async function run(): Promise<void> {
     const head_sha =
       commit || (pullRequest && pullRequest.head.sha) || github.context.sha
     core.info(
-      `Posting status '${status}' with conclusion '${conclusion}' to ${link} (sha: ${head_sha})`
+      `ℹ️ Posting status '${status}' with conclusion '${conclusion}' to ${link} (sha: ${head_sha})`
     )
 
     const createCheckRequest = {
@@ -66,13 +72,15 @@ export async function run(): Promise<void> {
 
       if (failOnFailure && conclusion === 'failure') {
         core.setFailed(
-          `Tests reported ${testResult.annotations.length} failures`
+          `❌ Tests reported ${testResult.annotations.length} failures`
         )
       }
     } catch (error) {
-      core.error(`Failed to create checks using the provided token. (${error})`)
+      core.error(
+        `❌ Failed to create checks using the provided token. (${error})`
+      )
       core.warning(
-        `This usually indicates insufficient permissions. More details: https://github.com/mikepenz/action-junit-report/issues/32`
+        `⚠️ This usually indicates insufficient permissions. More details: https://github.com/mikepenz/action-junit-report/issues/32`
       )
     }
 

src/testParser.ts

@@ -47,7 +47,9 @@ export async function resolveFileAndLine(
 
     return {fileName, line: parseInt(line)}
   } catch (error) {
-    core.warning(`⚠️ Failed to resolve file and line for ${file} and ${className}`)
+    core.warning(
+      `⚠️ Failed to resolve file and line for ${file} and ${className}`
+    )
     return {fileName, line: 1}
   }
 }