Xqt submitted this change.

View Change

Approvals: Xqt: Verified; Looks good to me, approved
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(-)

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."""

To view, visit change 1154265. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: I26ac9271cf05270456f0a3af9a7adb9de913f33d
Gerrit-Change-Number: 1154265
Gerrit-PatchSet: 1
Gerrit-Owner: Xqt <info@gno.de>
Gerrit-Reviewer: Xqt <info@gno.de>
Gerrit-Reviewer: jenkins-bot