Skip to content

test(amm): keep required liquidity burn check priced#5617

Merged
grandizzy merged 1 commit into
mainfrom
centaur/1781158245-30486-27249
Jun 12, 2026
Merged

test(amm): keep required liquidity burn check priced#5617
grandizzy merged 1 commit into
mainfrom
centaur/1781158245-30486-27249

Conversation

@decofe

@decofe decofe commented Jun 11, 2026

Copy link
Copy Markdown
Member

Summary

  • Set an explicit priority fee on test_cant_burn_required_liquidity burn tx.
  • Keeps the fee-swap reservation large enough under TIP-1067 dynamic base fees so the test continues to assert the intended invariant.

Why

The mutation coverage job failed in coverage with tip_fee_amm::test_cant_burn_required_liquidity: the burn receipt succeeded when the test expected it to fail. Under dynamic base fees, max_fee_per_gas alone can leave the tx with a lower effective gas price than the test assumed.

Verification

  • cargo test -p tempo-node --test it tip_fee_amm::test_cant_burn_required_liquidity -- --nocapture
  • cargo +nightly fmt --check

@grandizzy grandizzy marked this pull request as ready for review June 11, 2026 06:40
@grandizzy grandizzy requested a review from 0xrusowsky June 11, 2026 06:41
@grandizzy grandizzy added this pull request to the merge queue Jun 12, 2026
Merged via the queue into main with commit 45b586e Jun 12, 2026
38 checks passed
@grandizzy grandizzy deleted the centaur/1781158245-30486-27249 branch June 12, 2026 07:00
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.

3 participants