ci: Only run affected unit tests on PRs

Author: mydea

.github/workflows/build.yml

@@ -446,8 +446,15 @@ jobs:
         uses: ./.github/actions/restore-cache
         env:
           DEPENDENCY_CACHE_KEY: ${{ needs.job_build.outputs.dependency_cache_key }}
-      - name: Run tests
-        run: yarn test-ci-browser
+
+      - name: Run affected tests
+        run: yarn test:pr:browser
+        if: github.event_name == 'pull_request'
+
+      - name: Run all tests
+        run: yarn test:ci:browser
+        if: github.event_name != 'pull_request'
+
       - name: Compute test coverage
         uses: codecov/codecov-action@v4
         with:
@@ -478,7 +485,7 @@ jobs:
           DEPENDENCY_CACHE_KEY: ${{ needs.job_build.outputs.dependency_cache_key }}
       - name: Run tests
         run: |
-          yarn test-ci-bun
+          yarn test:ci:bun
 
   job_deno_unit_tests:
     name: Deno Unit Tests
@@ -536,7 +543,7 @@ jobs:
       - name: Run tests
         env:
           NODE_VERSION: ${{ matrix.node }}
-        run: yarn test-ci-node
+        run: yarn test:ci:node
       - name: Compute test coverage
         uses: codecov/codecov-action@v4
         with:

package.json

@@ -33,10 +33,13 @@
     "postpublish": "lerna run --stream --concurrency 1 postpublish",
     "test": "lerna run --ignore \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests,overhead-metrics}\" test",
     "test:unit": "lerna run --ignore \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests,overhead-metrics}\" test:unit",
-    "test-ci-browser": "lerna run test --ignore \"@sentry/{bun,deno,node,profiling-node,serverless,google-cloud,nextjs,remix,gatsby,sveltekit,vercel-edge}\" --ignore \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests,overhead-metrics}\"",
-    "test-ci-node": "ts-node ./scripts/node-unit-tests.ts",
-    "test-ci-bun": "lerna run test --scope @sentry/bun",
     "test:update-snapshots": "lerna run test:update-snapshots",
+    "test:pr": "nx affected -t test --exclude \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests,overhead-metrics}\"",
+    "test:pr:browser": "yarn test:pr --exclude \"@sentry/{bun,deno,node,profiling-node,aws-serverless,google-cloud-serverless,nextjs,nextjs,nuxt,remix,gatsby,sveltekit,vercel-edge}\"",
+    "test:pr:node": "ts-node ./scripts/node-unit-tests.ts --affected",
+    "test:ci:browser": "lerna run test --ignore \"@sentry/{bun,deno,node,profiling-node,aws-serverless,google-cloud-serverless,nextjs,nestjs,nuxt,remix,gatsby,sveltekit,vercel-edge}\" --ignore \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests,overhead-metrics}\"",
+    "test:ci:node": "ts-node ./scripts/node-unit-tests.ts",
+    "test:ci:bun": "lerna run test --scope @sentry/bun",
     "yalc:publish": "lerna run yalc:publish"
   },
   "volta": {

scripts/node-unit-tests.ts

@@ -8,6 +8,8 @@ interface VersionConfig {
 
 const CURRENT_NODE_VERSION = process.version.replace('v', '').split('.')[0] as NodeVersion;
 
+const RUN_AFFECTED = process.argv.includes('--affected');
+
 const DEFAULT_SKIP_TESTS_PACKAGES = [
   '@sentry-internal/eslint-plugin-sdk',
   '@sentry/ember',
@@ -70,6 +72,14 @@ function runWithIgnores(skipPackages: string[] = []): void {
   run(`yarn test ${ignoreFlags}`);
 }
 
+/**
+ * Run affected tests, ignoring the given packages
+ */
+function runAffectedWithIgnores(skipPackages: string[] = []): void {
+  const ignoreFlags = skipPackages.map(dep => `--exclude="${dep}"`).join(' ');
+  run(`yarn test:pr ${ignoreFlags}`);
+}
+
 /**
  * Run the tests, accounting for compatibility problems in older versions of Node.
  */
@@ -83,7 +93,11 @@ function runTests(): void {
     versionConfig.ignoredPackages.forEach(dep => ignores.add(dep));
   }
 
-  runWithIgnores(Array.from(ignores));
+  if (RUN_AFFECTED) {
+    runAffectedWithIgnores(Array.from(ignores));
+  } else {
+    runWithIgnores(Array.from(ignores));
+  }
 }
 
 runTests();