Added the following test : 1)setaccount 2)getrawchangeaddress 3)getnewaddress 4)getaccountaddress 5)getaccount 6)signmessage 7)getaddressesbyaccount 8)getreceivedbyaddress
[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-
alonmuroch commented at 12:34 PM on March 10, 2014: contributor
-
commit before pull from original bitcoin repo 920f5b068f
-
a8e7915aac
Merge branch 'master' of https://github.com/bitcoin/bitcoin
Periodic merge with bitcoin project
-
added sign message rpc test 7013df5ac7
-
lots of tests added, next step, add init balance 919139f104
-
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).
-
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.
-
Merge commit 'b76733d8e89487d0f03fb5ee09432bd4161acd87' into wallet_rpc_tests 65423939e2
-
alonmuroch commented at 1:12 PM on March 10, 2014: contributor
I will organize the files and create a new one (changed branch)
-
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.
- alonmuroch closed this on Mar 10, 2014
- alonmuroch deleted the branch on Mar 10, 2014
-
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.
- DrahtBot locked this on Sep 8, 2021