[qa] wallet: Check if maintenance changes the balance #7229

Merged
merged 3 commits into from Jan 7, 2016

Conversation

Projects
None yet
3 participants
@MarcoFalke
Member

MarcoFalke commented Dec 18, 2015

Takes about 13 seconds on my system so one could argue to put this into extended.

MarcoFalke added some commits Dec 15, 2015

@MarcoFalke

This comment has been minimized.

Show comment
Hide comment
@MarcoFalke

MarcoFalke Dec 18, 2015

Member

Looks like there is a bug with lock order: #7231

Member

MarcoFalke commented Dec 18, 2015

Looks like there is a bug with lock order: #7231

@laanwj

This comment has been minimized.

Show comment
Hide comment
@laanwj

laanwj Jan 7, 2016

Member

utACK

Member

laanwj commented Jan 7, 2016

utACK

@laanwj laanwj merged commit fa33d97 into bitcoin:master Jan 7, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

laanwj added a commit that referenced this pull request Jan 7, 2016

Merge pull request #7229
fa33d97 [walletdb] Add missing LOCK() in Recover() for dummyWallet (MarcoFalke)
fa14d99 [qa] check if wallet or blochchain maintenance changes the balance (MarcoFalke)
fa0765d [qa] Cleanup wallet.py test (MarcoFalke)

laanwj added a commit that referenced this pull request Jan 7, 2016

[qa] wallet: Check if maintenance changes the balance
- [qa] Cleanup wallet.py test
- [qa] check if wallet or blochchain maintenance changes the balance
- [walletdb] Add missing LOCK() in Recover() for dummyWallet

Github-Pull: #7229
Rebased-From: fa0765d fa14d99 fa33d97

@MarcoFalke MarcoFalke deleted the MarcoFalke:MarcoFalke-2015-rpcWalletMaintenance branch Jan 7, 2016

@dagurval dagurval referenced this pull request in bitcoinxt/bitcoinxt Jan 17, 2018

Merged

[qa] wallet: Check if maintenance changes the balance #301

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment