Mirrors/super-productivity
1
0
Fork 0
mirror of https://github.com/johannesjo/super-productivity.git synced 2026-07-17 16:37:43 +00:00
Code Issues Projects Releases Wiki Activity

test(daily-summary): guard /active/daily-summary route resolution (#8449)

Browse source 
Add an e2e regression guard asserting the context-resolved
/active/daily-summary route opens the daily summary instead of falling
through to the wildcard and redirecting to the start page. The unit spec
only pins the navigation string; this verifies the string is a navigable
route.
This commit is contained in:
Johannes Millan 2026-06-17 13:33:13 +02:00
parent 07d3e1d722
commit 797451427e
1 changed files with 16 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

16
e2e/tests/daily-summary/daily-summary.spec.ts
View file

@ -15,6 +15,22 @@ test.describe('Daily Summary', () => {
await expect(doneHeadline).toContainText('Take a moment to celebrate');
});
// Regression guard for #8449: the before-close "Finish Day" dialog navigates
// via the context-resolved `/active/daily-summary` route. A bare
// `/daily-summary` has no matching route and silently redirects to the start
// page, so the summary never opens. Assert the `/active/...` route actually
// resolves to the daily summary instead of falling through to the wildcard.
test('the /active/daily-summary route resolves to the daily summary', async ({
page,
}) => {
await page.goto('/#/active/daily-summary');
// ActiveWorkContextGuard rewrites it to the active context (TODAY tag).
await page.waitForURL(/daily-summary/);
await expect(page.locator('daily-summary')).toBeVisible();
await page.waitForSelector('.done-headline', { state: 'visible' });
});
test('show any added task in table', async ({ page, workViewPage }) => {
// First navigate to work view to add task
await page.goto('/');