-
Notifications
You must be signed in to change notification settings - Fork 130
Clarify and test cancel_bid versus withdraw_bid semantics #711
Copy link
Copy link
Open
Labels
Stellar WaveIssues in the Stellar wave programIssues in the Stellar wave programaccess-controlAuto label for contract issues: access-controlAuto label for contract issues: access-controlbiddingDesign / UX backlog: biddingDesign / UX backlog: biddingpriority-highAuto label for contract issues: priority-highAuto label for contract issues: priority-highsmart-contractsAuto label for contract issues: smart-contractsAuto label for contract issues: smart-contractstestingAuto label for contract issues: testingAuto label for contract issues: testing
Description
Description
Document and test differences between cancel and withdraw paths, status transitions, and events.
Requirements and context
- Must be secure, tested, and documented
- Should be efficient and easy to review
- Align with existing Soroban contract patterns in
quicklendx-contracts
Suggested execution
- Fork the repo and create a branch
git checkout -b feature/cancel-vs-withdraw-bid- Implement changes
- Write contract:
src/bid.rs,src/lib.rs - Write comprehensive tests:
src/test_bid.rs - Add documentation:
docs/contracts/bidding.md - Include NatSpec-style comments (Rust doc comments on public items)
- Validate security assumptions
Test and commit
- Run tests
- Cover edge cases
- Include test output and security notes
Example commit message
feat: document cancel_bid versus withdraw_bid
Guidelines
- Minimum 95 percent test coverage
- Clear documentation
- Timeframe: 96 hours
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Stellar WaveIssues in the Stellar wave programIssues in the Stellar wave programaccess-controlAuto label for contract issues: access-controlAuto label for contract issues: access-controlbiddingDesign / UX backlog: biddingDesign / UX backlog: biddingpriority-highAuto label for contract issues: priority-highAuto label for contract issues: priority-highsmart-contractsAuto label for contract issues: smart-contractsAuto label for contract issues: smart-contractstestingAuto label for contract issues: testingAuto label for contract issues: testing