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

test(adminfront): fix failing E2E tests for audit and users

Browse Source 
- Relaxed audit log mock route matching to prevent empty state failures
- Fixed strict mode violation on appointment '추가' button by scoping to tabpanel
- Fixed UserDetailPage typescript compilation error during test build
This commit is contained in:
조찬영
2026-05-29 16:10:01 +09:00
parent 2da470922b
commit 8d2e2c58fe
3 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
adminfront/src/features/users/UserDetailPage.tsx
View File

@@ -661,15 +661,15 @@ function UserDetailPage() {
string, string,
Record<string, string | number | boolean> Record<string, string | number | boolean>
>) || {}), >) || {}),
sub_email: (Array.isArray(user.metadata?.sub_email) sub_email: Array.isArray(user.metadata?.sub_email)
? user.metadata.sub_email ? user.metadata.sub_email
: typeof user.metadata?.sub_email === "string" : typeof user.metadata?.sub_email === "string"
? user.metadata.sub_email ? user.metadata.sub_email
.split(/[;,\n\r\t]/) .split(/[;,\n\r\t]/)
.map((e) => e.trim()) .map((e) => e.trim())
.filter((e) => e.includes("@")) .filter((e) => e.includes("@"))
: []) as unknown[], : [],
}, } as any,
}); });
const isUserHanmacFamily = isHanmacFamilyUser( const isUserHanmacFamily = isHanmacFamilyUser(
user, user,

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

@@ -59,7 +59,7 @@ test.describe("Audit Logs Management", () => {
}; };
test("should load initial logs and display correctly", async ({ page }) => { test("should load initial logs and display correctly", async ({ page }) => {
await page.route("**/api/v1/audit?limit=50*", async (route) => { await page.route("**/api/v1/audit*", async (route) => {
return route.fulfill({ return route.fulfill({
json: { items: generateMockLogs(20, 0), total: 20 }, json: { items: generateMockLogs(20, 0), total: 20 },
headers: { "Access-Control-Allow-Origin": "*" }, headers: { "Access-Control-Allow-Origin": "*" },
@@ -83,7 +83,7 @@ test.describe("Audit Logs Management", () => {
page, page,
}) => { }) => {
let callCount = 0; let callCount = 0;
await page.route("**/api/v1/audit?limit=50*", async (route) => { await page.route("**/api/v1/audit*", async (route) => {
const logs = generateMockLogs(20, callCount * 20); const logs = generateMockLogs(20, callCount * 20);
callCount++; callCount++;
return route.fulfill({ return route.fulfill({
@@ -108,7 +108,7 @@ test.describe("Audit Logs Management", () => {
}); });
test("should filter logs by Action and User ID locally", async ({ page }) => { test("should filter logs by Action and User ID locally", async ({ page }) => {
await page.route("**/api/v1/audit?limit=50*", async (route) => { await page.route("**/api/v1/audit*", async (route) => {
return route.fulfill({ return route.fulfill({
json: { items: generateMockLogs(20, 0), total: 20 }, json: { items: generateMockLogs(20, 0), total: 20 },
headers: { "Access-Control-Allow-Origin": "*" }, headers: { "Access-Control-Allow-Origin": "*" },
@@ -146,7 +146,7 @@ test.describe("Audit Logs Management", () => {
}); });
test("should filter logs by Status locally", async ({ page }) => { test("should filter logs by Status locally", async ({ page }) => {
await page.route("**/api/v1/audit?limit=50*", async (route) => { await page.route("**/api/v1/audit*", async (route) => {
return route.fulfill({ return route.fulfill({
json: { items: generateMockLogs(20, 0), total: 20 }, json: { items: generateMockLogs(20, 0), total: 20 },
headers: { "Access-Control-Allow-Origin": "*" }, headers: { "Access-Control-Allow-Origin": "*" },

2
adminfront/tests/users.spec.ts
View File

@@ -721,7 +721,7 @@ test.describe("User Management", () => {
await expect(page.locator("input#department")).toHaveCount(0); await expect(page.locator("input#department")).toHaveCount(0);
await expect(page.getByText(/대표 소속/i)).toHaveCount(0); await expect(page.getByText(/대표 소속/i)).toHaveCount(0);
await page.getByRole("button", { name: /^추가$/i }).click(); await page.getByRole("tabpanel").getByRole("button", { name: /^추가$/i }).click();
await expect(page.getByTestId("appointment-row-0")).toBeVisible(); await expect(page.getByTestId("appointment-row-0")).toBeVisible();
await expect( await expect(
page.getByTestId("appointment-tenant-owner-line-0"), page.getByTestId("appointment-tenant-owner-line-0"),