jenkins-bot has submitted this change. ( https://gerrit.wikimedia.org/r/c/pywikibot/core/+/894206 )
Change subject: [tests] Skip test on proofwiki on github action
......................................................................
[tests] Skip test on proofwiki on github action
Access to proofwiki is forbidden (status 403) if called from
github action. Skip this test then instead of expect the failure.
Bug: T331223
Change-Id: Ice47cf3ae7883bfa4012e7b8fea4b7a2008d9dd7
---
M tests/site_detect_tests.py
1 file changed, 16 insertions(+), 4 deletions(-)
Approvals:
Xqt: Looks good to me, approved
jenkins-bot: Verified
diff --git a/tests/site_detect_tests.py b/tests/site_detect_tests.py
index 918d76f..6173ff7 100755
--- a/tests/site_detect_tests.py
+++ b/tests/site_detect_tests.py
@@ -18,7 +18,7 @@
from pywikibot.site_detect import MWSite
from tests.aspects import PatchingTestCase, TestCase
-from tests.utils import DrySite, expected_failure_if, skipping
+from tests.utils import DrySite, skipping
class SiteDetectionTestCase(TestCase):
@@ -58,7 +58,6 @@
"""Test detection of MediaWiki sites."""
standard_version_sites = (
- # 'http://www.proofwiki.org/wiki/$1',
'http://www.ck-wissen.de/ckwiki/index.php?title=$1',
'http://en.citizendium.org/wiki/$1',
# Server that hosts www.wikichristian.org is unreliable - it
@@ -113,10 +112,10 @@
with self.subTest(url=urlparse(url).netloc):
self.assertSite(url)
- @expected_failure_if(os.getenv('GITHUB_ACTIONS')) # T331223
def test_proofreadwiki(self):
"""Test detection of proofwiki.org site."""
- # test is failing on github, see T331223
+ if os.getenv('GITHUB_ACTIONS'):
+ self.skipTest('Skip test on github due to T331223')
self.assertSite('http://www.proofwiki.org/wiki/$1')
def test_non_standard_version_sites(self):
--
To view, visit https://gerrit.wikimedia.org/r/c/pywikibot/core/+/894206
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: Ice47cf3ae7883bfa4012e7b8fea4b7a2008d9dd7
Gerrit-Change-Number: 894206
Gerrit-PatchSet: 1
Gerrit-Owner: Xqt <info(a)gno.de>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged
jenkins-bot has submitted this change. ( https://gerrit.wikimedia.org/r/c/pywikibot/core/+/892415 )
Change subject: Revert "[IMPR] flush: log the traceback when exiting by exception"
......................................................................
Revert "[IMPR] flush: log the traceback when exiting by exception"
This reverts commit d90ef8ab9d057d241a8547d446b3d5da127299f0.
Reason for revert: Did not work as indented - only showed the object
Change-Id: Ic50846d7fa4d18c25c56cd387c3cf5b79bbd38e2
---
M pywikibot/comms/http.py
1 file changed, 13 insertions(+), 1 deletion(-)
Approvals:
JJMC89: Looks good to me, approved
jenkins-bot: Verified
diff --git a/pywikibot/comms/http.py b/pywikibot/comms/http.py
index 1a8f580..7828acf 100644
--- a/pywikibot/comms/http.py
+++ b/pywikibot/comms/http.py
@@ -115,7 +115,6 @@
session.close()
if hasattr(sys, 'last_type'):
- log(sys.last_traceback)
critical(f'Exiting due to uncaught exception {sys.last_type}')
log('Network session closed.')
--
To view, visit https://gerrit.wikimedia.org/r/c/pywikibot/core/+/892415
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: Ic50846d7fa4d18c25c56cd387c3cf5b79bbd38e2
Gerrit-Change-Number: 892415
Gerrit-PatchSet: 1
Gerrit-Owner: JJMC89 <JJMC89.Wikimedia(a)gmail.com>
Gerrit-Reviewer: JJMC89 <JJMC89.Wikimedia(a)gmail.com>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged