[Tests] added unit tests for the RPC Wallet #3834

pull alonmuroch wants to merge 5 commits into bitcoin:master from alonmuroch:wallet_rpc_tests changing 16 files +6185 −3
  1. alonmuroch commented at 12:34 PM on March 10, 2014: contributor

    Added the following test : 1)setaccount 2)getrawchangeaddress 3)getnewaddress 4)getaccountaddress 5)getaccount 6)signmessage 7)getaddressesbyaccount 8)getreceivedbyaddress

  2. commit before pull from original bitcoin repo 920f5b068f
  3. Merge branch 'master' of https://github.com/bitcoin/bitcoin
    Periodic merge with bitcoin project
    a8e7915aac
  4. added sign message rpc test 7013df5ac7
  5. lots of tests added, next step, add init balance 919139f104
  6. jgarzik commented at 12:42 PM on March 10, 2014: contributor

    Test contributions are very useful, but it looks like quite a bit of junk was added along with the tests:

    • several private eclipse files were added
    • unrelated whitespace changes were made to some C++ modules

    This will need to be cleaned up, and those extra changes removed. Please "rebase" with the updated changes (i.e. avoid adding a junk file, then adding another commit that removes the junk file).

  7. Diapolo commented at 12:48 PM on March 10, 2014: none

    Looks like ALL changes eccept the except ones in src/test/rpc_wallet_tests.cpp should be disccarded from this pull.

  8. Merge commit 'b76733d8e89487d0f03fb5ee09432bd4161acd87' into wallet_rpc_tests 65423939e2
  9. alonmuroch commented at 1:12 PM on March 10, 2014: contributor

    I will organize the files and create a new one (changed branch)

  10. BitcoinPullTester commented at 1:15 PM on March 10, 2014: none

    Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/919139f1048e9c9fd081852dc45b154925d583c4 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.

  11. alonmuroch closed this on Mar 10, 2014

  12. alonmuroch deleted the branch on Mar 10, 2014
  13. jgarzik commented at 1:44 PM on March 10, 2014: contributor

    Note, you do not need to close the pull request or delete the branch.

    You can, locally, create a new branch, and then "git push --force" to overwrite the remote github branch. This will automatically revise the pull request to include only the new commits in the overwritten branch.

  14. 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 03:16 UTC

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