-
Notifications
You must be signed in to change notification settings - Fork 24
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
feat: add ALM vault to LpSugar #78
Conversation
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.
Left some minor notes. Would be great to have a test with an ALM deposit if possible.
0838877
to
0b2766d
Compare
tests/test_lp_sugar.py
Outdated
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.
Leaving this here as a suggestion (also fine if it's not implemented). This has already been tested in the FE so it's only for redundancy.
Instead of having the
assert pos.id is not None
assert pos.lp is not None
assert pos.alm is not None
we would
assert pos.id == <expected_id>
assert pos.lp == <expected_lp>
assert pos.alm == <expected_alm>
7ee54d9
to
d112ff2
Compare
No description provided.