Skip to content

test: cover attestation fee accounting#525

Closed
dangelo352 wants to merge 1 commit into
Commitlabs-Org:masterfrom
dangelo352:test/attestation-fee-accounting-478
Closed

test: cover attestation fee accounting#525
dangelo352 wants to merge 1 commit into
Commitlabs-Org:masterfrom
dangelo352:test/attestation-fee-accounting-478

Conversation

@dangelo352

Copy link
Copy Markdown

Summary

  • Add negative record_fees rejection coverage
  • Add withdrawal tests for missing recipient, non-positive amounts, over-withdraw, and per-asset isolation
  • Verify collected fee conservation against token balances after partial withdrawal
  • Cover large bounded fee accumulation and withdrawal
  • Document the attestation fee-accounting invariant in docs/FEE_MODEL_CROSS_CHECK.md

Closes #478.

Tests

  • CARGO_TARGET_DIR=/tmp/commitlabs-attestation-fee-target cargo test -p attestation_engine fee -- --nocapture
  • git diff --check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add fee-accounting tests for attestation_engine::record_fees and withdraw_fees

2 participants