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

Fix rpc-tests #720

Closed
wants to merge 1 commit into from
Closed

Fix rpc-tests #720

wants to merge 1 commit into from

Conversation

UdjinM6
Copy link

@UdjinM6 UdjinM6 commented Mar 6, 2016

Relies on PRs #715-#719
Requires https://github.com/dashpay/dash_hash to be installed (we need to fix gitian to do this too, not part of this PR).

@schinzelh
Copy link

Merged all of them, many thanks! Let's see if Travis agrees :)

@UdjinM6
Copy link
Author

UdjinM6 commented Mar 6, 2016

thanks! travis most likely will complain though because there is no setup step for dash_hash yet... :/

@schinzelh
Copy link

Yep, dash_hash not found :-)

@UdjinM6
Copy link
Author

UdjinM6 commented Mar 6, 2016

I actually have a fix for this UdjinM6@9892813 ;)
but now test fails on proxy_test.py https://travis-ci.org/UdjinM6/dash/jobs/114089206#L6234 and I have no idea why because it's identical to bitcoin's... :/
and btw make check also fails so there is still a lot to fix...

@schinzelh
Copy link

It fails at https://github.com/dashpay/dash/blob/v0.12.1.x/qa/rpc-tests/proxy_test.py#L59 - guessing without looking deeper: something with IPv6 is broken, as conf3 is the only one using IPv6

@UdjinM6 UdjinM6 deleted the fixrpctests branch March 7, 2016 03:03
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.

2 participants