test: fix incorrect named args in txpackage tests #24724

pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:txpackage_named_args changing 1 files +54 −54
  1. fanquake commented at 3:35 pm on March 31, 2022: member

    Final non-scripted-diff commit split from #24661.

    Could be tested with: ./autogen.sh && ./configure CC=clang-12 CXX=clang++-12 && make clean && bear make -j9 && ( cd ./src/ && run-clang-tidy-12 -j9 ).

    Motivation:

    Incorrect named args are source of bugs, like #22979.

    To allow them being checked by clang-tidy, use a format it can understand.

  2. test: fix incorrect named args in txpackage tests bf77fea3c1
  3. fanquake added the label Refactoring on Mar 31, 2022
  4. fanquake requested review from MarcoFalke on Mar 31, 2022
  5. fanquake requested review from glozow on Mar 31, 2022
  6. ajtowns commented at 5:53 am on April 1, 2022: member
    ACK bf77fea3c154f3df6f05fcdcc9b89d560470c940
  7. MarcoFalke merged this on Apr 1, 2022
  8. MarcoFalke closed this on Apr 1, 2022

  9. fanquake deleted the branch on Apr 1, 2022
  10. fanquake removed review request from MarcoFalke on Apr 1, 2022
  11. fanquake removed review request from glozow on Apr 1, 2022
  12. sidhujag referenced this in commit 85407b771f on Apr 3, 2022
  13. DrahtBot locked this on Apr 1, 2023

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-24 09:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me