Maniphest T113081

Rewrite Echo browser tests for the new split notification change
Closed, ResolvedPublic
Actions

Description

The current browser tests are checking for the old flyout. They should be adjusted to test the new split notifications change.

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

Mooeypoo created this task.Sep 18 2015, 6:43 PM

Mooeypoo raised the priority of this task from to Needs Triage.

Mooeypoo updated the task description. (Show Details)

Mooeypoo added projects: Notifications, Collaboration-Team-Triage.

Mooeypoo added a subscriber: Mooeypoo.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptSep 18 2015, 6:43 PM

Catrope triaged this task as High priority.Sep 23 2015, 11:16 PM

Catrope set Security to None.

DannyH moved this task from Untriaged to Near-Term Interest on the Collaboration-Team-Triage board.Sep 23 2015, 11:16 PM

Catrope assigned this task to Mooeypoo.Sep 28 2015, 5:33 PM

Catrope edited projects, added Collaboration-Team-Archive-2015-2016; removed Collaboration-Team-Triage.

Catrope moved this task from Untriaged to In Development on the Collaboration-Team-Archive-2015-2016 board.Sep 28 2015, 5:43 PM

Change 242016 had a related patch set uploaded (by Mooeypoo):
[wip] Add browser test for notifications in Echo

https://gerrit.wikimedia.org/r/242016

gerritbot added a project: Patch-For-Review.Sep 29 2015, 12:09 AM

Change 242422 had a related patch set uploaded (by Mooeypoo):
Add browser test to "mark all as read" in Echo notifications

https://gerrit.wikimedia.org/r/242422

Mooeypoo moved this task from In Development to Needs Review on the Collaboration-Team-Archive-2015-2016 board.Sep 30 2015, 4:34 PM

SBisson moved this task from Needs Review to Code Review Started on the Collaboration-Team-Archive-2015-2016 board.Oct 1 2015, 1:59 PM

SBisson moved this task from Code Review Started to QA Review on the Collaboration-Team-Archive-2015-2016 board.Oct 13 2015, 5:27 PM

Change 242016 merged by jenkins-bot:
Add browser test for notifications in Echo

https://gerrit.wikimedia.org/r/242016

ReleaseTaggerBot added a project: MW-1.27-release (WMF-deploy-2015-10-27_(1.27.0-wmf.4)).Oct 13 2015, 9:01 PM

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/)

Etonkovidova moved this task from QA Review to Product Review on the Collaboration-Team-Archive-2015-2016 board.Oct 28 2015, 1:39 AM

Catrope closed this task as Resolved.Nov 22 2015, 10:50 PM

Catrope added a subscriber: Catrope.

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.

https://gerrit.wikimedia.org/r/242422