# HG changeset patch
# User Kyle Lippincott <spectral@google.com>
# Date 1547177107 28800
# Node ID 4f0ae5c64c1b7c3261e2633c0e009c4412bcb3df
# Parent  172296c6db914e08a38442946b4bc2daa50d3589
tests: make HGCATAPULTSERVERPIPE imply HGTESTCATAPULTSERVERPIPE

I had attempted to do this before, but missed this case. This makes it so that
one can do the following to get catapult traces that include both the .t test
name (and non-hg commands run by that .t test) *and* the hg-internal tracing, in
one trace:

    HGCATAPULTSERVERPIPE=/tmp/catapult.pipe run-tests.py <args>

Without this change, we need to specify both `HG{,TEST}CATAPULTSERVERPIPE`; if
we specify just the TEST one, we only get the .t tests (no hg-internals), which
is working as intended. If we specify the non-TEST one, we only get the
hg-internals (not the rest of the .t test), which was not intended.

If you want to restore the previous behavior (just hg internals, not the stuff
from the .t tests), run like:

    HGTESTCATAPULTSERVERPIPE=/dev/null \
      HGCATAPULTSERVERPIPE=/tmp/catapult.pipe \
      run-tests.py <args>

Differential Revision: https://phab.mercurial-scm.org/D5569

diff -r 172296c6db91 -r 4f0ae5c64c1b tests/run-tests.py
--- a/tests/run-tests.py	Thu Jan 10 18:31:46 2019 +0530
+++ b/tests/run-tests.py	Thu Jan 10 19:25:07 2019 -0800
@@ -1409,7 +1409,8 @@
         session = str(uuid.uuid4())
         if PYTHON3:
             session = session.encode('ascii')
-        hgcatapult = os.getenv('HGTESTCATAPULTSERVERPIPE')
+        hgcatapult = os.getenv('HGTESTCATAPULTSERVERPIPE') or \
+            os.getenv('HGCATAPULTSERVERPIPE')
         def toggletrace(cmd=None):
             if not hgcatapult or hgcatapult == os.devnull:
                 return