Edits sometimes fail for new users erroneously reporting them as anon
Closed, DeclinedPublic
Actions

Assigned To

None

Authored By

	Jdlrobson
	Aug 8 2017, 10:05 PM

Description

This video demonstrates problem: https://saucelabs.com/beta/tests/d6529f0489274febbf9979e1f4b3e800/commands#255
Not sure if issue with beta cluster or software, but I suspect CentralAuth.

In Chrome browser
Register a new account on https://en.m.wikipedia.beta.wmflabs.org/
Visit a new page and click edit
Add some text
Hit save
Save fails

Inspecting the console I see an error relating to not being able to create while anonymous
A refresh of the page shows me as logged out
Refreshing again I am logged in

I think something in CentralAuth is not behaving here as it should.

This issue is causing MobileFrontend's browser tests to occasionally fail:
https://integration.wikimedia.org/ci/view/Reading-Web/job/selenium-MinervaNeue/56/BROWSER=chrome,MEDIAWIKI_ENVIRONMENT=beta,PLATFORM=Linux,label=BrowserTests/

Related Objects

Mentioned In: T170890: [4 hrs] MinervaNeue browser test are flaking (waiting for {:class=>"mw-notification", :tag_name=>"div"} to become present )
Mentioned Here: T139740: Port Selenium tests from Ruby to Node.js

Event Timeline

Jdlrobson created this task.Aug 8 2017, 10:05 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 8 2017, 10:05 PM

Jdlrobson added a project: Web-Team-Backlog (Tracking).Aug 8 2017, 10:06 PM

Change 370806 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/skins/MinervaNeue@master] QA: Workaround for T172835

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

gerritbot added a project: Patch-For-Review.Aug 9 2017, 2:00 PM

Jdlrobson mentioned this in T170890: [4 hrs] MinervaNeue browser test are flaking (waiting for {:class=>"mw-notification", :tag_name=>"div"} to become present ).Aug 9 2017, 2:02 PM

The patch is a workaround, not a fix.

Jdlrobson moved this task from Untriaged to Untag on the Web-Team-Backlog (Tracking) board.Aug 9 2017, 2:15 PM

Change 370806 abandoned by Jdlrobson:
QA: Workaround for T172835

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

zeljkofilipin removed a project: Browser-Tests-Infrastructure.Aug 14 2017, 2:28 PM

Re-adding tag. @zeljkofilipin while this bug is open, I believe the browser test infrastructure should do more to ensure that sign in is successful (feel free to open a new task if more appropriate)

The "I am logged into the mobile website" step uses the log_in method from mediawiki_selenium which always defaults to a new user.

In Minerva we have a specific step for new users "I am logged in as a new user" and it is not needed here (in fact it seems to be harming stability).
Please can we update the log_in step so that it uses MEDIAWIKI_USER by default unless requested?

Jdlrobson removed a project: MinervaNeue.Aug 16 2017, 8:46 PM

zeljkofilipin moved this task from Inbox to Ruby on the Browser-Tests-Infrastructure board.Aug 17 2017, 8:38 AM

zeljkofilipin added a project: User-zeljkofilipin.Aug 25 2017, 4:23 PM

Restricted Application added a subscriber: jeblad. · View Herald TranscriptAug 25 2017, 4:23 PM

jeblad unsubscribed.Aug 25 2017, 9:28 PM

• Deskana moved this task from To Triage to Freezer on the VisualEditor board.Aug 29 2017, 7:18 PM

Unlikely to ever be resolved because of T139740: Port Selenium tests from Ruby to Node.js.

MarcoAurelio moved this task from Backlog to Done on the MediaWiki-extensions-CentralAuth board.Nov 22 2017, 9:46 AM

Edits sometimes fail for new users erroneously reporting them as anonClosed, DeclinedPublicActions

Description

Related Objects

Event Timeline

Edits sometimes fail for new users erroneously reporting them as anon
Closed, DeclinedPublic
Actions