diff options
| author | Victor Stinner <victor.stinner@gmail.com> | 2017-04-19 22:57:30 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2017-04-19 22:57:30 (GMT) |
| commit | ec4b17239d899550be4ee6104b61751bb3c70382 (patch) | |
| tree | d60156ee4fab31749e94bbda39b1042ea419cd38 | |
| parent | 7b9619ae249ed637924d1c76687b411061753e5a (diff) | |
| download | cpython-ec4b17239d899550be4ee6104b61751bb3c70382.zip cpython-ec4b17239d899550be4ee6104b61751bb3c70382.tar.gz cpython-ec4b17239d899550be4ee6104b61751bb3c70382.tar.bz2 | |
regrtest: always show before/after of modified env (#1192)
Buildbots don't run tests with -vv and so only log "xxx was modified
by test_xxx" which is not enough to debug such random issue. In many
cases, I'm unable to reproduce the warning and so unable to fix it.
Always logging the value before and value after should help to debug
such warning on buildbots.
| -rw-r--r-- | Lib/test/libregrtest/save_env.py | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/Lib/test/libregrtest/save_env.py b/Lib/test/libregrtest/save_env.py index 96ad3af..8309f26 100644 --- a/Lib/test/libregrtest/save_env.py +++ b/Lib/test/libregrtest/save_env.py @@ -279,7 +279,6 @@ class saved_test_environment: if not self.quiet and not self.pgo: print(f"Warning -- {name} was modified by {self.testname}", file=sys.stderr, flush=True) - if self.verbose > 1: - print(f" Before: {original}\n After: {current} ", - file=sys.stderr, flush=True) + print(f" Before: {original}\n After: {current} ", + file=sys.stderr, flush=True) return False |