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: faa3e684118bffa7a98cf76eeeb59243219df900
This commit is contained in:
MarcoFalke 2025-07-18 07:36:32 +02:00 committed by fanquake
parent a828e64b7d
commit 41fa1e0ee5
No known key found for this signature in database
GPG Key ID: 2EEB9F5CC09526C1

View File

@ -134,10 +134,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")