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

Add a few revert tests #23

Merged
merged 9 commits into from
Apr 15, 2024
Merged

Add a few revert tests #23

merged 9 commits into from
Apr 15, 2024

Conversation

chris124567
Copy link
Member

  • Handle balance updates similar how to walletd does in AddSiacoinElements/RemoveSiacoinElements by calling ForEachSiacoinElement, separating outputs into slices of spent and new outputs, then processing from there. This fixed a few issues that occurred with reverts and ephemeral outputs.
  • Fix bug which caused the genesis block to not be applied to the Explorer.
  • Fix bug which caused foreign key constraints to not be enforced.
  • Add tests similar to the apply block tests (TestRevertSendTransactions, TestRevertBalance) except that they revert some of the blocks and check that the state is correct afterwards.

@chris124567 chris124567 requested a review from n8maninger April 15, 2024 16:20
@n8maninger n8maninger merged commit abe0200 into master Apr 15, 2024
6 of 7 checks passed
@n8maninger n8maninger deleted the add-revert-tests branch April 15, 2024 20:23
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.

2 participants