Page MenuHomePhabricator

Tests: Ensure that RequireLoginMixin is in mro prior than NeedRightsMixin
Closed, ResolvedPublicBUG REPORT

Description

What happens?:
For example NeedRightsMixin can be used to ensure that needed rights are given during tests but the needed rigths can depend on the fqact that a user is logged in. Unfortunately the RequireLoginMixin is placed later in the mro so that the rights does not reflect the right situation after login is made. Currently NeedRightsMixin is mostly senseless.

Software version
Pywikibot 7.7.1: rPWBCd87931a

Event Timeline

Xqt triaged this task as Medium priority.Mar 6 2023, 8:46 AM
Xqt updated the task description. (Show Details)

Change 894530 had a related patch set uploaded (by Xqt; author: Xqt):

[pywikibot/core@master] [tests] Ensure that login is made prior than rights are checked

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

@Xqt, I was asked to review, but I cannot run tests properly because of the submodule update problem, and I don't know a test case for this, just subscribed to follow.

Change 894530 merged by jenkins-bot:

[pywikibot/core@master] [tests] Ensure that login is made prior than rights are checked

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