Xqt has submitted this change. ( https://gerrit.wikimedia.org/r/c/pywikibot/core/+/1154265?usp=email )

Change subject: tests: Remove login attribute from RequireLoginMixin ......................................................................

tests: Remove login attribute from RequireLoginMixin

The login attribute is already given if the RequireLoginMixin is added to the base class:

if dct.get('login'): bases = cls.add_base(bases, RequireLoginMixin)

Change-Id: I26ac9271cf05270456f0a3af9a7adb9de913f33d --- M tests/aspects.py 1 file changed, 0 insertions(+), 2 deletions(-)

Approvals: Xqt: Verified; Looks good to me, approved

diff --git a/tests/aspects.py b/tests/aspects.py index 0bca615..fe0f2df 100644 --- a/tests/aspects.py +++ b/tests/aspects.py @@ -559,8 +559,6 @@

"""Run tests against a specific site, with a login."""

- login = True - @classmethod def require_site_user(cls, family, code): """Check the user config has a valid login to the site."""