Skip to content

fix(test): correct alice balance1 token in mint transfer burn test#521

Open
anxbt wants to merge 1 commit into
Uniswap:mainfrom
anxbt:fix/mint-transfer-burn-balance1
Open

fix(test): correct alice balance1 token in mint transfer burn test#521
anxbt wants to merge 1 commit into
Uniswap:mainfrom
anxbt:fix/mint-transfer-burn-balance1

Conversation

@anxbt

@anxbt anxbt commented Apr 6, 2026

Copy link
Copy Markdown

Related Issue

N/A

Description of changes

  • Fixed token balance tracking bug in mint-transfer-burn test.
  • In test_mintTransferBurn, changed balance1BeforeAlice to read from currency1.balanceOf(alice) instead of currency0.balanceOf(alice).
  • This ensures the assertion for token1 payout after burn compares against the correct pre-burn balance.

Copilot AI review requested due to automatic review settings April 6, 2026 05:27

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Fixes an incorrect pre-burn balance read in test_mintTransferBurn, ensuring the token1 payout assertion compares against Alice’s actual token1 balance before burning the position.

Changes:

  • Corrected balance1BeforeAlice to use currency1.balanceOf(alice) in the mint-transfer-burn test.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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