test: Log KeyboardInterrupt as exception

log.exception is more verbose and useful to debug timeouts.

Also, log stderr for CalledProcessError to make debugging easier.

Github-Pull: #33001
Rebased-From: faa3e68411
This commit is contained in:
MarcoFalke
2025-07-18 07:36:32 +02:00
committed by fanquake
parent 79e1a3c9c6
commit 5e327e6703

View File

@@ -138,10 +138,7 @@ class BitcoinTestFramework(metaclass=BitcoinTestMetaClass):
self.log.warning("Test Skipped: %s" % e.message)
self.success = TestStatus.SKIPPED
except subprocess.CalledProcessError as e:
self.log.exception("Called Process failed with '{}'".format(e.output))
self.success = TestStatus.FAILED
except KeyboardInterrupt:
self.log.warning("Exiting after keyboard interrupt")
self.log.exception(f"Called Process failed with stdout='{e.stdout}'; stderr='{e.stderr}';")
self.success = TestStatus.FAILED
except BaseException:
self.log.exception("Unexpected exception")