Skip to content

test(validation): unmount wrappers to fix pending fetch errors#7020

Merged
vitormattos merged 1 commit intomainfrom
fix/validation-spec-pending-fetch
Feb 26, 2026
Merged

test(validation): unmount wrappers to fix pending fetch errors#7020
vitormattos merged 1 commit intomainfrom
fix/validation-spec-pending-fetch

Conversation

@vitormattos
Copy link
Copy Markdown
Member

No description provided.

Vitest was reporting 13 unhandled rejections with
'Closing rpc while fetch was pending' in Validation.spec.ts.

Root cause: Vue instances were never unmounted between tests,
so their beforeUnmount hooks never ran — isActiveView stayed
true and background microtasks (validate, refreshAfterAsyncSigning)
kept firing after the worker started winding down.

Fixes:
- Add outer afterEach(() => wrapper.unmount()) for the main wrapper
- Track local wrappers in the created() describe at scope level
  and unmount them in afterEach
- Unmount the inline localWrapper in the handleValidationSuccess
  describe immediately after use

Signed-off-by: Vitor Mattos <[email protected]>
@vitormattos vitormattos added this to the Next Major (34) milestone Feb 26, 2026
@vitormattos vitormattos self-assigned this Feb 26, 2026
@github-project-automation github-project-automation Bot moved this to 0. Needs triage in Roadmap Feb 26, 2026
@vitormattos
Copy link
Copy Markdown
Member Author

/backport to stable33

@vitormattos vitormattos merged commit 005d33f into main Feb 26, 2026
69 checks passed
@vitormattos vitormattos deleted the fix/validation-spec-pending-fetch branch February 26, 2026 21:36
@github-project-automation github-project-automation Bot moved this from 0. Needs triage to 4. to release in Roadmap Feb 26, 2026
@vitormattos
Copy link
Copy Markdown
Member Author

/backport to stable32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant