jenkins-bot submitted this change.

View Change

[tests] Skip when watchlist rights are unavailable

Bug: T279286
Change-Id: I6dc140afcc755ff83b21e73877dbbd27d1b16547
---
M tests/page_tests.py
M tests/site_tests.py
2 files changed, 16 insertions(+), 0 deletions(-)

diff --git a/tests/page_tests.py b/tests/page_tests.py
index f6ce4f8..7c1d476 100644
--- a/tests/page_tests.py
+++ b/tests/page_tests.py
@@ -1020,6 +1020,10 @@
 
     def test_watch(self):
         """Test Page.watch, with and without unwatch enabled."""
+        if not self.site.has_right('editmywatchlist'):
+            self.skipTest('user {} cannot edit its watch list'
+                          .format(self.site.user()))
+
         # Note: this test uses the userpage, so that it is unwatched and
         # therefore is not listed by script_tests test_watchlist_simulate.
         userpage = self.get_userpage()
diff --git a/tests/site_tests.py b/tests/site_tests.py
index 0e125b5..0b01ded 100644
--- a/tests/site_tests.py
+++ b/tests/site_tests.py
@@ -1402,6 +1402,10 @@
 
     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:
@@ -1414,6 +1418,10 @@
 
     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))
@@ -1806,6 +1814,10 @@
 
     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 change 676852. To unsubscribe, or for help writing mail filters, visit settings.