diff options
author | MarcoFalke <falke.marco@gmail.com> | 2017-09-29 20:23:52 +0200 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2017-09-29 20:23:59 +0200 |
commit | ff4cd6075b12fb32b9a906deea3ed033e3f9560a (patch) | |
tree | f2eb79a88a5ef8f9ba87a8f31094551e1cd368b0 /src/validation.cpp | |
parent | 9c3c9cdae3e20b5bdea91a0631edac5116bbc89f (diff) | |
parent | f97ab35fa9687fd5c110ad6cca5be5b4a5c2142d (diff) | |
download | bitcoin-ff4cd6075b12fb32b9a906deea3ed033e3f9560a.tar.xz |
Merge #11319: [qa] Fix error introduced into p2p-segwit.py, and prevent future similar errors
f97ab35fa qa: Fix bug introduced in p2p-segwit.py (Suhas Daftuar)
a7820422e qa: Treat mininode p2p exceptions as fatal (Suhas Daftuar)
Pull request description:
#11121 inadvertently broke the constructor for the `TestNode()` object in `p2p-segwit.py`, silently breaking at least one of the tests.
Although the python code was raising exceptions due to a `TestNode()` object not existing (or having the right type), mininode was masking these from anyone running the test through the test_runner (like travis), because it catches all exceptions during message delivery and just prints a log message and continues. Such "graceful" handling of errors is almost certainly something we don't want in our test suite, so the first commit here attempts to prevent that type of failure from ever being masked.
The second commit fixes the particular bug in `p2p-segwit.py`.
Tree-SHA512: b6646e3cb1e05c35c28e8899c44104bf2e2d0384643ca87042ab7f6ec0960d89f5bf25a7b95bab6e32d401c20a6018226160500f6ddceb923e81ffb04adb4f2f
Diffstat (limited to 'src/validation.cpp')
0 files changed, 0 insertions, 0 deletions