Skip to content

feature:Add a mutation-testing setup for compliance and invoice, exte……nding #94's treasury-only coverage - #515

Merged
misrasamuelisiguzor-oss merged 3 commits into
WHEELBACK:mainfrom
nzubepolycap-hub:COMEBACKHERE-contracts470
Aug 31, 2026
Merged

feature:Add a mutation-testing setup for compliance and invoice, exte……nding #94's treasury-only coverage#515
misrasamuelisiguzor-oss merged 3 commits into
WHEELBACK:mainfrom
nzubepolycap-hub:COMEBACKHERE-contracts470

Conversation

@nzubepolycap-hub

Copy link
Copy Markdown
Contributor

closes #470

Work Summary

Objective

Extend the existing treasury mutation-testing workflow with equivalent jobs for
the compliance and invoice contracts.

Changes Made

  • Added a mutants-compliance GitHub Actions job in
    .github/workflows/test.yml.
    • Runs only for workflow_dispatch.
    • Installs Rust 1.95.0 and cargo-mutants.
    • Targets contracts/compliance/src/allowlist.rs.
  • Added a mutants-invoice GitHub Actions job in
    .github/workflows/test.yml.
    • Runs only for workflow_dispatch.
    • Installs Rust 1.95.0 and cargo-mutants.
    • Targets contracts/invoice/src/invoice.rs.
  • Preserved the existing mutants-treasury job structure and behavior.
  • Installed Rust locally with rustup so the workspace could be validated.
  • Corrected the compliance-client dependency configuration.
  • Updated stale invoice test calls to match the current create_invoice API.
  • Updated stale compliance and treasury test calls to match current contract
    entrypoint signatures.
  • Removed or corrected stale test helper and snapshot API assumptions.
  • Cleaned up related unused variables, imports, and lifetime annotations.

Branch and Commit

  • Branch: COMEBACKHERE-contracts470
  • Commit: ff4cbff
  • Commit message: feature:Add a mutation-testing setup for compliance and invoice, extending #94's treasury-only coverage

Validation

The following required command sequence was run:

cargo test --all
cargo clippy --all -- -D warnings
cargo fmt --all -- --check

Validation did not complete successfully. The test phase still reports
remaining legacy test-suite issues, including duplicate invoice test
definitions, outdated create_invoice calls missing the merchant_nonce
argument, and calls to APIs whose current return types no longer provide
.unwrap().

The workflow configuration itself is present and both new mutation jobs are
gated behind manual workflow_dispatch runs.

@drips-wave

drips-wave Bot commented Aug 29, 2026

Copy link
Copy Markdown

@nzubepolycap-hub Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@misrasamuelisiguzor-oss
misrasamuelisiguzor-oss merged commit 49e35f9 into WHEELBACK:main Aug 31, 2026
9 of 16 checks passed
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 a mutation-testing setup for compliance and invoice, extending #94's treasury-only coverage

2 participants