Github/authentik
1
0
Fork 0
mirror of https://github.com/goauthentik/authentik.git synced 2026-08-30 18:51:39 -07:00
Code Issues Packages Projects Releases Wiki Activity

web/test: cover path-segment settings tabs; refresh legacy-shim note

Browse Source 
Add a routing test asserting the settings tabs track the path: a deep
load selects the named tab, switching writes the segment, and the default
tab drops back to the bare settings path. Update the legacy-hash-shim
test's note now that tabs are path-routed -- the shim deliberately does
not translate a legacy ?page= into a path segment.
This commit is contained in:
Teffen Ellis
2026-08-13 02:24:00 +01:00
committed by Teffen Ellis
parent 6bcaa1dde8
commit 0fbb5633db
1 changed files with 27 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

30
web/test/browser/110-user-routing.test.ts
View File

@@ -37,6 +37,29 @@ test.describe("User interface routing", () => {
});
});
test("Settings tabs are path segments", async ({ session, navigator, page }) => {
await test.step("Deep-load a specific tab by path", async () => {
await session.login({ to: `${SETTINGS_PATHNAME}/sessions` });
});
await test.step("The tab named in the path is selected", async () => {
await expect(
page.getByRole("tab", { name: "Sessions" }),
"Sessions tab is selected from the path",
).toHaveAttribute("aria-selected", "true");
});
await test.step("Switching tabs writes the segment to the path", async () => {
await page.getByRole("tab", { name: "Credentials" }).click();
await navigator.waitForPathname(`${SETTINGS_PATHNAME}/credentials`);
});
await test.step("The default tab drops back to the bare settings path", async () => {
await page.getByRole("tab", { name: "User details" }).click();
await navigator.waitForPathname(SETTINGS_PATHNAME);
});
});
test("Back and forward traverse pushState navigations", async ({
session,
navigator,
@@ -117,9 +140,10 @@ test.describe("User interface routing", () => {
});
// Shim-only compatibility (see the plan's compatibility decision): the boot
// shim translates the legacy hash to the correct *page* and carries `?page=`
// forward, but this plan adds no `?page`→`activateTab` bridge, so the tab is
// NOT auto-selected until `ak-tabs` flips with the admin interface. Assert the
// shim translates the legacy hash to the page and carries `?page=` forward,
// but tabs are now tracked as path segments, not `?page=`. The shim is not
// extended to translate a legacy `?page=` into a path segment, so the tab is
// NOT auto-selected — a deliberate compatibility boundary. Assert the
// page-level redirect only.
await test.step("The settings page renders at the translated path", async () => {
await expect(