Merge pull request #458 from morluto/fix/test-subprocess-coverage
fix(test): retain subprocess coverage and cleanup
This commit is contained in:
commit
7f46bb1824
|
|
@ -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
|
||||
|
|
|
|||
|
|
@ -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 |
|
||||
|
|
|
|||
|
|
@ -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",
|
||||
|
|
|
|||
|
|
@ -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;
|
||||
};
|
||||
|
||||
|
|
|
|||
Loading…
Reference in New Issue