ci: fix conditions

antonbsa · Sep 15, 2023 · 8a93d02 · 8a93d02
1 parent 8ce6c2f
commit 8a93d02
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/.github/workflows/automated-tests.yml b/.github/workflows/automated-tests.yml
@@ -372,23 +372,24 @@ jobs:
       - if: steps.pr-data-artifact.outcome == 'success'
         id: failure-check
         run: |
+          # "failure-reason" file is created for build and installation failure only
           if [ -f "./pr-comment-data/failure-reason" ]; then
             echo "isAutomatedTestFailure=false" >> $GITHUB_OUTPUT
           else
             echo "isAutomatedTestFailure=true" >> $GITHUB_OUTPUT
           fi
       - name: Download all blob reports from GitHub Actions Artifacts
-        if: needs.install-and-run-tests.result == 'skipped' || !steps.failure-check.outputs.isAutomatedTestFailure
+        if: steps.failure-check.outputs.isAutomatedTestFailure
         uses: actions/download-artifact@v3
         with:
           name: all-blob-reports
           path: bigbluebutton-tests/playwright/all-blob-reports
       - name: Merge into HTML Report
-        if: needs.install-and-run-tests.result == 'skipped' || !steps.failure-check.outputs.isAutomatedTestFailure
+        if: steps.failure-check.outputs.isAutomatedTestFailure
         working-directory: ./bigbluebutton-tests/playwright
         run: npx playwright merge-reports --reporter html ./all-blob-reports
       - name: Upload HTML tests report
-        if: needs.install-and-run-tests.result == 'skipped' || !steps.failure-check.outputs.isAutomatedTestFailure
+        if: steps.failure-check.outputs.isAutomatedTestFailure
         uses: actions/upload-artifact@v3
         with:
           name: tests-report