jenkins-bot merged this change.

View Change

Approvals: Xqt: Looks good to me, approved jenkins-bot: Verified
[bugfix] Fix parameters in OauthLoginManager()

Fix parameters when calling OauthLoginManager(), after
removal of sysop parameter.

Change-Id: I56ede124fe2f6f3add26a315f22b127cd34e66a2
---
M scripts/login.py
M tests/oauth_tests.py
2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/scripts/login.py b/scripts/login.py
index cfabfb6..e90f814 100755
--- a/scripts/login.py
+++ b/scripts/login.py
@@ -80,8 +80,7 @@

def _oauth_login(site):
consumer_key, consumer_secret = _get_consumer_token(site)
- login_manager = OauthLoginManager(consumer_secret, False, site,
- consumer_key)
+ login_manager = OauthLoginManager(consumer_secret, site, consumer_key)
login_manager.login()
identity = login_manager.identity
if identity is None:
diff --git a/tests/oauth_tests.py b/tests/oauth_tests.py
index cc44abd..00cbdee 100644
--- a/tests/oauth_tests.py
+++ b/tests/oauth_tests.py
@@ -79,7 +79,7 @@
self.assertIsNotNone(login_manager.access_token)
self.assertIsInstance(login_manager.identity, dict)
self.assertEqual(login_manager.identity['username'],
- self.site.username(sysop=False))
+ self.site.username())


if __name__ == '__main__': # pragma: no cover

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

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I56ede124fe2f6f3add26a315f22b127cd34e66a2
Gerrit-Change-Number: 569101
Gerrit-PatchSet: 1
Gerrit-Owner: Mpaa <mpaa.wiki@gmail.com>
Gerrit-Reviewer: D3r1ck01 <xsavitar.wiki@aol.com>
Gerrit-Reviewer: Xqt <info@gno.de>
Gerrit-Reviewer: jenkins-bot (75)