diff options
| author | Sebastian Falbesoner <sebastian.falbesoner@gmail.com> | 2023-10-12 17:24:16 +0200 |
|---|---|---|
| committer | Sebastian Falbesoner <sebastian.falbesoner@gmail.com> | 2023-10-13 12:42:49 +0200 |
| commit | ac4caf3366a85617641394a97aa9f029550d77d4 (patch) | |
| tree | 9547ae5696f45d74f056dcc96bd5ef8967a85fc5 /doc | |
| parent | 73dfa6da0801e3116e7e84cd5e089b98a9f70212 (diff) | |
| download | bitcoin-ac4caf3366a85617641394a97aa9f029550d77d4.tar.xz | |
test: fix `assert_debug_log` call-site bugs, add type checks
Two recently added tests (PR #28625 / commit 2e31250027ac580a7a72221fe2ff505b30836175
and PR #28634 / commit 3bb51c29df596aab2c1fde184667cee435597715)
introduced a bug by wrongly using the `assert_debug_log` helper.
Instead of passing the expected debug string in a list as expected, it
was passed as bare string, which is then interpretered as a list of
characters, very likely leading the debug log assertion pass even if the
intended message is not appearing.
In order to avoid bugs like this in the future, enforce that the
`{un}expected_msgs` parameters are lists.
Diffstat (limited to 'doc')
0 files changed, 0 insertions, 0 deletions