Page MenuHomePhabricator

Update browser tests to reflect absence of watchlist in main menu when anon
Closed, ResolvedPublic0.5 Story Points

Description

The test Menus open correct page for anonymous users.Watchlist URL is set correctly is currently failing on the beta cluster after our changes here. This test doesn't run on @integration. We should remove this test and I guess the code for the message box at the top of the login page.

Given I am using the mobile site
And I am on the "Main Page" page
When I click on "Watchlist" in the main navigation menu
Then I should see a message box at the top of the login page

Event Timeline

Jdlrobson removed dr0ptp4kt as the assignee of this task.Aug 24 2016, 5:08 PM
Jdlrobson moved this task from Needs Analysis to To Do on the Reading-Web-Sprint-79-Uh-oh board.
jhobs moved this task from To Do to Doing on the Reading-Web-Sprint-79-Uh-oh board.

This shouldn't be too hard jhobs so if you are having trouble grab me to pair Thursday so we can get our builds green again.

Let's make our builds green again! Also, if @Jdlrobson isn't around, then ping me @jhobs – I'm the one that broke the scenario after all!

Change 306715 had a related patch set uploaded (by Jhobs):
Remove irrelevant browser test for Watchlist

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

Only a really minor fix required…

MBinder_WMF set the point value for this task to 0.5.

Change 306715 merged by jenkins-bot:
Remove irrelevant browser test for Watchlist

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

jhobs removed jhobs as the assignee of this task.Aug 25 2016, 8:46 PM
jhobs added subscribers: bmansurov, Jhernandez.

@bmansurov or @Jhernandez should probably sign this one off.

I've triggered a build.

phuedx closed this task as Resolved.Aug 25 2016, 9:44 PM
phuedx claimed this task.

The build passed. @bmansurov, @Jhernandez: Feel free to do the same, if you wish.