From bb6b928dc7e1ce022aca9fac60a6002a5c68b060 Mon Sep 17 00:00:00 2001 From: Sheraff Date: Sun, 9 Aug 2026 23:17:45 +0200 Subject: [PATCH] test(router-core): add missing test cleanup in 3920 --- ...ue-3920-on-before-navigate-after-initial-redirect.test.ts | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/packages/router-core/tests/issue-3920-on-before-navigate-after-initial-redirect.test.ts b/packages/router-core/tests/issue-3920-on-before-navigate-after-initial-redirect.test.ts index f0de5df8dd..6df66a08fa 100644 --- a/packages/router-core/tests/issue-3920-on-before-navigate-after-initial-redirect.test.ts +++ b/packages/router-core/tests/issue-3920-on-before-navigate-after-initial-redirect.test.ts @@ -1,5 +1,5 @@ import { createMemoryHistory } from '@tanstack/history' -import { expect, test } from 'vitest' +import { expect, onTestFinished, test } from 'vitest' import { BaseRootRoute, BaseRoute, redirect } from '../src' import { createTestRouter } from './routerTestUtils' @@ -30,9 +30,10 @@ test('onBeforeNavigate fires after the initial load redirects', async () => { expect(router.state.location.pathname).toBe('/target') const navigatedPaths: Array = [] - router.subscribe('onBeforeNavigate', (event) => { + const unsub = router.subscribe('onBeforeNavigate', (event) => { navigatedPaths.push(event.toLocation.pathname) }) + onTestFinished(unsub) await router.navigate({ to: '/third' })