From 1a0e38fb171e1c2af6720017d6bff798ea18433f Mon Sep 17 00:00:00 2001 From: Johannes Millan Date: Sun, 15 Mar 2026 18:41:54 +0100 Subject: [PATCH] test(e2e): use correct route /#/tag/TODAY/tasks instead of /#/tag/TODAY MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The deadlines feature commit accidentally reverted e2e URL fixes. The route /#/tag/TODAY has no default child component — only /#/tag/TODAY/tasks renders the work view with task-list. --- e2e/helpers/plugin-test.helpers.ts | 2 +- e2e/pages/settings.page.ts | 2 +- e2e/tests/app-features/app-features.spec.ts | 2 +- e2e/tests/app-features/focus-mode-feature.spec.ts | 4 ++-- e2e/tests/app-features/time-tracking-feature.spec.ts | 4 ++-- e2e/tests/break/take-a-break.spec.ts | 4 ++-- e2e/tests/navigation/basic-navigation.spec.ts | 2 +- e2e/tests/navigation/context-switching.spec.ts | 2 +- e2e/tests/planner/planner-basic.spec.ts | 4 ++-- e2e/tests/planner/planner-multiple-days.spec.ts | 2 +- e2e/tests/planner/planner-navigation.spec.ts | 6 +++--- e2e/tests/planner/planner-scheduled-tasks.spec.ts | 2 +- e2e/tests/planner/planner-time-estimates.spec.ts | 2 +- e2e/tests/plugins/plugin-lifecycle.spec.ts | 4 ++-- e2e/tests/recurring/recurring-task.spec.ts | 2 +- e2e/tests/settings/settings-basic.spec.ts | 2 +- e2e/tests/work-view/work-view-features.spec.ts | 2 +- 17 files changed, 24 insertions(+), 24 deletions(-) diff --git a/e2e/helpers/plugin-test.helpers.ts b/e2e/helpers/plugin-test.helpers.ts index 7d7e96a9db..fb035d827a 100644 --- a/e2e/helpers/plugin-test.helpers.ts +++ b/e2e/helpers/plugin-test.helpers.ts @@ -249,7 +249,7 @@ export const waitForPluginInMenu = async ( ): Promise => { try { // Navigate to main view to see the menu - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); // Wait for magic-side-nav to exist await page.waitForSelector('magic-side-nav', { diff --git a/e2e/pages/settings.page.ts b/e2e/pages/settings.page.ts index 18b81a4b51..139cb981c9 100644 --- a/e2e/pages/settings.page.ts +++ b/e2e/pages/settings.page.ts @@ -235,7 +235,7 @@ export class SettingsPage extends BasePage { * Navigate back to work view */ async navigateBackToWorkView(): Promise { - await this.page.goto('/#/tag/TODAY'); + await this.page.goto('/#/tag/TODAY/tasks'); await this.page.waitForLoadState('networkidle'); await waitForAngularStability(this.page); } diff --git a/e2e/tests/app-features/app-features.spec.ts b/e2e/tests/app-features/app-features.spec.ts index 40f6ac8aab..2260c4cb8c 100644 --- a/e2e/tests/app-features/app-features.spec.ts +++ b/e2e/tests/app-features/app-features.spec.ts @@ -10,7 +10,7 @@ const goToConfig = async (page: Page): Promise => { // Navigate to main view and wait for it to be ready const goToMainView = async (page: Page): Promise => { - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); await waitForAngularStability(page); }; diff --git a/e2e/tests/app-features/focus-mode-feature.spec.ts b/e2e/tests/app-features/focus-mode-feature.spec.ts index 9a3440869a..ec9cdf8730 100644 --- a/e2e/tests/app-features/focus-mode-feature.spec.ts +++ b/e2e/tests/app-features/focus-mode-feature.spec.ts @@ -43,7 +43,7 @@ test.describe('App Features - Focus Mode', () => { await expect(focusModeSwitch).not.toBeChecked(); // Navigate to main view - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); // Focus Mode button in main button bar should not be present when feature is disabled await expect(mainFocusButton).not.toBeAttached(); // Focus Mode in the task context menu should not be visible @@ -66,7 +66,7 @@ test.describe('App Features - Focus Mode', () => { await expect(focusModeSwitch).toBeChecked(); // Go back to main view and expect play button to be visible - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); await expect(mainFocusButton).toBeAttached(); await firstTask.click({ button: 'right' }); diff --git a/e2e/tests/app-features/time-tracking-feature.spec.ts b/e2e/tests/app-features/time-tracking-feature.spec.ts index 91b445b526..8308c46ab1 100644 --- a/e2e/tests/app-features/time-tracking-feature.spec.ts +++ b/e2e/tests/app-features/time-tracking-feature.spec.ts @@ -38,7 +38,7 @@ test.describe('App Features - Time Tracking', () => { await expect(timeTrackingSwitch).not.toBeChecked(); // Navigate to main view - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); // Play button in main button bar should not be present when feature is disabled await expect(mainPlayButton).not.toBeAttached(); // Play button in the task hover menu should not be visible @@ -63,7 +63,7 @@ test.describe('App Features - Time Tracking', () => { await expect(timeTrackingSwitch).toBeChecked(); // Go back to main view and expect play button to be visible - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); await expect(mainPlayButton).toBeAttached(); await firstTask.hover(); diff --git a/e2e/tests/break/take-a-break.spec.ts b/e2e/tests/break/take-a-break.spec.ts index ee7f723aa8..e38431ae5d 100644 --- a/e2e/tests/break/take-a-break.spec.ts +++ b/e2e/tests/break/take-a-break.spec.ts @@ -63,7 +63,7 @@ test.describe('Take-a-Break Feature', () => { await expect(page.locator('.page-settings').first()).toBeVisible(); // Navigate away - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); await expect(page.locator('task-list').first()).toBeVisible(); // Navigate back to settings @@ -105,7 +105,7 @@ test.describe('Take-a-Break Feature', () => { await expect(page.locator('.page-settings')).toBeVisible(); // Navigate back to work view - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); // Verify we're back at work view await expect(page).toHaveURL(/tag\/TODAY/); diff --git a/e2e/tests/navigation/basic-navigation.spec.ts b/e2e/tests/navigation/basic-navigation.spec.ts index d25bbbf6d8..9f005503d9 100644 --- a/e2e/tests/navigation/basic-navigation.spec.ts +++ b/e2e/tests/navigation/basic-navigation.spec.ts @@ -52,7 +52,7 @@ test.describe('Basic Navigation', () => { await expect(page.locator('.page-settings')).toBeVisible(); // Navigate back to work view - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); await page.waitForLoadState('networkidle'); await expect(page).toHaveURL(/\/#\/tag\/TODAY/); await expect(page.locator('task-list').first()).toBeVisible(); diff --git a/e2e/tests/navigation/context-switching.spec.ts b/e2e/tests/navigation/context-switching.spec.ts index cd02b7fd7d..0cf9e9bd2c 100644 --- a/e2e/tests/navigation/context-switching.spec.ts +++ b/e2e/tests/navigation/context-switching.spec.ts @@ -241,7 +241,7 @@ test.describe('Context Switching', () => { await page.waitForTimeout(300); // Navigate back to TODAY via URL (more reliable than click) - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); await page.waitForSelector('task-list', { state: 'visible', timeout: 15000 }); await expect(page).toHaveURL(/tag\/TODAY/); }); diff --git a/e2e/tests/planner/planner-basic.spec.ts b/e2e/tests/planner/planner-basic.spec.ts index d3a85e087d..aaa0998a8e 100644 --- a/e2e/tests/planner/planner-basic.spec.ts +++ b/e2e/tests/planner/planner-basic.spec.ts @@ -54,7 +54,7 @@ test.describe('Planner Basic', () => { test('should switch between work view and planner', async ({ page, workViewPage }) => { // Start in work view - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); await workViewPage.waitForTaskList(); // Add a task @@ -66,7 +66,7 @@ test.describe('Planner Basic', () => { await plannerPage.waitForPlannerView(); // Go back to work view - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); await workViewPage.waitForTaskList(); // Task should still be there diff --git a/e2e/tests/planner/planner-multiple-days.spec.ts b/e2e/tests/planner/planner-multiple-days.spec.ts index 01c1ce7dc0..914841d232 100644 --- a/e2e/tests/planner/planner-multiple-days.spec.ts +++ b/e2e/tests/planner/planner-multiple-days.spec.ts @@ -72,7 +72,7 @@ test.describe('Planner Multiple Days', () => { await plannerPage.waitForPlannerView(); // Return to work view - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); await workViewPage.waitForTaskList(); // Tasks should still be present diff --git a/e2e/tests/planner/planner-navigation.spec.ts b/e2e/tests/planner/planner-navigation.spec.ts index 342494fd6a..deb390ac2f 100644 --- a/e2e/tests/planner/planner-navigation.spec.ts +++ b/e2e/tests/planner/planner-navigation.spec.ts @@ -14,7 +14,7 @@ test.describe('Planner Navigation', () => { workViewPage, }) => { // Start at work view - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); await workViewPage.waitForTaskList(); await expect(page).toHaveURL(/\/tag\/TODAY/); @@ -24,7 +24,7 @@ test.describe('Planner Navigation', () => { await expect(page).toHaveURL(/\/(planner|tasks)/); // Go back to work view - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); await workViewPage.waitForTaskList(); await expect(page).toHaveURL(/\/tag\/TODAY/); }); @@ -40,7 +40,7 @@ test.describe('Planner Navigation', () => { await plannerPage.waitForPlannerView(); // Go back to work view - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); await workViewPage.waitForTaskList(); // Task should still be there diff --git a/e2e/tests/planner/planner-scheduled-tasks.spec.ts b/e2e/tests/planner/planner-scheduled-tasks.spec.ts index e524d05516..aa1efd467b 100644 --- a/e2e/tests/planner/planner-scheduled-tasks.spec.ts +++ b/e2e/tests/planner/planner-scheduled-tasks.spec.ts @@ -53,7 +53,7 @@ test.describe('Planner Scheduled Tasks', () => { await plannerPage.waitForPlannerView(); // Go back to work view - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); await workViewPage.waitForTaskList(); // Task should still exist diff --git a/e2e/tests/planner/planner-time-estimates.spec.ts b/e2e/tests/planner/planner-time-estimates.spec.ts index 980cb20e13..98d84cb231 100644 --- a/e2e/tests/planner/planner-time-estimates.spec.ts +++ b/e2e/tests/planner/planner-time-estimates.spec.ts @@ -72,7 +72,7 @@ test.describe('Planner Time Estimates', () => { await plannerPage.waitForPlannerView(); // Go back to work view - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); await workViewPage.waitForTaskList(); // Tasks should still be there diff --git a/e2e/tests/plugins/plugin-lifecycle.spec.ts b/e2e/tests/plugins/plugin-lifecycle.spec.ts index ca833c2d25..f0699af566 100644 --- a/e2e/tests/plugins/plugin-lifecycle.spec.ts +++ b/e2e/tests/plugins/plugin-lifecycle.spec.ts @@ -87,7 +87,7 @@ test.describe('Plugin Lifecycle', () => { }).toPass({ timeout: 10000, intervals: [500, 1000] }); // Go back to work view - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); }); test('disable plugin and verify cleanup', async ({ page }) => { @@ -107,7 +107,7 @@ test.describe('Plugin Lifecycle', () => { expect(disabled).toBe(true); // Go back to work view - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); // Wait for navigation and work view to be ready await page.locator('.route-wrapper').waitFor({ state: 'visible', timeout: 10000 }); diff --git a/e2e/tests/recurring/recurring-task.spec.ts b/e2e/tests/recurring/recurring-task.spec.ts index 2813df1912..e2791ce396 100644 --- a/e2e/tests/recurring/recurring-task.spec.ts +++ b/e2e/tests/recurring/recurring-task.spec.ts @@ -129,7 +129,7 @@ test.describe('Scheduled Task Operations', () => { await page.waitForTimeout(1000); // Navigate back to work view to verify app is responsive - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); await page.waitForLoadState('networkidle'); // Verify the app is still responsive diff --git a/e2e/tests/settings/settings-basic.spec.ts b/e2e/tests/settings/settings-basic.spec.ts index d23e3816de..e14c340053 100644 --- a/e2e/tests/settings/settings-basic.spec.ts +++ b/e2e/tests/settings/settings-basic.spec.ts @@ -126,7 +126,7 @@ test.describe('Settings', () => { await page.waitForLoadState('networkidle'); // Navigate back to work view via URL (more reliable) - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); await page.waitForLoadState('networkidle'); // Verify we're back at work view diff --git a/e2e/tests/work-view/work-view-features.spec.ts b/e2e/tests/work-view/work-view-features.spec.ts index cecf2b0d90..7be7606031 100644 --- a/e2e/tests/work-view/work-view-features.spec.ts +++ b/e2e/tests/work-view/work-view-features.spec.ts @@ -119,7 +119,7 @@ test.describe('Work View Features', () => { await expect(page.locator('.page-settings')).toBeVisible({ timeout: 10000 }); // Navigate back - await page.goto('/#/tag/TODAY'); + await page.goto('/#/tag/TODAY/tasks'); await workViewPage.waitForTaskList(); // Verify task is still there