Repository navigation
Add boundary tests for the fees logic #294
Copy link
Copy link
Labels
GRANTFOX OSSGrantFox open-source campaign taskGrantFox open-source campaign taskMAYBE REWARDEDMay be rewarded under the GrantFox campaignMay be rewarded under the GrantFox campaignOFFICIAL CAMPAIGNOfficial GrantFox campaign issueOfficial GrantFox campaign issueOfficial Campaign | FWC26GrantFox FWC26 official campaignGrantFox FWC26 official campaignarea:feesfeesfeespriority:highHigh priorityHigh prioritystack:rustRustRuststack:sorobanSoroban smart contractSoroban smart contracttype:testTest coverage workTest coverage work
Description
Cover fees boundaries
Description
The fees logic is thinly tested at its boundaries. This issue adds focused boundary and rejection tests.
Requirements and context
- Repository scope: Agentpay-Org/Agentpay-contracts only.
- Add tests for the accept/reject boundaries of fees, asserting exact typed error codes.
- Use the test-utils helpers; assert events where the flow emits them.
- Do not change contract logic unless a defect is found (note it).
Suggested execution
- Fork the repo and create a branch
git checkout -b test/fees-01-boundaries- Implement changes
- Write code in: the relevant module.
- Write comprehensive tests in: cover the new behaviour and edge cases.
- Test and commit
Test and commit
- Run
cargo fmt,cargo clippy --all-targets -- -D warnings, andcargo test. - Cover edge cases: exactly-at boundary, one over, unauthorized caller.
- Include the full test output in the PR description.
Example commit message
test(fees): cover boundaries and rejections
Guidelines
- Minimum 95 percent test coverage for impacted modules.
- Clear, reviewer-focused documentation.
- Timeframe: 96 hours.
Community & contribution rewards
- 💬 Join the AgentPay community on Discord: https://discord.gg/eXvRKkgcv
- ⭐ This is a GrantFox OSS / Official Campaign task and may be rewarded. When your PR is merged you'll be prompted to rate the project — a 5-star rating is much appreciated.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
GRANTFOX OSSGrantFox open-source campaign taskGrantFox open-source campaign taskMAYBE REWARDEDMay be rewarded under the GrantFox campaignMay be rewarded under the GrantFox campaignOFFICIAL CAMPAIGNOfficial GrantFox campaign issueOfficial GrantFox campaign issueOfficial Campaign | FWC26GrantFox FWC26 official campaignGrantFox FWC26 official campaignarea:feesfeesfeespriority:highHigh priorityHigh prioritystack:rustRustRuststack:sorobanSoroban smart contractSoroban smart contracttype:testTest coverage workTest coverage work
Activity
Jagadeeshftw commentedon Jul 28, 2026
Hi, I’d love to take on this issue! With my expertise in Rust, Stellar, and smart contract development, I can implement a clean and efficient solution.
🚀 Why Me?
⏱ Timeline
I estimate completing this within 24 hours. Let me know if I can proceed!
grantfox-oss commentedon Jul 28, 2026
🦊 GrantFox — @Jagadeeshftw has been assigned to this issue as part of the Official Campaign | FWC26 campaign!
Next steps:
Closes #294)Good luck! Track your progress on GrantFox.
mikewheeleer commentedon Jul 28, 2026
🎉 You're assigned, @Jagadeeshftw! Part of the Stellar Wave Program — join our Telegram to coordinate: https://t.me/+OhR36aTevMYwYzI9 🚀
grantfox-oss commentedon Jul 29, 2026
🎉 This issue has been marked as completed on GrantFox as part of the Official Campaign | FWC26 campaign!
@Jagadeeshftw's PR #439 was approved and merged by @mikewheeleer.
🏆 @Jagadeeshftw: You earned 45 FoxPoints for this contribution! Your current tier: Specialist (3,626 total points). Track your full progress on GrantFox.
👏 Great work, @Jagadeeshftw! Keep contributing to Agentpay-Org.