Open Bug 1049082 Opened 6 years ago Updated 6 months ago

Page markup for the watch notification message differs between dev and staging/prod

Categories

(Websites :: wiki.mozilla.org, defect)

defect
Not set
normal

Tracking

(Not tracked)

People

(Reporter: bsilverberg, Assigned: spike)

References

()

Details

(Whiteboard: [fromAutomation])

One of our automated tests for the wiki has started failing when run against dev [1] because the markup differs between dev and stage [2] for the message that is displayed when watching and unwatching a page. Perhaps this difference stems from the two sites running different versions of MediaWiki? I would like to resolve this issue and get the test passing on both dev and staging again, but I am not sure how to best address it.

Is it expected that stage will be updated to have similar behaviour to dev in the near future?

[1] https://wiki-dev.allizom.org/
[2] https://wiki.allizom.org/
The same question stands for prod as well, as we run our tests on all three environments.
Summary: Page markup for the watch notification message differs between dev and staging → Page markup for the watch notification message differs between dev and staging/prod
For future reference, the change that needs to be made to support the markup on dev is in /pages/watch_page.py and is:

-    _watchlist_message_locator = (By.CSS_SELECTOR, 'div.mw-js-message-ajaxwatch > p')
+    _watchlist_message_locator = (By.CSS_SELECTOR, 'div.mw-notification-tag-watch-self')
OS: Mac OS X → All
Hardware: x86 → All
We're in the middle of upgrading Mozilla Wiki to the most current version of Mediawiki (1.23 branch). Dev has been updated to 1.23 and we're testing it now. Stage and production will be made to match Dev once we've completed upgrading and testing Dev. 

Also note that as part of this upgrade, we'll be changing the default theme of the wiki to Vector (from GMO). This might change some of the rendered css which might affect your tests.
Depends on: 1032351
Thanks for the update :ckoehler :)

I will leave this bug open as a reminder to update the test when the changes make it to staging and production.
No longer depends on: 1032351
Depends on: 1058914

Mass-removing myself from cc; search for 12b9dfe4-ece3-40dc-8d23-60e179f64ac1 or any reasonable part thereof, to mass-delete these notifications (and sorry!)

Not sure if this was ever resolved or not - but we are currently trying to migrate from mediawiki version 1.27 to version 1.31 (and then probably 1.35 which is due to be released in August 2020).

Adding self as assignee to keep this bug visible.

Spike

Assignee: nobody → spike
You need to log in before you can comment on or make changes to this bug.