FN-6206: quarantine flaky engine tests
Move undocumented flaky engine tests into the formal quarantine path. - add quarantine ledger entries for flaky engine test files observed under concurrent/full-suite load - exclude the quarantined engine test files from vitest projects so they no longer run in gate and reliability pools - remove the undocumented it.skip markers by quarantining at the file/config level instead Files changed: .../__tests__/merger-file-scope-invariant.test.ts | 4 ++-- .../src/__tests__/project-engine-manager.test.ts | 2 +- packages/engine/vitest.config.ts | 10 ++++++++-- scripts/lib/test-quarantine.json | 23 +++++++++++++++++++++- 4 files changed, 33 insertions(+), 6 deletions(-) Fusion-Task-Id: FN-6206 Fusion-Task-Lineage: 8d88725e-1e92-4a06-aaa6-be34287e613a
This commit is contained in:
@@ -157,7 +157,7 @@ describe("assertSquashOverlapsFileScope", () => {
|
||||
// (which reports staged files unrelated to the test scope and trips the
|
||||
// FileScopeViolationError). The same logic is covered by the existing
|
||||
// real-git fixture tests in reliability-interactions/workflow-and-file-scope.
|
||||
it.skip("accepts declared scope as a single changeset file when staged matches exactly", async () => {
|
||||
it("accepts declared scope as a single changeset file when staged matches exactly", async () => {
|
||||
const store = createInvariantStore([".changeset/fn-4767-pr-flow.md"]);
|
||||
mockStagedFiles([".changeset/fn-4767-pr-flow.md"]);
|
||||
|
||||
@@ -170,7 +170,7 @@ describe("assertSquashOverlapsFileScope", () => {
|
||||
});
|
||||
|
||||
// Skipped: same flake mode as the test above.
|
||||
it.skip("accepts declared scope as a changeset glob when staged file matches", async () => {
|
||||
it("accepts declared scope as a changeset glob when staged file matches", async () => {
|
||||
const store = createInvariantStore([".changeset/*.md"]);
|
||||
mockStagedFiles([".changeset/fn-4767-pr-flow.md"]);
|
||||
|
||||
|
||||
@@ -550,7 +550,7 @@ describe("ProjectEngineManager", () => {
|
||||
// Flake under full reliability-suite load: 30s timeout, but passes in ~46ms
|
||||
// standalone. Setinterval-driven reconciliation appears to race with vitest
|
||||
// fake-timer contention when other reliability-pool files are co-resident.
|
||||
it.skip("retries failed project starts on subsequent reconciliation ticks", async () => {
|
||||
it("retries failed project starts on subsequent reconciliation ticks", async () => {
|
||||
// Track how many times start() is called to fail only the FIRST set
|
||||
let startCallCount = 0;
|
||||
const manager = new ProjectEngineManager(centralCore);
|
||||
|
||||
@@ -68,7 +68,6 @@ export default defineConfig({
|
||||
"src/__tests__/merger-post-merge.test.ts",
|
||||
"src/__tests__/merger-conflict-resolution.test.ts",
|
||||
"src/__tests__/merger-diff-scope.test.ts",
|
||||
"src/__tests__/merger-file-scope-invariant.test.ts",
|
||||
"src/__tests__/merger-landed-files-capture.test.ts",
|
||||
"src/__tests__/branch-attribution.test.ts",
|
||||
"src/__tests__/executor-core.test.ts",
|
||||
@@ -85,7 +84,11 @@ export default defineConfig({
|
||||
"src/__tests__/heartbeat-monitor.test.ts",
|
||||
"src/__tests__/workflow-node-handlers.test.ts",
|
||||
],
|
||||
exclude: ["node_modules/**", "dist/**"],
|
||||
exclude: [
|
||||
"node_modules/**",
|
||||
"dist/**",
|
||||
"src/__tests__/merger-file-scope-invariant.test.ts",
|
||||
],
|
||||
},
|
||||
},
|
||||
{
|
||||
@@ -101,6 +104,9 @@ export default defineConfig({
|
||||
"src/**/*.slow.test.ts",
|
||||
"node_modules/**",
|
||||
"dist/**",
|
||||
"src/__tests__/merger-file-scope-invariant.test.ts",
|
||||
"src/__tests__/project-engine-manager.test.ts",
|
||||
"src/__tests__/merger-ai-cleanup.test.ts",
|
||||
],
|
||||
},
|
||||
},
|
||||
|
||||
Reference in New Issue
Block a user