Test for walletbackup/walletdump RPC calls #3700

pull gavinandresen wants to merge 1 commits into bitcoin:master from gavinandresen:backuptest changing 2 files +314 −2
  1. gavinandresen commented at 2:45 PM on February 18, 2014: contributor

    This is also a test for rescanning for wallet transactions; I wrote it to help test #3694

  2. laanwj commented at 3:13 PM on February 18, 2014: member
    Starting nodes...
    Generating initial blockchain...
    Creating transactions...
    Backing up...
    More transactions...
    error: {"code":-4,"message":"Insufficient funds"}
    Restoring using wallet.dat
    Restoring using dumped wallet
    Tests successful, cleaning up
    

    Is the error: expected?

  3. gavinandresen commented at 3:17 PM on February 18, 2014: contributor

    No, the wallets should have plenty of funds, that is unexpected.

  4. laanwj commented at 3:38 PM on February 18, 2014: member

    Hm just re-tried and cannot reproduce it now. Weird. I'm testing without #3694 BTW.

    Edit: I wonder if it would make sense to store pre-mined block chain data for some qa tests (and share it between them), as the generation of the initial blockchain makes the tests quite slow.

    Edit.2: Ran it 4 times, got the error 2 times out of 4. Seems like a flipper issue. However, the test overall still claims succesful every time.

  5. BitcoinPullTester commented at 3:08 AM on February 19, 2014: none

    Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/170bbcba05618cea1b908fe77d04f4f5583ed6f2 for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.

  6. laanwj added this to the milestone 0.9.0 on Feb 24, 2014
  7. laanwj commented at 3:44 PM on February 24, 2014: member

    Probably time to merge this. @gavinandresen can you squash the 'tweak' commit into its parent?

  8. Test for walletbackup/walletdump RPC calls 41dcf61504
  9. gavinandresen referenced this in commit 05619d7290 on Feb 24, 2014
  10. gavinandresen merged this on Feb 24, 2014
  11. gavinandresen closed this on Feb 24, 2014

  12. gavinandresen deleted the branch on Feb 24, 2014
  13. DrahtBot locked this on Sep 8, 2021

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-04-29 00:16 UTC

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