Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates to make the wallet.py and stakeimmaturebalance.py test more reliable #624

Merged
merged 2 commits into from
Nov 21, 2019

Conversation

mxaddict
Copy link
Contributor

@mxaddict mxaddict commented Nov 12, 2019

I saw these test was failing intermittently, I think this change will fix that.

@aguycalled
Copy link
Member

utACK

@aguycalled
Copy link
Member

wallet.py test still fails

@mxaddict
Copy link
Contributor Author

@aguycalled I've updated the commit

@mxaddict mxaddict changed the title Updates to make the wallet.py test more reliable Updates to make the wallet.py and stakeimmaturebalance.py test more reliable Nov 13, 2019
@mxaddict
Copy link
Contributor Author

I updated the stakeimmaturebalance.py test as well

@aguycalled
Copy link
Member

restarting travis build to check it still passes

@mxaddict
Copy link
Contributor Author

The build failed, but due to memonic test, I've restarted the failed test.

@navbuilder
Copy link

A new build of 62f6bb9 has completed succesfully!
Binaries available at https://build.nav.community/binaries/update-wallet-rpc-test

@mxaddict
Copy link
Contributor Author

@proletesseract @chasingkirkjufell could you spare a moment to review the changes to the py tests on this PR?

Copy link
Contributor

@chasingkirkjufell chasingkirkjufell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested and code makes sense

@aguycalled aguycalled merged commit 7058550 into navcoin:master Nov 21, 2019
aguycalled pushed a commit to aguycalled/navcoin-core that referenced this pull request Nov 21, 2019
…eliable (navcoin#624)

* Updates to make the wallet.py test more reliable

* Updated qa/rpc-tests/stakeimmaturebalance.py to be more reliable
aguycalled pushed a commit to aguycalled/navcoin-core that referenced this pull request Nov 21, 2019
…eliable (navcoin#624)

* Updates to make the wallet.py test more reliable

* Updated qa/rpc-tests/stakeimmaturebalance.py to be more reliable
aguycalled pushed a commit to aguycalled/navcoin-core that referenced this pull request Dec 5, 2019
…eliable (navcoin#624)

* Updates to make the wallet.py test more reliable

* Updated qa/rpc-tests/stakeimmaturebalance.py to be more reliable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants