Only report failure if fail-on-error is set

The flag is checked when exiting early, but the report update at the end ignores it, causing the test run to be reported as a failure. Fixes #161. Signed-off-by: Trond Einar Snekvik <Trond.Einar.Snekvik@nordicsemi.no>
dorny · Nov 30, 2022 · e7733f4 · e7733f4
1 parent 8bf0c7d
commit e7733f4
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/main.ts b/src/main.ts
@@ -174,7 +174,7 @@ class TestReporter {
     core.info('Creating annotations')
     const annotations = getAnnotations(results, this.maxAnnotations)
 
-    const isFailed = results.some(tr => tr.result === 'failed')
+    const isFailed = this.failOnError && results.some(tr => tr.result === 'failed')
     const conclusion = isFailed ? 'failure' : 'success'
     const icon = isFailed ? Icon.fail : Icon.success