From dae2b9cc4e8753827702ada6e51f2ee01811cfed Mon Sep 17 00:00:00 2001 From: morluto <76467478+morluto@users.noreply.github.com> Date: Thu, 30 Jul 2026 04:50:34 +0000 Subject: [PATCH] fix(test): retain subprocess coverage and cleanup --- .github/workflows/ci.yml | 5 +++++ docs/testing.md | 2 +- package.json | 2 +- tests/support/process/processFixture.ts | 1 + 4 files changed, 8 insertions(+), 2 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index d368abab..c92e1780 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -302,6 +302,11 @@ jobs: tests/boundary/providers/ghidra/ghidraClient.test.ts tests/boundary/providers/ghidra/ghidraLauncher.test.ts tests/boundary/process/processOwnership.test.ts + tests/boundary/process/processOwnershipPart2.test.ts + tests/boundary/process/processOwnershipPart3.test.ts + tests/boundary/process/processOwnershipValidation.test.ts + tests/boundary/process/processOwnershipValidationPart2.test.ts + tests/boundary/process/processOwnershipValidationPart3.test.ts tests/conformance/providers/ghidra/realGhidraWindowsWorkflow.test.ts - run: npm run verify:package:windows - uses: actions/upload-artifact@v4 diff --git a/docs/testing.md b/docs/testing.md index dde6832e..d0b91f2a 100644 --- a/docs/testing.md +++ b/docs/testing.md @@ -44,7 +44,7 @@ are: | Command | Scope | | ------------------------- | -------------------------------------------------------------------------------------------------------- | -| `npm run test:fast` | Domain, service, adapter, and composition projects | +| `npm run test:fast` | Domain, service, adapter, composition, boundary, and conformance projects | | `npm run test:boundary` | Boundary and process-global projects | | `npm run test:acceptance` | Complete CLI and MCP acceptance workflows | | `npm run test:changed` | Changed tests in non-serial projects via Vitest's import graph | diff --git a/package.json b/package.json index 51f16408..d1135388 100644 --- a/package.json +++ b/package.json @@ -85,7 +85,7 @@ "test": "vitest run", "test:run": "vitest run", "pretest:fast": "npm run build:cached", - "test:fast": "vitest run --project domain --project services --project adapters --project composition", + "test:fast": "vitest run --project domain --project services --project adapters --project composition --project boundary --project conformance", "pretest:boundary": "npm run build:cached", "test:boundary": "vitest run --project boundary --project process-global", "pretest:acceptance": "npm run build:cached", diff --git a/tests/support/process/processFixture.ts b/tests/support/process/processFixture.ts index f96497c7..fbcd964e 100644 --- a/tests/support/process/processFixture.ts +++ b/tests/support/process/processFixture.ts @@ -83,6 +83,7 @@ export const createTestProcessRegistry = (): TestProcesses => { }); children.add(child); child.once("exit", () => children.delete(child)); + child.once("close", () => children.delete(child)); return child; };