Skip to content

Add check for balance increase on yield increase#180

Open
Kay-Zee wants to merge 1 commit intomainfrom
kan/add-balance-check
Open

Add check for balance increase on yield increase#180
Kay-Zee wants to merge 1 commit intomainfrom
kan/add-balance-check

Conversation

@Kay-Zee
Copy link
Member

@Kay-Zee Kay-Zee commented Feb 19, 2026

Closes:

Description

Add check for balance increase on yield increase

12:51PM INF LOG: "Deploying Morpho contracts..."
12:51PM INF LOG: "deploy PunchswapV3Factory"
12:51PM INF LOG: "PunchswapV3Factory address 8b8725c1281610b6b59203c91c5961dbce4eba73"
12:51PM INF LOG: "deploy NFTDescriptor"
12:51PM INF LOG: "NFTDescriptor address cfaf1b59f7a0b8714084c551ff0d705d29453028"
12:51PM INF LOG: "deploy NFTPositionDescriptor"
12:51PM INF LOG: "NFTPositionDescriptor address a4ff9e51fe723a2cfd9ef6ec4f1dba20e03d4633"
12:51PM INF LOG: "deploy UniswapV2Factory"
12:51PM INF LOG: "UniswapV2Factory address b8ef4a292ead7459b54b3adfab491d86ba39511c"
12:51PM INF LOG: "deploy NonfungiblePositionManager"
12:51PM INF LOG: "NonfungiblePositionManager address bc3ff1b178c3f6e439264bcfe042e8afaa799090"
12:51PM INF LOG: "deploy SwapRouter02"
12:51PM INF LOG: "SwapRouter address 511beb3c362ab7aecd22ec354c5a06d6e83fa576"
12:51PM INF LOG: "deploy QuoterV2"
12:51PM INF LOG: "QuoterV2 address 44c925fbabd5c722526a81a7b584832e6759ab45"
12:51PM INF LOG: "[TEST] YieldVault ID: 0"
12:51PM INF LOG: "[TEST] Initial yield vault balance: 1625.00000000"
12:51PM INF LOG: "[TEST] YieldVault balance before yield price 1.01000000: 1625.00000000"
12:51PM INF LOG: "[TEST] YieldVault balance before yield price 1.01000000 rebalance: 1634.09090909"
12:51PM INF LOG: "[TEST] YieldVault balance after yield before 1.01000000 rebalance: 1634.09090909"
12:51PM INF LOG: "[TEST] flow balance after 1634.09090909"

Test results: "cadence/tests/balance_test.cdc"
- PASS: test_RebalanceYieldVaultScenario2

For contributor use:

  • Targeted PR against master branch
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the standards mentioned here.
  • Updated relevant documentation
  • Re-reviewed Files changed in the Github PR explorer
  • Added appropriate labels

@Kay-Zee Kay-Zee requested a review from a team as a code owner February 19, 2026 20:52
Copy link
Collaborator

@nialexsan nialexsan left a comment

Choose a reason for hiding this comment

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

that's odd that the balance matches even without rebalancing

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

Comments