This will make the test fail as soon as the bug is fixed, forcing it to be updated. Otherwise, the bug might be fixed (or made worse) accidentally, leaving the test in an inconsistent state.
test: Replace log with assert_equal in wallet_abandonconflict #23331
pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2110-testAssert changing 1 files +1 −1-
MarcoFalke commented at 9:15 AM on October 21, 2021: member
-
fa6c62f34b
test: Replace log with assert_equal in wallet_abandonconflict
This will make the test fail as soon as the bug is fixed, forcing it to be updated. Otherwise, the bug might be fixed (or made worse) accidentally, leaving the test in an inconsistent state.
- fanquake added the label Tests on Oct 21, 2021
- theStack approved
-
theStack commented at 4:50 PM on October 21, 2021: member
Concept and code-review ACK fa6c62f34b50818102ad58f2ffd152189f54d973
- brunoerg approved
-
brunoerg commented at 6:10 PM on October 21, 2021: member
tACK fa6c62f34b50818102ad58f2ffd152189f54d973
- MarcoFalke merged this on Oct 21, 2021
- MarcoFalke closed this on Oct 21, 2021
- MarcoFalke deleted the branch on Oct 21, 2021
- sidhujag referenced this in commit 5c431c93ba on Oct 21, 2021
- PastaPastaPasta referenced this in commit 62ec84d21a on Apr 3, 2022
- DrahtBot locked this on Oct 30, 2022
Contributors
Labels