kevin/baron-sso
1
0
Fork 0
forked from baron/baron-sso
Code Pull Requests Activity

fix(adminfront): resolve biome lint errors and refine test stability

Browse Source
This commit is contained in:
조찬영
2026-05-29 18:26:22 +09:00
parent bf94c7a3d6
commit 520d7404cf
5 changed files with 21 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
adminfront/tests/audit.spec.ts
View File

@@ -120,8 +120,10 @@ test.describe("Audit Logs Management", () => {
const responsePromise = page.waitForResponse(/.*\/api\/v1\/audit.*/);
await page.goto("/audit-logs");
await responsePromise;
await expect(page.locator("tbody tr").first()).toBeVisible({ timeout: 15000 });
await expect(page.locator("tbody tr").first()).toBeVisible({
timeout: 15000,
});
await expect(page.locator("tbody tr")).toHaveCount(20);
const loadMoreBtn = page.getByRole("button", {
@@ -129,7 +131,7 @@ test.describe("Audit Logs Management", () => {
});
await expect(loadMoreBtn).toBeVisible({ timeout: 10000 });
await expect(loadMoreBtn).toBeEnabled();
const nextResponsePromise = page.waitForResponse(/.*\/api\/v1\/audit.*/);
await loadMoreBtn.click();
await nextResponsePromise;
@@ -144,7 +146,9 @@ test.describe("Audit Logs Management", () => {
await page.goto("/audit-logs");
await responsePromise;
await expect(page.locator("tbody tr").first()).toBeVisible({ timeout: 15000 });
await expect(page.locator("tbody tr").first()).toBeVisible({
timeout: 15000,
});
await expect(page.locator("tbody tr")).toHaveCount(20);
// Search by User ID
@@ -174,7 +178,9 @@ test.describe("Audit Logs Management", () => {
await page.goto("/audit-logs");
await responsePromise;
await expect(page.locator("tbody tr").first()).toBeVisible({ timeout: 15000 });
await expect(page.locator("tbody tr").first()).toBeVisible({
timeout: 15000,
});
await expect(page.locator("tbody tr")).toHaveCount(20);
// Select "Failure" status

12
adminfront/tests/tenants.spec.ts
View File

@@ -1,4 +1,4 @@
import { expect, test } from "@playwright/test";
import { type Download, expect, test } from "@playwright/test";
test.describe("Tenants Management", () => {
test.beforeEach(async ({ page }) => {
@@ -731,9 +731,11 @@ test.describe("Tenants Management", () => {
const item = page.getByTestId(testId);
await item.waitFor({ state: "attached" });
let downloadObj;
let downloadObj: Download;
await expect(async () => {
const downloadPromise = page.waitForEvent("download", { timeout: 10000 });
const downloadPromise = page.waitForEvent("download", {
timeout: 10000,
});
// Use dispatchEvent for more reliable trigger in webkit
await item.dispatchEvent("click");
downloadObj = await downloadPromise;
@@ -750,9 +752,7 @@ test.describe("Tenants Management", () => {
await expect(
page.getByTestId("tenant-export-with-ids-menu-item"),
).toBeVisible();
const exportWithIdsDownload = await safeDownload(
"tenant-export-with-ids-menu-item",
);
await safeDownload("tenant-export-with-ids-menu-item");
expect(exportUrl).toContain("includeIds=true");
await openDataManagementMenu();

4
adminfront/tests/users_bulk_secondary.spec.ts
View File

@@ -118,7 +118,7 @@ test.describe("Users Bulk Upload Secondary Emails", () => {
expect(bulkPayload.users).toHaveLength(1);
// The most important check - does it parse to the metadata
expect(bulkPayload!.users[0].metadata.sub_email).toContain("sub1@test.com");
expect(bulkPayload!.users[0].metadata.sub_email).toContain("sub2@test.com");
expect(bulkPayload?.users[0].metadata.sub_email).toContain("sub1@test.com");
expect(bulkPayload?.users[0].metadata.sub_email).toContain("sub2@test.com");
});
});