-
Notifications
You must be signed in to change notification settings - Fork 381
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
Simulator coverage for get_account_{transfers,balances} #2004
Conversation
8dfc5ea
to
f1a1ecf
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.
Realized that I don't understand one big picture aspect of this: in validate_account_filter_result_count
, we compare the number of transfers the account has at a given moment with the amount of results. But why than works? More transfers could get created in the meantime, no?
removed vopr label until this is rebased to 0.13 :) |
4684efb
to
639baba
Compare
Implemented assertions based on the expected number of results in variations of filters such as `debits | credits` and timestamp range. It's not able to fine-check intermediate states such as the expected `account_balance` between transfers, although the path covered is good enough to check the assertions already in place along the code.
639baba
to
6ef6d2c
Compare
Implemented assertions based on the expected number of results in variations of filters such as
debits | credits
and timestamp range.It's not able to fine-check intermediate states such as the expected
account_balance
between transfers, although the path covered is good enough to check the assertions already in place along the code.