jenkins-bot has submitted this change and it was merged.
Change subject: Deadlock problem fixed by using requests
......................................................................
Deadlock problem fixed by using requests
Bug: T71781
Change-Id: Iae9454cc0c7d4c815b2d9def7a8510d9d38f24e3
---
M scripts/makecat.py
M tests/script_tests.py
2 files changed, 3 insertions(+), 16 deletions(-)
Approvals:
John Vandenberg: Looks good to me, but someone else must approve
XZise: Looks good to me, approved
jenkins-bot: Verified
diff --git a/scripts/makecat.py b/scripts/makecat.py
index b2885f5..d1a80f9 100755
--- a/scripts/makecat.py
+++ b/scripts/makecat.py
@@ -194,7 +194,7 @@
main = True
workingcatname = ''
tocheck = DequeGenerator()
- for arg in pywikibot.handleArgs():
+ for arg in pywikibot.handle_args():
if arg.startswith('-nodate'):
skipdates = True
elif arg.startswith('-forward'):
diff --git a/tests/script_tests.py b/tests/script_tests.py
index bf11a8b..f93d126 100644
--- a/tests/script_tests.py
+++ b/tests/script_tests.py
@@ -66,10 +66,6 @@
'script_wui', # depends on lua compiling
]
-deadlock_script_list = [
- 'makecat', # bug 69781
-]
-
script_list = (['login'] +
[name[0:-3] for name in os.listdir(scripts_path) # strip '.py'
if name.endswith('.py') and
@@ -173,10 +169,6 @@
# cause the loader to fallback to its own
# discover() ordering of unit tests.
- if deadlock_script_list:
- print('Skipping deadlock scripts:\n %s'
- % ', '.join(deadlock_script_list))
-
if unrunnable_script_list:
print('Skipping execution of unrunnable scripts:\n %r'
% unrunnable_script_list)
@@ -189,14 +181,12 @@
tests = (['test__login_help'] +
['test_' + name + '_help'
for name in sorted(script_list)
- if name != 'login' and
- name not in deadlock_script_list] +
+ if name != 'login'] +
['test__login_simulate'])
tests += ['test_' + name + '_simulate'
for name in sorted(script_list)
if name != 'login' and
- name not in deadlock_script_list and
name not in failed_dep_script_list and
name not in unrunnable_script_list and
(enable_autorun_tests or name not in auto_run_script_list)]
@@ -343,9 +333,6 @@
# TODO: after bug 68611 and 68664 (and makecat), split -help
# execution to a separate test class which uses site=False.
- if script_name in deadlock_script_list:
- dct[test_name].__test__ = False
-
if script_name == 'login':
test_name = 'test__' + script_name + '_simulate'
else:
@@ -372,7 +359,7 @@
dct[test_name].__name__ = test_name
# Disable test by default in nosetests
- if script_name in unrunnable_script_list + deadlock_script_list:
+ if script_name in unrunnable_script_list:
dct[test_name].__test__ = False
# TODO: Ideally any script not on the auto_run_script_list
--
To view, visit
https://gerrit.wikimedia.org/r/216407
To unsubscribe, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iae9454cc0c7d4c815b2d9def7a8510d9d38f24e3
Gerrit-PatchSet: 2
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: John Vandenberg <jayvdb(a)gmail.com>
Gerrit-Reviewer: John Vandenberg <jayvdb(a)gmail.com>
Gerrit-Reviewer: Ladsgroup <ladsgroup(a)gmail.com>
Gerrit-Reviewer: Merlijn van Deen <valhallasw(a)arctus.nl>
Gerrit-Reviewer: XZise <CommodoreFabianus(a)gmx.de>
Gerrit-Reviewer: jenkins-bot <>