fix: make native tests cancellable

Author: YogendraShelke

.github/workflows/NativePipeline.yml

@@ -734,7 +734,10 @@ jobs:
         # always(): the test step exits non-zero on a failing shard, which would otherwise
         # SKIP this step — i.e. we'd capture screenshots/logs/videos for green shards but NOT
         # for the failing ones we actually need to debug. Run regardless of test outcome.
-        if: ${{ always() }}
+        # !cancelled() rather than always(): always() is exempt from cancellation, so a
+        # cancelled run still uploaded five artifact sets per shard before it would die.
+        # This still covers pass AND fail (the cases whose artifacts we want).
+        if: ${{ !cancelled() }}
         uses: ./.github/actions/archive-test-results
         with:
           platform: android
@@ -803,8 +806,10 @@ jobs:
           bash maestro/run_maestro_widget_tests.sh ios "${{ matrix.widget }}" || [ "$UPDATE_BASELINES" = "true" ]
       
       - name: Archive test results
-        # always(): capture failure artifacts (the failing shard's test step exits non-zero).
-        if: ${{ always() }}
+        # !cancelled(): capture failure artifacts (the failing shard's test step exits
+        # non-zero), but do NOT hold a cancelled run open uploading artifacts nobody wants
+        # -- always() is exempt from cancellation, !cancelled() is not.
+        if: ${{ !cancelled() }}
         uses: ./.github/actions/archive-test-results
         with:
           platform: ios
@@ -883,8 +888,10 @@ jobs:
             bash maestro/run_maestro_jsactions_tests.sh android || [ "$UPDATE_BASELINES" = "true" ]
       
       - name: Archive test results
-        # always(): capture failure artifacts (the failing shard's test step exits non-zero).
-        if: ${{ always() }}
+        # !cancelled(): capture failure artifacts (the failing shard's test step exits
+        # non-zero), but do NOT hold a cancelled run open uploading artifacts nobody wants
+        # -- always() is exempt from cancellation, !cancelled() is not.
+        if: ${{ !cancelled() }}
         uses: ./.github/actions/archive-test-results
         with:
           platform: android
@@ -941,8 +948,10 @@ jobs:
           bash maestro/run_maestro_jsactions_tests.sh ios || [ "$UPDATE_BASELINES" = "true" ]
       
       - name: Archive test results
-        # always(): capture failure artifacts (the failing shard's test step exits non-zero).
-        if: ${{ always() }}
+        # !cancelled(): capture failure artifacts (the failing shard's test step exits
+        # non-zero), but do NOT hold a cancelled run open uploading artifacts nobody wants
+        # -- always() is exempt from cancellation, !cancelled() is not.
+        if: ${{ !cancelled() }}
         uses: ./.github/actions/archive-test-results
         with:
           platform: ios