From eec475cc8e4c64b40026c63e269e0460ad53fb74 Mon Sep 17 00:00:00 2001
From: Sirz Benjie <142067137+SirzBenjie@users.noreply.github.com>
Date: Thu, 1 May 2025 20:46:17 -0500
Subject: [PATCH] Make tests skip if filters are matched

---
 .github/test-filters.yml    | 18 +++++++++++++
 .github/workflows/tests.yml | 53 +++++++++----------------------------
 2 files changed, 31 insertions(+), 40 deletions(-)
 create mode 100644 .github/test-filters.yml

diff --git a/.github/test-filters.yml b/.github/test-filters.yml
new file mode 100644
index 00000000000..5d1c31e17b8
--- /dev/null
+++ b/.github/test-filters.yml
@@ -0,0 +1,18 @@
+all:
+  - "src/**"
+  - "test/**"
+  - "public/**"
+  # Workflows that can impact tests
+  - ".github/workflows/test*.yml"
+  # top-level files
+  - "package*.json"
+  - ".nvrmc" # Updates to node version can break tests
+  - "vite*" # vite.config.ts, vite.vitest.config.ts, vitest.workspace.ts
+  - "tsconfig*.json" # tsconfig.json tweaking can impact compilation
+  - "global.d.ts"
+  - ".env.*"
+  # Blanket negations for files that cannot impact tests
+  - "!**/*.py" # No .py files
+  - "!**/*.sh" # No .sh files
+  - "!**/*.md" # No .md files
+  - "!**/.git*" # .gitkeep and family
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index ccc8604ff7e..daaee24447e 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -7,56 +7,29 @@ on:
     branches:
       - main # Trigger on push events to the main branch
       - beta # Trigger on push events to the beta branch
-    # go upvote https://github.com/actions/runner/issues/1182 and yell at microsoft until they fix this or ditch yml for workflows
-    paths:
-      # src and test files
-      - "src/**"
-      - "test/**"
-      - "public/**"
-      # Workflows that can impact tests
-      - ".github/workflows/test*.yml"
-      # top-level files
-      - "package*.json"
-      - ".nvrmc" # Updates to node version can break tests
-      - "vite.*.ts" # vite.config.ts, vite.vitest.config.ts, vitest.workspace.ts
-      - "tsconfig*.json" # tsconfig.json tweaking can impact compilation
-      - "global.d.ts"
-      - ".env.*"
-      # Blanket negations for files that cannot impact tests
-      - "!**/*.py" # No .py files
-      - "!**/*.sh" # No .sh files
-      - "!**/*.md" # No .md files
-      - "!**/.git*" # .gitkeep and family
-
   pull_request:
     branches:
       - main # Trigger on pull request events targeting the main branch
       - beta # Trigger on pull request events targeting the beta branch
-    paths: # go upvote https://github.com/actions/runner/issues/1182 and yell at microsoft because until then we have to duplicate this
-      # src and test files
-      - "src/**"
-      - "test/**"
-      - "public/**"
-      # Workflows that can impact tests
-      - ".github/workflows/test*.yml"
-      # top-level files
-      - "package*.json"
-      - ".nvrmc" # Updates to node version can break tests
-      - "vite*" # vite.config.ts, vite.vitest.config.ts, vitest.workspace.ts
-      - "tsconfig*.json" # tsconfig.json tweaking can impact compilation
-      - "global.d.ts"
-      - ".env.*"
-      # Blanket negations for files that cannot impact tests
-      - "!**/*.py" # No .py files
-      - "!**/*.sh" # No .sh files
-      - "!**/*.md" # No .md files
-      - "!**/.git*" # .gitkeep and family
   merge_group:
     types: [checks_requested]
 
 jobs:
+  check-path-change-filter:
+    runs-on: ubuntu-latest
+    permissions:
+      pull-requests: read
+    outputs:
+      all: ${{ steps.filter.outputs.all }}
+    steps:
+      - uses: dorny/paths-filter@de90cc6fb38fc0963ad72b210f1f284cd68cea36
+        with:
+          filters: ./.github/test-filters.yml
+
   run-tests:
     name: Run Tests
+    needs: check-path-change-filter
+    if: ${{ needs.changes.outputs.all == 'true'}}
     strategy:
       matrix:
         shard: [1, 2, 3, 4, 5, 6, 7, 8, 9, 10]