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

unit tests for transaction undo #8810

Merged
merged 3 commits into from May 11, 2021
Merged

unit tests for transaction undo #8810

merged 3 commits into from May 11, 2021

Conversation

deepthiskumar
Copy link
Member

@deepthiskumar deepthiskumar commented May 5, 2021

-tests undoing on masks on which transactions were applied as well as new masks
-doesn't include snapps

Closes #8777

@deepthiskumar deepthiskumar requested review from a team as code owners May 5, 2021 01:23
@deepthiskumar deepthiskumar added the ci-build-me Add this label to trigger a circle+buildkite build for this branch label May 5, 2021
src/lib/transaction_snark/transaction_snark.ml Outdated Show resolved Hide resolved
src/lib/mina_base/coinbase.ml Show resolved Hide resolved
@aneesharaines aneesharaines linked an issue May 5, 2021 that may be closed by this pull request
@mrmr1993 mrmr1993 merged commit aec417c into compatible May 11, 2021
@mrmr1993 mrmr1993 deleted the fix/undo-tests branch May 11, 2021 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-build-me Add this label to trigger a circle+buildkite build for this branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Writing unit tests for undos
2 participants