Skip to content

Fixed rpc tests#405

Merged
marcus290 merged 24 commits into
navcoin:masterfrom
djksa:fixed-rpc-tests
Feb 3, 2019
Merged

Fixed rpc tests#405
marcus290 merged 24 commits into
navcoin:masterfrom
djksa:fixed-rpc-tests

Conversation

@djksa
Copy link
Copy Markdown
Contributor

@djksa djksa commented Jan 29, 2019

Fixed certain rpc-tests from Issue #358.

This pull-request contains a branch based off of the branch update-rpc-tests from my original pull request here. We decided that it would be safer and easier to cherry-pick the working rpc-tests from the update-rpc-tests branch and to manually copy them over to the fixed-rpc-tests branch rather than attempt to resolve the old pull-request.

Copy link
Copy Markdown
Contributor

@marcus290 marcus290 left a comment

Choose a reason for hiding this comment

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

I've tested locally and all the revised tests pass

@marcus290
Copy link
Copy Markdown
Contributor

marcus290 commented Jan 29, 2019

To summarise, all uncommented tests in rpc-tests.py are newly implemented into the test suite.

It seems that using initialize_chain() instead of initialize_chain_clean() in util.py was causing TimeoutErrors in Travis for some tests, which is addressed in 5cb057e.

@matt-auckland matt-auckland added this to the 4.5.2 milestone Jan 31, 2019
@marcus290 marcus290 merged commit ed3f590 into navcoin:master Feb 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants