The current browser tests are checking for the old flyout. They should be adjusted to test the new split notifications change.
Description
Description
Details
Details
Related Gerrit Patches:
mediawiki/extensions/Echo : master | [wip] Add browser test to "mark all as read" in Echo notifications |
mediawiki/extensions/Echo : master | Add browser test for notifications in Echo |
Event Timeline
Comment Actions
Change 242016 had a related patch set uploaded (by Mooeypoo):
[wip] Add browser test for notifications in Echo
Comment Actions
Change 242422 had a related patch set uploaded (by Mooeypoo):
Add browser test to "mark all as read" in Echo notifications
Comment Actions
https://gerrit.wikimedia.org/r/#/c/242016/12/tests/browser/features/notifications.feature - Feature: Testing notification types is passing since Build #650 (https://integration.wikimedia.org/ci/view/BrowserTests/view/Echo/job/browsertests-Echo-en.wikipedia.beta.wmflabs.org-linux-chrome-sauce/)
Comment Actions
Change 242422 abandoned by Mooeypoo:
[wip] Add browser test to "mark all as read" in Echo notifications
Reason:
Moving to nodejs at some point anyways. Tracked in the ticket.