Xqt has submitted this change. ( https://gerrit.wikimedia.org/r/c/pywikibot/core/+/609583 )

Change subject: [tests] replace os.system usage in ui_tests.py ......................................................................

[tests] replace os.system usage in ui_tests.py

As suggested replace os.system by subprocess.run; see https://docs.python.org/3.5/library/subprocess.html#replacing-os-system

Change-Id: Idb2f48423e0bd664b1d0618141950f186a5925cd --- M tests/ui_tests.py 1 file changed, 4 insertions(+), 2 deletions(-)

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

diff --git a/tests/ui_tests.py b/tests/ui_tests.py index 6337eaa..3920567 100644 --- a/tests/ui_tests.py +++ b/tests/ui_tests.py @@ -129,7 +129,7 @@ setattr(pywikibot.ui, key, value)

def cls(self): - os.system('cls') + subprocess.run('cls', shell=True)

class pywikibotManager(BaseManager):

@@ -683,7 +683,9 @@

def testOutputUnicodeText_no_transliterate(self): self.sendstdin( - "python -c "import os, pywikibot; os.system('cls'); " + 'python -c "' + 'import subprocess, pywikibot; ' + "subprocess.run('cls', shell=True); " "pywikibot.output('\n'.join(pywikibot.handleArgs()))" " 'Alpha Bετα Гамма دلتا\n') lines = []