jenkins-bot has submitted this change. (
https://gerrit.wikimedia.org/r/c/pywikibot/core/+/894534 )
Change subject: [tests] use NeedRightsMixin within site_generators_tests
......................................................................
[tests] use NeedRightsMixin within site_generators_tests
Change-Id: I81d69741d13216f4bcb7f69ffb064215d60691e2
---
M tests/site_generators_tests.py
1 file changed, 17 insertions(+), 18 deletions(-)
Approvals:
Xqt: Looks good to me, approved
jenkins-bot: Verified
diff --git a/tests/site_generators_tests.py b/tests/site_generators_tests.py
index b2d5b01..86fcb78 100755
--- a/tests/site_generators_tests.py
+++ b/tests/site_generators_tests.py
@@ -1027,18 +1027,18 @@
"""Test recentchanges method requiring a user."""
login = True
+ rights = 'patrol'
def test_patrolled(self):
"""Test the site.recentchanges() with patrolled boolean
flags."""
mysite = self.site
for change in mysite.recentchanges(patrolled=True, total=5):
self.assertIsInstance(change, dict)
- if mysite.has_right('patrol'):
- self.assertIn('patrolled', change)
+ self.assertIn('patrolled', change)
+
for change in mysite.recentchanges(patrolled=False, total=5):
self.assertIsInstance(change, dict)
- if mysite.has_right('patrol'):
- self.assertNotIn('patrolled', change)
+ self.assertNotIn('patrolled', change)
class TestUserWatchedPages(DefaultSiteTestCase):
@@ -1046,29 +1046,22 @@
"""Test user watched pages."""
login = True
+ rights = 'viewmywatchlist'
def test_watched_pages(self):
"""Test the site.watched_pages() method."""
- if not self.site.has_right('viewmywatchlist'):
- self.skipTest('user {} cannot view its watch list'
- .format(self.site.user()))
-
gen = self.site.watched_pages(total=5, force=False)
self.assertIsInstance(gen.request, api.CachedRequest)
for page in gen:
self.assertIsInstance(page, pywikibot.Page)
- # repeat to use the cache
- gen = self.site.watched_pages(total=5, force=False)
+
+ gen.restart() # repeat to use the cache
self.assertIsInstance(gen.request, api.CachedRequest)
for page in gen:
self.assertIsInstance(page, pywikibot.Page)
def test_watched_pages_uncached(self):
"""Test the site.watched_pages() method
uncached."""
- if not self.site.has_right('viewmywatchlist'):
- self.skipTest('user {} cannot view its watch list'
- .format(self.site.user()))
-
gen = self.site.watched_pages(total=5, force=True)
self.assertIsInstance(gen.request, api.Request)
self.assertFalse(issubclass(gen.request_class, api.CachedRequest))
@@ -1440,13 +1433,10 @@
"""Test site method watchlist_revs()."""
login = True
+ rights = 'viewmywatchlist'
def test_watchlist_revs(self):
"""Test the site.watchlist_revs() method."""
- if not self.site.has_right('viewmywatchlist'):
- self.skipTest('user {} cannot view its watch list'
- .format(self.site.user()))
-
mysite = self.get_site()
wl = list(mysite.watchlist_revs(total=10))
self.assertLessEqual(len(wl), 10)
--
To view, visit
https://gerrit.wikimedia.org/r/c/pywikibot/core/+/894534
To unsubscribe, or for help writing mail filters, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: I81d69741d13216f4bcb7f69ffb064215d60691e2
Gerrit-Change-Number: 894534
Gerrit-PatchSet: 2
Gerrit-Owner: Xqt <info(a)gno.de>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged