test(app-testing): PR on failure correct

Opentrons · Dec 19, 2023 · 8399aac · 8399aac
1 parent 499f7f7
commit 8399aac
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/.github/workflows/analyses-snapshot-test.yaml b/.github/workflows/analyses-snapshot-test.yaml
@@ -51,19 +51,19 @@ jobs:
       run: make snapshot-test
 
     - name: Upload Report
-      if: always()
+      if: '!cancelled()'
       uses: actions/upload-artifact@v4
       with:
         name: test-report
         path: app-testing/results/
 
     - name: Handle Test Failure
-      if: steps.run_test.outcome == 'failure'
+      if: failure()
       working-directory: app-testing
       run: make snapshot-test-update
 
     - name: Create Snapshot update Request
-      if: steps.run_test.outcome == 'failure'
+      if: failure()
       uses: peter-evans/create-pull-request@v5
       with:
           commit-message: 'fix(app-testing): snapshot failure capture'