/PIVX forked from PIVX-Project/PIVX

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

[WIP] [Test] Backport 5852 and 5883 #8

Closed
wants to merge 2 commits into
base: master
from

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@Warrows @sipa @laanwj
@Warrows
Copy link
Owner

Warrows commented Feb 3, 2019

No description provided.

sipa and others added some commits Mar 3, 2015

@sipa @Warrows
Reinitialize state in between individual unit tests. 
This changes the TestingSetup fixture to be per-unit-test rather than global.
Most tests don't need it, so it's only invoked in a few.
Verified
This commit was signed with a verified signature.
@Warrows Warrows Warrows
GPG key ID: 7E07C315C3BD9E93 Learn about signing commits
6ed9025
@laanwj @Warrows
tests: add a BasicTestingSetup and apply to all tests 
Make sure that chainparams and logging is properly initialized. Doing
this for every test may be overkill, but this initialization is so
simple that that does not matter.

This should fix the travis issues.
Verified
This commit was signed with a verified signature.
@Warrows Warrows Warrows
GPG key ID: 7E07C315C3BD9E93 Learn about signing commits
Loading status checks…
a72634f

@Warrows Warrows force-pushed the 2019_bitcoin-bp_5883 branch from 8176bb4 to a72634f Feb 3, 2019

@Warrows Warrows closed this Feb 3, 2019

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