Skip to content

Revenue pool: prove receive_payment cannot move tokens (docs + tests) #159

@greatest0fallt1me

Description

@greatest0fallt1me

Description

Add tests demonstrating receive_payment does not invoke token transfers; document the threat-model note for ops.

Requirements and context

  • Prevents confusion with real settlement.
  • Align with SECURITY.md external call checklist.

Suggested execution

  • Fork the repo and create a branch: git checkout -b test/revenue-pool-receive-payment-security
  • Implement changes in the Callora-Contracts Soroban workspace (contracts/*/src/).
  • Run cargo fmt, cargo clippy --all-targets --all-features -- -D warnings, and cargo test from the workspace root.
  • For WASM release builds: cargo build --target wasm32-unknown-unknown --release -p callora-revenue-pool (adjust -p if the crate name differs).

Deliverables

  • test.rs + rustdoc.

Test and commit

  • Run tests and ./scripts/coverage.sh (or cargo tarpaulin per tarpaulin.toml).
  • Cover edge cases; include summarized test output and brief security notes in the PR description.

Example commit message

test(revenue-pool): receive_payment does not transfer tokens

Guidelines

  • Minimum 95% line coverage for touched crates (workspace policy).
  • Clear documentation (Rust /// on public items where applicable; repo markdown as needed).
  • Timeframe: 96 hours from assignment unless agreed otherwise.

Metadata

Metadata

Assignees

No one assigned

    Type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions