test: Extend wallet_dump test to cover comments #18597

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2004-qaWalletDump changing 1 files +41 −6
  1. MarcoFalke commented at 4:57 PM on April 11, 2020: member

    No description provided.

  2. test: Extend wallet_dump test to cover comments 555567ace9
  3. DrahtBot added the label Tests on Apr 11, 2020
  4. ryanofsky cross-referenced this on Apr 13, 2020 from issue wallet: Remove calls to Chain::Lock methods by ryanofsky
  5. ryanofsky approved
  6. ryanofsky commented at 3:25 AM on April 13, 2020: contributor

    Code review ACK 555567ace9baae3c80e118eeca434d5c424a3487. Nice new checks in this test. I confirmed this catches the missing FormatISO8601DateTime call you discovered in #17954 (review)

  7. MarcoFalke merged this on Apr 13, 2020
  8. MarcoFalke closed this on Apr 13, 2020
  9. MarcoFalke deleted the branch on Apr 13, 2020
  10. Fabcien referenced this in commit e8fb01c940 on Jan 15, 2021
  11. bitcoin locked this on Feb 15, 2022
Contributors
MarcoFalkeryanofsky
Labels
Tests
Linked (view graph)
#17954 wallet: Remove calls to Chain::Lock methods
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: 2026-05-20 06:54 UTC