From cfadd328b05452c7511c39414fda150754d85110 Mon Sep 17 00:00:00 2001
From: Yaron Yamin <yaronyam@users.noreply.github.com>
Date: Sat, 9 Mar 2024 23:26:59 +0200
Subject: [PATCH] ci: avoid failing job on test report failure

---
 .github/workflows/test.yml | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index e369969f..09a9d0fb 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -41,21 +41,22 @@ jobs:
 
       - name: Test Report
         uses: dorny/test-reporter@v1
-        if: success() || failure()
+        if: always()
+        continue-on-error: true
         with:
           name: Test Results (${{inputs.IDEA_VERSION}},${{inputs.WITH_IDEA_PLUGINS}})
           path: build/test-results/**/*.xml
           reporter: java-junit
+
       - name: Test Results artifacts
         uses: actions/upload-artifact@v2
-        if: success() || failure()
+        if: always()
         with:
           name: test results ${{inputs.IDEA_VERSION}} plugins=${{inputs.WITH_IDEA_PLUGINS}}
           path: build/reports/**
 
       - name: On failure - log report
-        #      if: failure()
-        if: ( success() || failure() ) && steps.run-test.outcome != 'success'
+        if: steps.run-test.outcome != 'success'
         run: ./print_surefire_reports.sh
 
       - name: trace file system state
@@ -65,6 +66,7 @@ jobs:
           
       - name: Upload to Codecov
         uses: codecov/codecov-action@v4
+        if: github.event.pull_request.head.repo.fork == false
         with:
 #          files: ./coverage1.xml,./coverage2.xml # optional
 #          flags: unittests # optional