jenkins-bot has submitted this change. ( 
https://gerrit.wikimedia.org/r/c/pywikibot/core/+/763949 )

Change subject: [tests] remove code duplication in site_tests.py
......................................................................

[tests] remove code duplication in site_tests.py

Change-Id: I2fbe7424723454f2e2499cd2e632c7b2479614e1
---
M tests/site_tests.py
1 file changed, 6 insertions(+), 9 deletions(-)

Approvals:
  Xqt: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/tests/site_tests.py b/tests/site_tests.py
index 6cac649..11f52a1 100644
--- a/tests/site_tests.py
+++ b/tests/site_tests.py
@@ -2235,10 +2235,13 @@
                         reason='pywikibot unit tests',
                         target='File:T276726.png')

+        ts1 = pywikibot.Timestamp(2021, 3, 14, 18, 43, 57)
+        ts2 = pywikibot.Timestamp(2021, 3, 14, 18, 44, 17)
+
         fp1 = pywikibot.FilePage(site, 'File:T276726.png')
         site.loadimageinfo(fp1, history=True)
         for idx, v in fp1._file_revisions.items():
-            if v['timestamp'] == pywikibot.Timestamp(2021, 3, 14, 18, 43, 57):
+            if v['timestamp'] == ts1:
                 self.assertTrue(hasattr(v, 'userhidden'))

         # Multiple revisions
@@ -2249,9 +2252,7 @@
         fp2 = pywikibot.FilePage(site, 'File:T276726.png')
         site.loadimageinfo(fp2, history=True)
         for idx, v in fp2._file_revisions.items():
-            if v['timestamp'] == pywikibot.Timestamp(2021, 3, 14, 18, 43, 57):
-                self.assertTrue(hasattr(v, 'commenthidden'))
-            if v['timestamp'] == pywikibot.Timestamp(2021, 3, 14, 18, 44, 17):
+            if v['timestamp'] in (ts1, ts2):
                 self.assertTrue(hasattr(v, 'commenthidden'))

         # Concurrently show and hide
@@ -2263,11 +2264,7 @@
         fp3 = pywikibot.FilePage(site, 'File:T276726.png')
         site.loadimageinfo(fp3, history=True)
         for idx, v in fp3._file_revisions.items():
-            if v['timestamp'] == pywikibot.Timestamp(2021, 3, 14, 18, 43, 57):
-                self.assertFalse(hasattr(v, 'commenthidden'))
-                self.assertFalse(hasattr(v, 'userhidden'))
-                self.assertFalse(hasattr(v, 'filehidden'))
-            if v['timestamp'] == pywikibot.Timestamp(2021, 3, 14, 18, 44, 17):
+            if v['timestamp'] in (ts1, ts2):
                 self.assertFalse(hasattr(v, 'commenthidden'))
                 self.assertFalse(hasattr(v, 'userhidden'))
                 self.assertFalse(hasattr(v, 'filehidden'))

--
To view, visit https://gerrit.wikimedia.org/r/c/pywikibot/core/+/763949
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: I2fbe7424723454f2e2499cd2e632c7b2479614e1
Gerrit-Change-Number: 763949
Gerrit-PatchSet: 1
Gerrit-Owner: Xqt <[email protected]>
Gerrit-Reviewer: Xqt <[email protected]>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged
_______________________________________________
Pywikibot-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to