Page MenuHomePhabricator

Special:NewPagesFeed "new article appears in feed" test occasionally fails
Closed, ResolvedPublic

Description

Special:NewPagesFeed "new article appears in feed" test occasionally fails. I've seen this two or three times this week.

Most recent: https://integration.wikimedia.org/ci/job/wmf-quibble-selenium-php74-docker/65872/console

Error in "Special:NewPagesFeed.new article appears in feed"
Error: expect(received).toContain(expected) // indexOf

Expected substring: "NewArticle-0.22096586190889655-Iñtërnâtiônàlizætiøn"
Received string:    "Showing:
State (Unreviewed), Type (All others)
Sort by:
Newest
Oldest
Set filters ▸
 Auto-refresh Refresh list"
    at Context.<anonymous> (/workspace/src/extensions/PageTriage/tests/selenium/specs/newpagesfeed.js:44:51)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Event Timeline

Novem_Linguae added a subscriber: jsn.sherman.
Novem_Linguae subscribed.

PageTriage is in the gate, so we should probably prioritize this ticket. Can disable the test or try to fix it.

My understanding is that this is a race condition, and the stringification of the UI is happening between the time the load occurs and the response of the fetch for the articles.

Change 969197 had a related patch set uploaded (by Sohom Datta; author: Sohom Datta):

[mediawiki/extensions/PageTriage@master] selenium: Wait until articles are loaded before querying article names

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

Change 969197 merged by jenkins-bot:

[mediawiki/extensions/PageTriage@master] selenium: Wait until articles are loaded before querying article names

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

matmarex assigned this task to Soda.

Thanks, hopefully that fixes it (and if anyone sees the error again, we can reopen).