Skip to content

Write Lending Pool Unit Tests#779

Merged
ogazboiz merged 27 commits intoLabsCrypt:mainfrom
sweetesty:lending-pool
Apr 28, 2026
Merged

Write Lending Pool Unit Tests#779
ogazboiz merged 27 commits intoLabsCrypt:mainfrom
sweetesty:lending-pool

Conversation

@sweetesty
Copy link
Copy Markdown
Contributor

📌 Pull Request

🔗 Related Issue
Closes #25

📝 Description

  • Adds comprehensive unit tests for the LendingPool smart contract
  • Necessary to ensure contract correctness since bugs in smart contracts are immutable once deployed

🚀 Changes Made

  • Testing:
    • Added unit tests for deposit functionality
    • Added unit tests for withdraw functionality covering both success and failure cases
    • Added unit tests for liquidity checks
    • All tests written using the Soroban test environment

🧪 Testing & Validation

  • Tested locally
  • No runtime errors
  • All test cases passing
  • Edge cases handled (insufficient liquidity, failed withdrawals, zero deposits)

⚠️ Breaking Changes

  • No breaking changes

📋 Contributor Checklist (MANDATORY)

  • I was assigned to this issue
  • My code follows the project structure and conventions
  • I have tested my changes thoroughly
  • I did not introduce unnecessary dependencies
  • I have linked the issue (Closes Write Lending Pool Unit Tests #25)
  • This PR is ready for review

💡 Notes for Reviewer

  • Key areas to review: deposit test cases, withdraw failure scenarios, liquidity check assertions
  • Known limitations: None

Spagero763 and others added 27 commits February 20, 2026 16:32
@ogazboiz ogazboiz merged commit d7d44ce into LabsCrypt:main Apr 28, 2026
1 of 4 checks passed
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.

Write Lending Pool Unit Tests

3 participants