Merge pull request 'fix(test): await async data in auditLog test' (#51) from fix/audit-log-test into master
Some checks failed
Auto Tag / autotag (push) Successful in 15s
Auto Tag / wiki-sync (push) Successful in 15s
Test / rust-fmt-check (push) Successful in 1m4s
Test / frontend-typecheck (push) Successful in 1m22s
Auto Tag / changelog (push) Successful in 53s
Test / frontend-tests (push) Successful in 1m29s
Test / rust-clippy (push) Successful in 8m5s
Test / rust-tests (push) Successful in 11m30s
Auto Tag / build-linux-amd64 (push) Successful in 16m13s
Auto Tag / build-linux-arm64 (push) Successful in 17m54s
Auto Tag / build-windows-amd64 (push) Successful in 18m51s
Auto Tag / build-macos-arm64 (push) Failing after 11m59s

This commit is contained in:
sarman 2026-04-20 01:21:55 +00:00
commit 5a12718566

View File

@ -42,11 +42,8 @@ describe("Audit Log", () => {
it("displays audit entries", async () => {
render(<Security />);
// Wait for audit log to load
await screen.findByText("Audit Log");
// Check that the table has rows (header + data rows)
const table = screen.getByRole("table");
// Wait for table to appear after async audit data loads
const table = await screen.findByRole("table");
expect(table).toBeInTheDocument();
const rows = screen.getAllByRole("row");
@ -56,9 +53,7 @@ describe("Audit Log", () => {
it("provides way to view transmitted data details", async () => {
render(<Security />);
await screen.findByText("Audit Log");
// Should have View/Hide buttons for expanding details
// Wait for async data to load and render the table
const viewButtons = await screen.findAllByRole("button", { name: /View/i });
expect(viewButtons.length).toBeGreaterThan(0);
});
@ -66,14 +61,13 @@ describe("Audit Log", () => {
it("details column or button exists for viewing data", async () => {
render(<Security />);
await screen.findByText("Audit Log");
// Wait for async data to load and render the table
await screen.findByRole("table");
// The audit log should have a Details column header
const detailsHeader = screen.getByText("Details");
expect(detailsHeader).toBeInTheDocument();
// Should have view buttons
const viewButtons = await screen.findAllByRole("button", { name: /View/i });
const viewButtons = screen.getAllByRole("button", { name: /View/i });
expect(viewButtons.length).toBe(2); // One for each mock entry
});
});