Test – get_fee_analytics and collect_transaction_fees

[Baskarayelu]

Description
Add tests for get_fee_analytics (period); collect_transaction_fees (internal) called after fee calculation; treasury and platform receive correct amounts.

Requirements and context

• Achieve minimum 95% test coverage for fee analytics and collection
• Smart contracts only (Soroban/Rust)

Suggested execution

1. Fork the repo and create a branch
   git checkout -b test/fee-analytics-collect
2. Implement changes
   • Add tests in src/test_fees.rs
3. Test and commit
   • Run cargo test
   • Attach test output

Example commit message
test: get_fee_analytics and collect_transaction_fees

Guidelines

• Minimum 95 percent test coverage
• Clear documentation
• Timeframe: 96 hours

[drips-wave]

This issue has been added to the Stellar Wave Program and is worth 200 Points.

🧑‍💻 Interested in contributing? Apply to work on this issue on Drips Wave, earn points, and receive rewards.

Warning

Only applications submitted via the apply link above will be considered. Please do not apply by commenting on the issue or opening a PR directly.

🤠 Repo maintainers: You can manage this issue's Points and Complexity on your Maintainer Dashboard here. Please keep an eye on applications and respond quickly 💙

ℹ️ Learn more about Drips Wave

[Emmyt24]

@Emmyt24 has applied to work on this issue as part of the Stellar Wave Program's 2nd wave.

Hi team, I’d like to work on this issue.

I’m a Soroban/Rust smart contract developer with a security-first mindset, and I’ve written high-coverage test suites for fee accounting (period analytics, correct splits, and treasury/platform payouts) to prevent silent revenue bugs.

What I’ll deliver

A new/updated src/test_fees.rs covering:

get_fee_analytics(period) returns correct totals for the selected period (and doesn’t leak values across periods)

collect_transaction_fees is triggered after fee calculation (validated via observable effects: balances/state counters updated)

Treasury + platform receive the exact expected amounts based on the fee split rules (including rounding behavior)

Negative/edge cases where applicable (zero-fee transactions, multiple transactions aggregated, period boundary)

Tests documented with clear assertions and #[should_panic] where failures are expected

Target ≥95% coverage for fee analytics + collection logic, with cargo test output included in PR notes

Approach

Use soroban_sdk::testutils to set up mock addresses (user/treasury/platform) and deterministic ledger timing for period windows

Drive the contract through the public flow that performs fee calculation so collect_transaction_fees executes naturally

Assert:

analytics state (per-period totals/counters)

token balances/escrow movement for treasury/platform

invariants across multiple calls (no double-collection)

I’ll work from test/fee-analytics-collect and submit a PR with the commit message: test: get_fee_analytics and collect_transaction_fees.

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

[drips-wave]

Congratulations, @Emmyt24! 🎉 Your application was accepted by the repo's maintainers, and the issue is due on February 26, 2026.

🧑‍💻 @Emmyt24: 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 issued 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 🌊

[Tola-byte]

@Tola-byte has applied to work on this issue as part of the Stellar Wave Program's 2nd wave.

I will like to work on this

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