diff --git a/mobile/src/tasks/mobile-task-navigation.test.ts b/mobile/src/tasks/mobile-task-navigation.test.ts index 5cd673c8c1c..ffdd564a531 100644 --- a/mobile/src/tasks/mobile-task-navigation.test.ts +++ b/mobile/src/tasks/mobile-task-navigation.test.ts @@ -1,6 +1,10 @@ -import { describe, expect, it, vi } from 'vitest' +import { afterEach, describe, expect, it, vi } from 'vitest' import { mobileTasksRoute, navigateToMobileTasks } from './mobile-task-navigation' +afterEach(() => { + vi.unstubAllGlobals() +}) + describe('mobileTasksRoute', () => { it('builds a concrete encoded host route', () => { expect(mobileTasksRoute('host/1')).toBe('/h/host%2F1/tasks') @@ -16,12 +20,11 @@ describe('mobileTasksRoute', () => { return 1 }) - navigateToMobileTasks({ push, replace }, 'host-1', 'github') - expect(push).toHaveBeenCalledWith('/h/host-1') + navigateToMobileTasks({ push, replace }, 'host/1', 'github') + expect(push).toHaveBeenCalledWith('/h/host%2F1') expect(replace).not.toHaveBeenCalled() nextFrame?.(0) - expect(replace).toHaveBeenCalledWith(mobileTasksRoute('host-1', 'github')) - vi.unstubAllGlobals() + expect(replace).toHaveBeenCalledWith(mobileTasksRoute('host/1', 'github')) }) }) diff --git a/mobile/src/tasks/mobile-task-navigation.ts b/mobile/src/tasks/mobile-task-navigation.ts index 52027d2f012..77de7596484 100644 --- a/mobile/src/tasks/mobile-task-navigation.ts +++ b/mobile/src/tasks/mobile-task-navigation.ts @@ -21,7 +21,7 @@ export function navigateToMobileTasks( provider?: TaskProvider ): void { // Why: a cold nested host navigator resolves a deep push to its index route. - router.push(`/h/${hostId}`) + router.push(`/h/${encodeURIComponent(hostId)}`) requestAnimationFrame(() => { router.replace(mobileTasksRoute(hostId, provider)) })