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

Feat/use aave client lib in tests #1

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

sakulstra
Copy link

Hello this is a little draft using aave client lib inside one of the tests.
The test succeeded so I guess this is a good start 🎉

Creating this pr so i can ask questions in code.


In theory the client should be capable of dositing eth directly (in fact at the moment aave app doesn't offer a way to directly deposit weth at all), but for some reason this didn't work - didn't yet figure out why.

@ColonelJ
Copy link
Contributor

ColonelJ commented Apr 6, 2021

Could you maybe move your changed script to a separate script file, and restore the original script to how it was? And take the token to test as a command line argument.

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