See #23119
test: Use asset_equal utils where possible in feature_nulldummy #23135
pull brunoerg wants to merge 1 commits into bitcoin:master from brunoerg:2021-09-assert_equal changing 1 files +1 −1-
brunoerg commented at 12:19 PM on September 29, 2021: contributor
-
test: Use asset_equal utils in feature_nulldummy 85238cd9c5
- brunoerg cross-referenced this on Sep 29, 2021 from issue tests: Use test framework utils where possible by vincenzopalazzo
- DrahtBot added the label Tests on Sep 29, 2021
-
vincenzopalazzo commented at 10:11 AM on October 5, 2021: none
NIT
I like to go next with the following idea #23127 (comment), put the work in different PR could be confusion on what it is done and whatnot.
-
brunoerg commented at 10:55 AM on October 5, 2021: contributor
NIT
I like to go next with the following idea #23127 (comment), put the work in different PR could be confusion on what it is done and whatnot.
I totally agree! Do you want me to close this PR?
-
vincenzopalazzo commented at 10:57 AM on October 5, 2021: none
I included this change in the #23127.
Sorry for the confusion, it is my fault.
-
brunoerg commented at 12:12 PM on October 5, 2021: contributor
No problem, going to close it and review/test that PR! Thanks!
- brunoerg closed this on Oct 5, 2021
- vincenzopalazzo cross-referenced this on Oct 8, 2021 from issue test: replace bare asserts with assertion helpers (assert_equal() etc.) by theStack
- HelpCombot cross-referenced this on Nov 24, 2021 from issue . by HelpCombot
- bitcoin locked this on Oct 30, 2022
