Skip to content

test: dedupe position balance helpers#203

Open
liobrasil wants to merge 1 commit intomainfrom
lionel/test-cleanup-brittleness
Open

test: dedupe position balance helpers#203
liobrasil wants to merge 1 commit intomainfrom
lionel/test-cleanup-brittleness

Conversation

@liobrasil
Copy link
Contributor

Summary

  • centralize Flow collateral and MOET debt position helpers in cadence/tests/test_helpers.cdc
  • remove duplicated per-file helper implementations from the rebalance scenario tests
  • replace the small hardcoded pid: 1 diagnostic reads in scenarios 3a and 3b with the local pid

Validation

  • flow test cadence/tests/rebalance_scenario2_test.cdc cadence/tests/rebalance_scenario3a_test.cdc cadence/tests/rebalance_scenario3b_test.cdc cadence/tests/rebalance_scenario3c_test.cdc cadence/tests/rebalance_scenario3d_test.cdc

Notes

  • The touched scenario files still fail at their existing closeYieldVault path with FlowALP minimum-balance assertions. The helper cleanup compiled and executed normally before that pre-existing close step.

@liobrasil liobrasil requested a review from a team as a code owner March 9, 2026 23:51
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.

1 participant