fix(07): WR-05 drop redundant unroute calls (per-test context isolation handles cleanup)
This commit is contained in:
@@ -87,9 +87,9 @@ test.describe('Rule 5 — lists empty state (network-simulated, seeded DB untouc
|
||||
|
||||
// Body contains "Tap + to create your first shared list" — use partial match
|
||||
await expect(page.getByText(/Tap \+ to create/)).toBeVisible()
|
||||
|
||||
// Unroute so the mock does not leak to subsequent tests (T-07-11)
|
||||
await page.unroute('/api/lists')
|
||||
// No manual unroute (WR-05): Playwright gives each test a fresh page/context, so route
|
||||
// handlers do not leak across tests; a trailing unroute also never runs if an `expect`
|
||||
// above throws.
|
||||
})
|
||||
|
||||
test('no horizontal overflow in empty lists state (Rule 2)', async ({ page }) => {
|
||||
@@ -115,7 +115,6 @@ test.describe('Rule 5 — lists empty state (network-simulated, seeded DB untouc
|
||||
overflow.scrollWidth,
|
||||
`scrollWidth (${overflow.scrollWidth}) must be ≤ clientWidth (${overflow.clientWidth}) in empty lists state`,
|
||||
).toBeLessThanOrEqual(overflow.clientWidth)
|
||||
|
||||
await page.unroute('/api/lists')
|
||||
// No manual unroute (WR-05): per-test context isolation handles route cleanup.
|
||||
})
|
||||
})
|
||||
|
||||
Reference in New Issue
Block a user