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

Fixing reserve pool functions #124

Merged
merged 4 commits into from
Feb 2, 2025

Conversation

casweeney
Copy link
Contributor

Fixing reserve pool functions - closes: #123

  • Fixed deposit_to_pool() function, removed duplicate struct fields in the PoolData struct.
  • FIxed withdraw_from_pool() and borrow_from_pool() functions, by updating the usage of struct fields in the functions.
  • Added test suites test_deposit_to_pool_should_panic_for_inactive_pool(), test_deposit_to_pool(), test_withdraw_from_pool_should_panic_for_inactive_pool(), test_withdraw_from_pool(), test_borrow_from_pool() and test_borrow_from_pool_should_panic_for_insufficient_pool_liquidity(), test_should_panic_for_trying_to_withdraw_locked_tokens() to test the functions deposit_to_pool(), withdraw_from_pool() and borrow_from_pool()

Copy link

vercel bot commented Feb 2, 2025

@casweeney is attempting to deploy a commit to the peerprotocol Team on Vercel.

A member of the Team first needs to authorize it.

@casweeney casweeney marked this pull request as ready for review February 2, 2025 23:00
Copy link
Collaborator

@Dprof-in-tech Dprof-in-tech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all done. Thank you for contributing.

@Dprof-in-tech Dprof-in-tech merged commit fbd536c into Peerprotocol:main Feb 2, 2025
1 check failed
@casweeney casweeney deleted the fixing-reserve-pool branch February 2, 2025 23:24
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.

Cairo: fixing reserve pool functions
2 participants