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

Typo in test-URI #351

Closed
wants to merge 1 commit into from
Closed

Typo in test-URI #351

wants to merge 1 commit into from

Conversation

crowning-
Copy link

PR so it won't get forgotten.

There are still issues with the test-data for alerts and mining, but I'll have a look at that once v0.12.0.x is out.

@UdjinM6
Copy link

UdjinM6 commented Jun 4, 2015

Hmm... it's not a typo but a real use case test imo
https://github.com/dashpay/dash/blob/v0.12.0.x/src/qt/guiutil.cpp#L186-L198

@crowning-
Copy link
Author

Good point, missed that call :-/
Will check later why the test fails.
Is there any docu on how to create valid test data?

@UdjinM6
Copy link

UdjinM6 commented Jun 4, 2015

Ah, it's ok :)

Haven't seen any doc about test data, I guess it's either considered as "trivial" by bitcoin devs or they just don't care 😠

@crowning-
Copy link
Author

Found out how to build the test data file for the alerting tests (alertTests.raw).

The good news: it's easy
The bad news: each dataset needs to be signed with the secret key of the person who can sign and send alerts (in Bitcoin's case, Gavin), so I can't do it.

Only @evan82 can do this :-)

@UdjinM6
Copy link

UdjinM6 commented Jun 4, 2015

Isn't it a bit off topic? ;)
And btw, I guess we can close this PR, right?

@crowning- crowning- closed this Jun 4, 2015
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.

None yet

2 participants