[tor-commits] [stem/master] Tor stacktraces missing if the crash occures on the last test

atagar at torproject.org atagar at torproject.org
Wed Oct 31 20:23:46 UTC 2018


commit c15c8a5af1f7a9a85a092b932f380b4823ebe86d
Author: Damian Johnson <atagar at torproject.org>
Date:   Wed Oct 31 13:01:49 2018 -0700

    Tor stacktraces missing if the crash occures on the last test
    
    Dumb me. We should check if the tor process is running *after* the test, not
    before. Otherwise tor segfaults that occure during our last test suite go
    unreported (such as with tor commit 075acf905).
---
 run_tests.py | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/run_tests.py b/run_tests.py
index bd0839c9..3afc5c9a 100755
--- a/run_tests.py
+++ b/run_tests.py
@@ -236,11 +236,11 @@ def main():
         println('Running tests...\n', STATUS)
 
         for test_class in get_integ_tests(args.specific_test):
-          if integ_runner.assert_tor_is_running():
-            run_result = _run_test(args, test_class, output_filters)
-            test.output.print_logging(logging_buffer)
-            skipped_tests += len(getattr(run_result, 'skipped', []))
-          else:
+          run_result = _run_test(args, test_class, output_filters)
+          test.output.print_logging(logging_buffer)
+          skipped_tests += len(getattr(run_result, 'skipped', []))
+
+          if not integ_runner.assert_tor_is_running():
             # our tor process died
 
             error_tracker.register_error()



More information about the tor-commits mailing list