Move the if evaluation of the test jobs onthe tests job itself, so we can skip it properly in PRs and block on it in merge groups

.github/workflows/ci.yml

@@ -59,26 +59,26 @@ jobs:
       system: x86_64-linux
 
   test_aarch64-linux:
-    if: github.event_name == 'merge_group'
     uses: ./.github/workflows/test.yml
     needs: build_aarch64-linux
     with:
+      if: github.event_name == 'merge_group'
       os: blacksmith-32vcpu-ubuntu-2204-arm
       system: aarch64-linux
 
   test_x86_64-darwin:
-    if: github.event_name == 'merge_group'
     uses: ./.github/workflows/test.yml
     needs: build_x86_64-darwin
     with:
+      if: github.event_name == 'merge_group'
       os: macos-latest-large
       system: x86_64-darwin
 
   test_aarch64-darwin:
-    if: github.event_name == 'merge_group'
     uses: ./.github/workflows/test.yml
     needs: build_aarch64-darwin
     with:
+      if: github.event_name == 'merge_group'
       os: macos-latest-xlarge
       system: aarch64-darwin
 

.github/workflows/test.yml

@@ -7,8 +7,13 @@ on:
       system:
         required: true
         type: string
+      if:
+        required: false
+        default: true
+        type: boolean
 jobs:
   tests:
+    if: ${{ inputs.if }}
     strategy:
       fail-fast: false
     runs-on: ${{ inputs.os }}