CT-033: `liquidity_pool` contract uses `test_snapshots/` directory — snapshot tests may be stale

[Dev-Zully]

Category: Testing · Priority: Medium
Labels: contract, liquidity, testing

The contracts/liquidity_pool/ directory contains a test_snapshots/ directory. If these are insta snapshots, they may be stale after code changes and need to be reviewed/updated.

Acceptance Criteria:

• Run cargo insta review to check for pending snapshots
• Update any stale snapshots
• Add CI check for snapshot drift

[onyekachi66]

@onyekachi66 has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

I would like to work on this issue having worked on similar issues in the past. Please assign

ℹ️ Repo Maintainers: To accept this application, review their application or assign @onyekachi66 to this issue.

[drips-wave]

Congratulations, @33ibrahimbuhari-wq! 🎉 Your application was accepted by the repo's maintainers, and the issue is due on August 31, 2026.

🧑‍💻 @33ibrahimbuhari-wq: Please resolve the issue such that the repo's maintainers have enough time to review your contribution before the due date. You'll earn Points for completing the issue on-time, which will make you eligible for a share of the Stellar Wave Program's reward pool.

Warning

When opening a PR, please link it to this issue to ensure it gets tracked accurately. Points are awarded when this issue is marked as completed by the maintainer.

🤠 Repo maintainers: Please keep an eye on the contributor's progress and review their work before the due date. You can manage this issue, including adjusting its complexity and points, here.

🌊 Happy Wave 🌊