diff options
| author | fanquake <fanquake@gmail.com> | 2023-12-13 11:21:35 +0000 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2023-12-13 11:23:59 +0000 |
| commit | 8431a195375bbf4a2dd2d646031b07beace3be80 (patch) | |
| tree | cf113a6d05a9368402c066d6c16bfce4a3651ee7 | |
| parent | 54f6756e52d07c099328bc9a99a75308e804da21 (diff) | |
| parent | fa0534d7e47d44428d3f9dea6d2f6b8e86df22d4 (diff) | |
| download | bitcoin-8431a195375bbf4a2dd2d646031b07beace3be80.tar.xz | |
Merge bitcoin/bitcoin#29068: test: Actually fail when a python unit test fails
fa0534d7e47d44428d3f9dea6d2f6b8e86df22d4 test: Actually fail when a python unit test fails (MarcoFalke)
Pull request description:
Currently python unit test failures are ignored.
Fix this.
ACKs for top commit:
theStack:
ACK fa0534d7e47d44428d3f9dea6d2f6b8e86df22d4
BrandonOdiwuor:
ACK fa0534d7e47d44428d3f9dea6d2f6b8e86df22d4
Tree-SHA512: c136be4c8d861d966f380e04d5d14b711b90c4011101302dae1332496e493207c5c673927586ed35b02b61a0b050bf45053a31e6ff766ec52f1d054caf0985e2
| -rwxr-xr-x | test/functional/test_runner.py | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/test/functional/test_runner.py b/test/functional/test_runner.py index 6016a482f8..bf52ba6d93 100755 --- a/test/functional/test_runner.py +++ b/test/functional/test_runner.py @@ -565,8 +565,7 @@ def run_tests(*, test_list, src_dir, build_dir, tmpdir, jobs=1, enable_coverage= test_framework_tests.addTest(unittest.TestLoader().loadTestsFromName("test_framework.{}".format(module))) result = unittest.TextTestRunner(verbosity=1, failfast=True).run(test_framework_tests) if not result.wasSuccessful(): - logging.debug("Early exiting after failure in TestFramework unit tests") - sys.exit(False) + sys.exit("Early exiting after failure in TestFramework unit tests") flags = ['--cachedir={}'.format(cache_dir)] + args |