Skip to content

fix(test): flaky persona / task filter spec failure #27880

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
chirag-madlani merged 4 commits into from
May 4, 2026
Merged

fix(test): flaky persona / task filter spec failure #27880

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
f891cd9
fix(test): flaky persona / task filter spec failure
chirag-madlani May 4, 2026
0fd1be0
fix wait for task card
chirag-madlani May 4, 2026
e603f96
fix checkstyle
chirag-madlani May 4, 2026
569e283
fix for task filter widget
chirag-madlani May 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions openmetadata-ui/src/main/resources/ui/playwright/e2e/Flow/CustomizeWidgets.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -203,6 +203,8 @@ test('Activity Feed Widget', async ({ page }) => {
await test.step('Test widget filters', async () => {
await waitForAllLoadersToDisappear(page);
await waitForAllLoadersToDisappear(page, 'entity-list-skeleton');
// wait for first card visible before applying filters
await expect(page.getByTestId('task-feed-card').first()).toBeVisible();
Comment thread
chirag-madlani marked this conversation as resolved.
Outdated
await verifyActivityFeedFilters(page, widgetKey);
});

Expand Down
13 changes: 13 additions & 0 deletions openmetadata-ui/src/main/resources/ui/playwright/e2e/Flow/PersonaFlow.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -307,6 +307,19 @@ test.describe.serial('Default persona setting and removal flow', () => {
await afterAction();
});

test.afterAll(
'Teardown user for default persona flow',
async ({ browser }) => {
const { apiContext, afterAction } = await createNewPage(browser);
await Promise.all([
user.delete(apiContext),
persona1.delete(apiContext),
persona2.delete(apiContext),
]);
Comment on lines +314 to +318
await afterAction();
Comment thread
chirag-madlani marked this conversation as resolved.
}
);

test('Set and remove default persona should work properly', async ({
adminPage,
browser,
Expand Down
Loading