-
Notifications
You must be signed in to change notification settings - Fork 213
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
[ADP-3344] Implement rollBackward
for the Deposit Wallet
#4776
[ADP-3344] Implement rollBackward
for the Deposit Wallet
#4776
Conversation
d699139
to
bbc4d1c
Compare
344d431
to
f28d652
Compare
4aff6d9
to
0990c61
Compare
6a7061d
to
d9d9799
Compare
testGenesis :: Read.GenesisData | ||
testGenesis = undefined | ||
|
||
payFromFaucet :: [(Write.Address, Write.Value)] -> Read.Tx |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about other types of txs, like spending from our addresses ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree, a test that rolling back a spent input works is desirable. I have added a mock function spendOneTxOut
and used it.
This is as far as I would like to take testing this feature for now. I'll add documentation for the properties that we would expect on a rollback to the Agda code.
3a3ed5e
to
3d903f0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙇
3d903f0
to
ce1ec9c
Compare
ce1ec9c
to
e2cdefd
Compare
This pull request implements the `rollBackward` function for the Deposit Wallet. * We use the `RollbackWindow` data type to decide whether we need to roll back all the way back to genesis. * The `Cardano.Wallet.Deposit.Pure.Submissions` module remains a mock implementation for now. ### Issue Number ADP-3344
This pull request implements the
rollBackward
function for the Deposit Wallet.RollbackWindow
data type to decide whether we need to roll back all the way back to genesis.Cardano.Wallet.Deposit.Pure.Submissions
module remains a mock implementation for now.Issue Number
ADP-3344