( )⚙ D6577 py3: make catapult usable from the test runner in py3

This is an archive of the discontinued Mercurial Phabricator instance.

Differential D6577

py3: make catapult usable from the test runner in py3
ClosedPublic

Authored by rdamazio on Jun 25 2019, 10:37 PM.

Details

Reviewers
pulkit
Group Reviewers
hg-reviewers
Commits
rHG9913fffd744b: py3: make catapult usable from the test runner in py3
Summary

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

rdamazio created this revision.Jun 25 2019, 10:37 PM
rdamazio updated this revision to Diff 15661.Jun 25 2019, 11:04 PM
rdamazio updated this revision to Diff 15662.
pulkit accepted this revision.Jun 27 2019, 11:42 AM
This revision is now accepted and ready to land.Jun 27 2019, 11:42 AM
rdamazio added a commit: rHG9913fffd744b: py3: make catapult usable from the test runner in py3.Jun 27 2019, 12:19 PM
Closed by commit rHG9913fffd744b: py3: make catapult usable from the test runner in py3 (authored by rdamazio). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mtests/run-tests.py (12 lines)

Diff 15662

tests/run-tests.py

if self._debug: if self._debug:
script.append(b'set -x\n') script.append(b'set -x\n')
if self._hgcommand != b'hg': if self._hgcommand != b'hg':
script.append(b'alias hg="%s"\n' % self._hgcommand) script.append(b'alias hg="%s"\n' % self._hgcommand)
if os.getenv('MSYSTEM'): if os.getenv('MSYSTEM'):
script.append(b'alias pwd="pwd -W"\n') script.append(b'alias pwd="pwd -W"\n')
if hgcatapult and hgcatapult != os.devnull: if hgcatapult and hgcatapult != os.devnull:
if PYTHON3:
hgcatapult = hgcatapult.encode('utf8')
cataname = self.name.encode('utf8')
else:
cataname = self.name
# Kludge: use a while loop to keep the pipe from getting # Kludge: use a while loop to keep the pipe from getting
# closed by our echo commands. The still-running file gets # closed by our echo commands. The still-running file gets
# reaped at the end of the script, which causes the while # reaped at the end of the script, which causes the while
# loop to exit and closes the pipe. Sigh. # loop to exit and closes the pipe. Sigh.
script.append( script.append(
b'rtendtracing() {\n' b'rtendtracing() {\n'
b' echo END %(session)s %(name)s >> %(catapult)s\n' b' echo END %(session)s %(name)s >> %(catapult)s\n'
b' rm -f "$TESTTMP/.still-running"\n' b' rm -f "$TESTTMP/.still-running"\n'
b'}\n' b'}\n'
b'trap "rtendtracing" 0\n' b'trap "rtendtracing" 0\n'
b'touch "$TESTTMP/.still-running"\n' b'touch "$TESTTMP/.still-running"\n'
b'while [ -f "$TESTTMP/.still-running" ]; do sleep 1; done ' b'while [ -f "$TESTTMP/.still-running" ]; do sleep 1; done '
b'> %(catapult)s &\n' b'> %(catapult)s &\n'
b'HGCATAPULTSESSION=%(session)s ; export HGCATAPULTSESSION\n' b'HGCATAPULTSESSION=%(session)s ; export HGCATAPULTSESSION\n'
b'echo START %(session)s %(name)s >> %(catapult)s\n' b'echo START %(session)s %(name)s >> %(catapult)s\n'
% { % {
'name': self.name, b'name': cataname,
'session': session, b'session': session,
'catapult': hgcatapult, b'catapult': hgcatapult,
} }
) )
if self._case: if self._case:
casestr = b'#'.join(self._case) casestr = b'#'.join(self._case)
if isinstance(self._case, str): if isinstance(self._case, str):
quoted = shellquote(casestr) quoted = shellquote(casestr)
else: else: