test

Author: andrewtruong

.github/workflows/test.yaml

@@ -10,52 +10,18 @@ name: test
 
 on:
   push:
-  workflow_call:
-    outputs:
-      should_run_core_integration_tests:
-        description: 'Whether core integration tests should run'
-        value: ${{ jobs.check_paths.outputs.should_run_query_tests }}
-
-env:
-  QUERY_SERVICE_PATHS: 'weave-js/ weave_query/'
 
 jobs:
-  check_paths:
-    runs-on: ubuntu-latest
-    outputs:
-      should_run_query_tests: ${{ steps.query_service.outputs.should_run_query_tests }}
-    steps:
-      - uses: actions/checkout@v3
-        with:
-          fetch-depth: 0
-          fetch-tags: true
-          ref: ${{ github.head_ref }}
-      - run: git fetch origin ${{ github.base_ref }}
-      - id: query_service
-        run: |
-          if [ "${{ github.event_name }}" = "pull_request" ]; then
-            base_sha=$(git rev-parse origin/${{ github.base_ref }})
-            head_sha=$(git rev-parse HEAD)
-            changed_files=$(git diff --name-only $base_sha $head_sha)
-          else
-            changed_files=$(git diff --name-only HEAD^)
-          fi
-
-          for path in ${{ env.WATCHED_PATHS }}; do
-            if echo "$changed_files" | grep -q "$path"; then
-              echo "should_run_query_tests=true" >> $GITHUB_OUTPUT
-              exit 0
-            fi
-          done
-          echo "should_run_query_tests=false" >> $GITHUB_OUTPUT
+  determine_tests:
+  uses: ./github/workflows/which-tests-to-run.yaml
 
   # ==== Query Service Jobs ====
   build-container-query-service:
     name: Build Legacy (Query Service) test container
     timeout-minutes: 30
     runs-on: [self-hosted, builder]
     needs: check_paths
-    if: ${{ needs.check_paths.outputs.should_run_query_tests == 'true' }}
+    if: ${{ needs.determine_tests.outputs.should_run_query_tests == 'true' }}
 
     # runs-on: ubuntu-latest
     env:
@@ -134,7 +100,7 @@ jobs:
     needs:
       - test-query-service
       - check_paths
-    if: ${{ needs.check_paths.outputs.should_run_query_tests == 'true' }}
+    if: ${{ needs.determine_tests.outputs.should_run_query_tests == 'true' }}
 
     runs-on: ubuntu-latest
 
@@ -149,7 +115,7 @@ jobs:
     name: WeaveJS Lint and Compile
     runs-on: ubuntu-latest
     needs: check_paths
-    if: ${{ needs.check_paths.outputs.should_run_query_tests == 'true' }}
+    if: ${{ needs.determine_tests.outputs.should_run_query_tests == 'true' }}
     # runs-on: [self-hosted, gke-runner]
     steps:
       - uses: actions/checkout@v2

.github/workflows/which-tests-to-run.yaml

@@ -0,0 +1,42 @@
+name: Determine which tests to run
+
+on:
+  push:
+  pull_request:
+  workflow_call:
+    outputs:
+      should_run_query_tests:
+        description: 'Whether query tests should run'
+        value: ${{ jobs.check_paths.outputs.should_run_query_tests }}
+
+env:
+  QUERY_SERVICE_PATHS: 'weave-js/ weave_query/'
+
+jobs:
+  check_paths:
+    runs-on: ubuntu-latest
+    outputs:
+      should_run_query_tests: ${{ steps.query_service.outputs.should_run_query_tests }}
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          fetch-depth: 0
+          fetch-tags: true
+          ref: ${{ github.head_ref }}
+      - run: git fetch origin ${{ github.base_ref }}
+      - id: query_service
+        run: |
+          if [ "${{ github.event_name }}" = "pull_request" ]; then
+            base_sha=$(git rev-parse origin/${{ github.base_ref }})
+            head_sha=$(git rev-parse HEAD)
+            changed_files=$(git diff --name-only $base_sha $head_sha)
+          else
+            changed_files=$(git diff --name-only HEAD^)
+          fi
+          for path in ${{ env.QUERY_SERVICE_PATHS }}; do
+            if echo "$changed_files" | grep -q "$path"; then
+              echo "should_run_query_tests=true" >> $GITHUB_OUTPUT
+              exit 0
+            fi
+          done
+          echo "should_run_query_tests=false" >> $GITHUB_OUTPUT