Split hereby test:all

Author: andrewbranch

.github/workflows/ci.yml

@@ -92,7 +92,21 @@ jobs:
 
       - name: Tests
         id: test
-        run: npx hereby test:all ${RACE_FLAG:+"$RACE_FLAG"} ${NOEMBED_FLAG:+"$NOEMBED_FLAG"} ${CONCURRENTTESTPROGRAM_FLAG:+"$CONCURRENTTESTPROGRAM_FLAG"}
+        run: npx hereby test ${RACE_FLAG:+"$RACE_FLAG"} ${NOEMBED_FLAG:+"$NOEMBED_FLAG"} ${CONCURRENTTESTPROGRAM_FLAG:+"$CONCURRENTTESTPROGRAM_FLAG"}
+        env:
+          RACE_FLAG: ${{ (matrix.race && '--race') || '' }}
+          NOEMBED_FLAG: ${{ (matrix.noembed && '--noembed') || '' }}
+          CONCURRENTTESTPROGRAM_FLAG: ${{ (matrix.concurrent-test-program && '--concurrentTestPrograms') || '' }}
+
+      - name: API Tests
+        run: npx hereby test:api ${RACE_FLAG:+"$RACE_FLAG"} ${NOEMBED_FLAG:+"$NOEMBED_FLAG"} ${CONCURRENTTESTPROGRAM_FLAG:+"$CONCURRENTTESTPROGRAM_FLAG"}
+        env:
+          RACE_FLAG: ${{ (matrix.race && '--race') || '' }}
+          NOEMBED_FLAG: ${{ (matrix.noembed && '--noembed') || '' }}
+          CONCURRENTTESTPROGRAM_FLAG: ${{ (matrix.concurrent-test-program && '--concurrentTestPrograms') || '' }}
+
+      - name: Benchmarks
+        run: npx hereby test:benchmarks ${RACE_FLAG:+"$RACE_FLAG"} ${NOEMBED_FLAG:+"$NOEMBED_FLAG"} ${CONCURRENTTESTPROGRAM_FLAG:+"$CONCURRENTTESTPROGRAM_FLAG"}
         env:
           RACE_FLAG: ${{ (matrix.race && '--race') || '' }}
           NOEMBED_FLAG: ${{ (matrix.noembed && '--noembed') || '' }}

Herebyfile.mjs

@@ -267,7 +267,7 @@ async function runTestTools() {
     await $test({ cwd: path.join(__dirname, "_tools") })`${gotestsum()} ./...`;
 }
 
-async function runTestJS() {
+async function runTestAPI() {
     await $`npm run -w @typescript/api test`;
 }
 
@@ -276,29 +276,17 @@ export const testTools = task({
     run: runTestTools,
 });
 
-export const buildJSTest = task({
-    name: "build:js:test",
+export const buildAPITests = task({
+    name: "build:api:test",
     run: async () => {
         await $`npm run -w @typescript/api build:test`;
     },
 });
 
-export const testJS = task({
-    name: "test:js",
-    dependencies: [tsgo, buildJSTest],
-    run: runTestJS,
-});
-
-export const testAll = task({
-    name: "test:all",
-    dependencies: [tsgo, buildJSTest],
-    run: async () => {
-        // Prevent interleaving by running these directly instead of in parallel.
-        await runTests();
-        await runTestBenchmarks();
-        await runTestTools();
-        await runTestJS();
-    },
+export const testAPI = task({
+    name: "test:api",
+    dependencies: [tsgo, buildAPITests],
+    run: runTestAPI,
 });
 
 const customLinterPath = "./_tools/custom-gcl";

_packages/api/package.json

@@ -38,6 +38,7 @@
     "scripts": {
         "build": "tsc -b",
         "build:test": "tsc -b test",
+        "build:bench": "tsc -b bench",
         "bench": "node --experimental-strip-types --no-warnings --conditions @typescript/source bench/api.bench.ts",
         "test": "node --test --experimental-strip-types --no-warnings --conditions @typescript/source ./test/**/*.test.ts"
     },