Skip to content

chore(test) unify notification interaction in e2e tests to stabilize them#1981

Open
edlerd wants to merge 1 commit intocanonical:mainfrom
edlerd:e2e-notifications
Open

chore(test) unify notification interaction in e2e tests to stabilize them#1981
edlerd wants to merge 1 commit intocanonical:mainfrom
edlerd:e2e-notifications

Conversation

@edlerd
Copy link
Copy Markdown
Collaborator

@edlerd edlerd commented Apr 28, 2026

Done

  • chore(test) unify notification interaction in e2e tests to stabilize them

Fixes WD-36302

@webteam-app
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@omarelkashef omarelkashef left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for fixing the CI. It looks great, one suggestion below.

Comment thread tests/helpers/instances.ts Outdated
@kimanhou
Copy link
Copy Markdown
Contributor

Should we also tackle occurrences of await page.getByTestId("notification-close-button").click(); ?

@edlerd edlerd force-pushed the e2e-notifications branch 4 times, most recently from 5d535d1 to 572235c Compare April 28, 2026 15:16
…them WD-36302

Signed-off-by: David Edler <david.edler@canonical.com>
@edlerd edlerd force-pushed the e2e-notifications branch from 572235c to 17a3663 Compare April 28, 2026 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants