Skip to content

fix: prevent "Closing rpc while fetch was pending" error in unit tests#27725

Merged
anikdhabal merged 4 commits intomainfrom
unit-test-issue
Feb 9, 2026
Merged

fix: prevent "Closing rpc while fetch was pending" error in unit tests#27725
anikdhabal merged 4 commits intomainfrom
unit-test-issue

Conversation

@anikdhabal
Copy link
Copy Markdown
Contributor

@anikdhabal anikdhabal commented Feb 7, 2026

Summary by cubic

Fixes flaky Vitest tests by using pool: "forks" across all workspaces to prevent “Closing rpc while fetch was pending” errors. Also centralizes this via a shared pool constant in vitest.workspace.ts for consistency across suites.

Written for commit e9075ae. Summary will update on new commits.

@graphite-app graphite-app bot added the core area: core, team members only label Feb 7, 2026
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 7, 2026

E2E results are ready!

@anikdhabal anikdhabal marked this pull request as draft February 7, 2026 08:00
@pull-request-size pull-request-size bot added size/L and removed size/S labels Feb 9, 2026
@anikdhabal anikdhabal marked this pull request as ready for review February 9, 2026 09:15
@anikdhabal anikdhabal enabled auto-merge (squash) February 9, 2026 09:16
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

@anikdhabal anikdhabal merged commit 7b9168e into main Feb 9, 2026
55 of 56 checks passed
@anikdhabal anikdhabal deleted the unit-test-issue branch February 9, 2026 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core area: core, team members only ready-for-e2e size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants