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

mempool: Accept test mungers for create tickets. #2721

Merged

Conversation

davecgh
Copy link
Member

@davecgh davecgh commented Aug 26, 2021

This modifies CreateTicketPurchase to accept a spendable output (instead of a transaction) as well as munge functions similar to how CreateSignedTx works and then introduces a new method named CreateTicketPurchaseFromTx which keeps the existing signature and is defined in terms of the aforementioned.

This allows the test code to more easily mutate the transactions as desired prior to them being signed which provides greater flexibility.

@davecgh davecgh added this to the 1.7.0 milestone Aug 26, 2021
@davecgh davecgh force-pushed the mempool_create_ticket_purchase_accept_mungers branch from a9e90db to 381e9e7 Compare August 27, 2021 19:41
This modifies CreateTicketPurchase to accept a spendable output (instead
of a transaction) as well as munge functions similar to how
CreateSignedTx works and then introduces a new method named
CreateTicketPurchaseFromTx which keeps the existing signature and is
defined in terms of the aforementioned.

This allows the test code to more easily mutate the transactions as
desired prior to them being signed which provides greater flexibility.
@davecgh davecgh force-pushed the mempool_create_ticket_purchase_accept_mungers branch from 381e9e7 to 2086029 Compare September 1, 2021 10:56
@davecgh davecgh merged commit 2086029 into decred:master Sep 1, 2021
@davecgh davecgh deleted the mempool_create_ticket_purchase_accept_mungers branch September 1, 2021 11:01
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.

3 participants