fix(test): keep CLI process suites in owner lane (#122160)

This commit is contained in:
Vincent Koc
2026-08-12 02:07:51 +08:00
committed by GitHub
parent 2c636dda0c
commit 5bc7efe746
3 changed files with 17 additions and 3 deletions
+7 -3
View File
@@ -610,9 +610,13 @@ describe("scoped vitest configs", () => {
expect(requireTestConfig(defaultCliConfig).exclude).toEqual(
expect.arrayContaining(cliProcessTestFiles.map((file) => file.replace("src/cli/", ""))),
);
expect(requireTestConfig(defaultCliProcessConfig).include).toEqual(cliProcessTestFiles);
expect(requireTestConfig(defaultCliProcessConfig).fileParallelism).toBe(false);
expect(requireTestConfig(defaultCliProcessConfig).env).toMatchObject({
const processTestConfig = requireTestConfig(defaultCliProcessConfig);
expect(processTestConfig.include).toEqual(cliProcessTestFiles);
for (const file of cliProcessTestFiles) {
expect(matchingExcludePatterns(processTestConfig.exclude ?? [], file), file).toEqual([]);
}
expect(processTestConfig.fileParallelism).toBe(false);
expect(processTestConfig.env).toMatchObject({
ESBUILD_WORKER_THREADS: "0",
});
});
+8
View File
@@ -2,6 +2,7 @@
import path from "node:path";
import { beforeAll, describe, expect, it } from "vitest";
import { spawnNodeEvalSync } from "../src/test-utils/node-process.js";
import { cliProcessTestFiles } from "./vitest/vitest.cli-process-paths.mjs";
import { createCommandsLightVitestConfig } from "./vitest/vitest.commands-light.config.ts";
import { createPluginSdkLightVitestConfig } from "./vitest/vitest.plugin-sdk-light.config.ts";
import { createUnitFastFakeTimersVitestConfig } from "./vitest/vitest.unit-fast-fake-timers.config.ts";
@@ -217,6 +218,13 @@ describe("unit-fast vitest lane", () => {
]);
});
it("keeps process-launching CLI files in their owner lane", () => {
for (const file of cliProcessTestFiles) {
expect(isUnitFastTestFile(file), file).toBe(false);
expect(unitFastTestFiles, file).not.toContain(file);
}
});
it("routes unit-fast source files to their unit-fast sibling tests", () => {
expect(resolveUnitFastTestIncludePattern("src/plugin-sdk/provider-entry.ts")).toBe(
"src/plugin-sdk/provider-entry.test.ts",
+2
View File
@@ -2,6 +2,7 @@
import { spawnSync } from "node:child_process";
import fs from "node:fs";
import path from "node:path";
import { cliProcessTestFiles } from "./vitest.cli-process-paths.mjs";
import {
commandsLightSourceFiles,
commandsLightTestFiles,
@@ -165,6 +166,7 @@ const broadUnitFastCandidateGlobs = [
"test/**/*.test.ts",
];
const ownerRoutedUnitTestPatterns = [
...cliProcessTestFiles,
"src/agents/embedded-agent-runner/run/attempt.abort-race.test.ts",
"src/agents/openai-transport-stream.*.test.ts",
"src/agents/embedded-agent-runner/run.shared-integration.test.ts",