Skip to content

Commit 2259e07

Browse files
committed
test(glossary): fix flaky "Async Delete - multiple deletes with mixed results"
Re-anchor on the glossary page after deleting the currently-selected glossary. Optimistically removing the active glossary navigates the page away from /glossary before the sidebar re-renders the remaining items, causing the final visibility checks to miss the untouched glossary C.
1 parent c9770bf commit 2259e07

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/Glossary.spec.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1456,6 +1456,8 @@ test.describe('Glossary tests', () => {
14561456
// Delete A (succeeds - not mocked, real deletion)
14571457
await selectActiveGlossary(page, glossaryA.data.displayName);
14581458
await initiateDelete(page);
1459+
await sidebarClick(page, SidebarItem.GLOSSARY);
1460+
await expectGlossaryNotVisible(page, glossaryA.data.displayName);
14591461

14601462
// Delete B (fails via mocked WebSocket event)
14611463
await selectActiveGlossary(page, glossaryB.data.displayName);
@@ -1466,6 +1468,8 @@ test.describe('Glossary tests', () => {
14661468
emitDeleteFailure(jobIdB, glossaryB.data.name);
14671469
await refetch;
14681470

1471+
await sidebarClick(page, SidebarItem.GLOSSARY);
1472+
14691473
// A deleted, B restored, C untouched
14701474
await expect(
14711475
page.getByRole('menuitem', { name: glossaryA.data.displayName })

0 commit comments

Comments
 (0)