-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
Refactor: split oversized test suites #106350
Copy link
Copy link
Closed
Labels
P3Low-priority cleanup, docs, polish, ergonomics, or speculative work.Low-priority cleanup, docs, polish, ergonomics, or speculative work.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.issue-rating: 🌊 off-meta tidepoolIssue quality rating does not apply to this item.Issue quality rating does not apply to this item.maintainerMaintainer-authored PRMaintainer-authored PR
Description
Metadata
Metadata
Assignees
Labels
P3Low-priority cleanup, docs, polish, ergonomics, or speculative work.Low-priority cleanup, docs, polish, ergonomics, or speculative work.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.issue-rating: 🌊 off-meta tidepoolIssue quality rating does not apply to this item.Issue quality rating does not apply to this item.maintainerMaintainer-authored PRMaintainer-authored PR
Type
Fields
Priority
None yet
Summary
Split five oversized test suites into focused test-support modules while preserving one Vitest entry and mocked module graph per suite.
Problem
The OpenAI transport, configured reply dispatch, gateway agent methods, plugin loader, and policy doctor suites range from roughly 9,500 to 13,800 lines each. Their size makes navigation, ownership review, and focused maintenance unnecessarily difficult.
Proposed solution
*.test.tspath as the sole Vitest entry.*.test-support.tsmodules.*.test-harness.tsmodules.Alternatives considered
Independent
*.test.tsfragments were rejected because the repository's non-isolated Vitest projects can reuse workers and mocked singleton state across files. A single entry importing support modules keeps the original module graph and execution semantics.Impact
Test-only refactor. No runtime, user-facing, configuration, or dependency changes.