fix(ci): match checkout-root test selectors

Author: juliusmarminge

scripts/lib/cached-tests.test.ts

@@ -6,9 +6,21 @@ import * as NodeOS from "node:os";
 import * as NodePath from "node:path";
 import * as NodeURL from "node:url";
 
-import { cachedTestTasks } from "./cached-tests.ts";
+import { cachedTestTasks, matchesTestPath } from "./cached-tests.ts";
 
 describe("cachedTestTasks", () => {
+  it("selects every test for a checkout-root path", () => {
+    const root = NodePath.resolve(import.meta.dirname, "../..");
+    for (const path of [".", root]) {
+      expect(
+        matchesTestPath("apps/server/src/auth/utils.test.ts", [
+          NodePath.relative(root, NodePath.resolve(root, path)),
+        ]),
+      ).toBe(true);
+    }
+    expect(matchesTestPath("apps/server/src/auth/utils.test.ts", ["apps/server"])).toBe(true);
+    expect(matchesTestPath("apps/server-other/a.test.ts", ["apps/server"])).toBe(false);
+  });
   it("keeps a file's cache task stable when unrelated tests are added", () => {
     const first = cachedTestTasks(JSON.stringify({ files: ["src/b.test.ts"], groupSize: 1 }));
     const second = cachedTestTasks(

scripts/lib/cached-tests.ts

@@ -15,6 +15,13 @@ interface CachedTestTask {
       };
 }
 
+export function matchesTestPath(file: string, filters: Array<string>) {
+  return (
+    filters.length === 0 ||
+    filters.some((filter) => filter === "" || file === filter || file.startsWith(`${filter}/`))
+  );
+}
+
 export function cachedTestTasks(
   raw: string | undefined,
   root = NodePath.resolve(import.meta.dirname, "../.."),

scripts/test-cached.ts

@@ -5,6 +5,7 @@ import * as NodeFS from "node:fs";
 import * as NodePath from "node:path";
 import * as NodeUtil from "node:util";
 
+import { matchesTestPath } from "./lib/cached-tests.ts";
 import { shardedTestFiles } from "./lib/sharded-tests.ts";
 
 const root = NodePath.resolve(import.meta.dirname, "..");
@@ -53,8 +54,7 @@ const files = [...new Set(candidates)].filter(
     /\.test\.[cm]?[jt]sx?$/.test(file) &&
     !file.startsWith(".repos/") &&
     NodeFS.existsSync(NodePath.join(root, file)) &&
-    (filters.length === 0 ||
-      filters.some((filter) => file === filter || file.startsWith(`${filter}/`))),
+    matchesTestPath(file, filters),
 );
 if (files.length === 0) {
   console.error("No test files matched");