[Pywikibot-commits] [Gerrit] ...core[master]: [bugfix] site_tests.TestAlldeletedrevisionsAsUser fails

jenkins-bot has submitted this change. ( https://gerrit.wikimedia.org/r/c/pywikibot/core/+/707869 ) Change subject: [bugfix] site_tests.TestAlldeletedrevisionsAsUser fails ...................................................................... [bugfix] site_tests.TestAlldeletedrevisionsAsUser fails This test only passed when the test user lacked any deleted revisions. This both corrects that and provides more helpful output in the case of failures. Bug: T287393 Change-Id: I5c04d8bc928472ceb15fc0f58130c2c20cabcff0 --- M tests/site_tests.py 1 file changed, 16 insertions(+), 13 deletions(-) Approvals: Xqt: Looks good to me, but someone else must approve JJMC89: Looks good to me, approved jenkins-bot: Verified diff --git a/tests/site_tests.py b/tests/site_tests.py index 56a9789..1ea2c8d 100644 --- a/tests/site_tests.py +++ b/tests/site_tests.py @@ -1604,22 +1604,25 @@ def setUpClass(cls): """Skip test if necessary.""" super().setUpClass() - if cls.site.mw_version < '1.34': - cls.skipTest(cls, 'site.alldeletedrevisions() needs mw 1.34') + if cls.site.mw_version < '1.25': + cls.skipTest(cls, 'site.alldeletedrevisions() needs mw 1.25') def test_basic(self): """Test the site.alldeletedrevisions() method.""" mysite = self.get_site() drev = list(mysite.alldeletedrevisions(user=mysite.user(), total=10)) - self.assertTrue(all(isinstance(data, dict) - for data in drev)) - self.assertTrue(all('revisions' in data - and isinstance(data['revisions'], dict) - for data in drev)) - self.assertTrue(all('user' in rev - and rev['user'] == mysite.user() - for data in drev - for rev in data)) + + if not drev: + self.skipTest('No deleted revisions available') + + for data in drev: + with self.subTest(data=data): + self.assertIsInstance(data, dict) + self.assertIn('revisions', data) + self.assertIsInstance(data['revisions'], list) + + for rev in data['revisions']: + self.assertEqual(mysite.user(), rev.get('user')) def test_namespaces(self): """Test the site.alldeletedrevisions() method using namespaces.""" @@ -1763,8 +1766,8 @@ def test_prefix(self): """Test the site.alldeletedrevisions() method with prefix.""" mysite = self.get_site() - if mysite.mw_version < '1.34': - self.skipTest('site.alldeletedrevisions() needs mw 1.34') + if mysite.mw_version < '1.25': + self.skipTest('site.alldeletedrevisions() needs mw 1.25') for data in mysite.alldeletedrevisions(prefix='John', total=5): self.assertIsInstance(data, dict) -- To view, visit https://gerrit.wikimedia.org/r/c/pywikibot/core/+/707869 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: I5c04d8bc928472ceb15fc0f58130c2c20cabcff0 Gerrit-Change-Number: 707869 Gerrit-PatchSet: 3 Gerrit-Owner: Damian &lt;atagar1(a)gmail.com&gt; Gerrit-Reviewer: JJMC89 &lt;JJMC89.Wikimedia(a)gmail.com&gt; Gerrit-Reviewer: Xqt &lt;info(a)gno.de&gt; Gerrit-Reviewer: jenkins-bot Gerrit-MessageType: merged