Skip to content

Payjoin ffi test coverage and mutants - #1895

Closed
benalleng wants to merge 5 commits into
payjoin:masterfrom
benalleng:payjoin-ffi-mutants
Closed

benalleng wants to merge 5 commits into
payjoin:masterfrom
benalleng:payjoin-ffi-mutants

Conversation

@benalleng

Copy link
Copy Markdown
Collaborator

Closes #1888

This adds both mutants coverage as well as coveralls tracking.

Pull Request Checklist

Please confirm the following before requesting review:

@coveralls

coveralls commented Sep 21, 2026 •

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 37464696876

Coverage increased (+0.2%) to 87.724%

Details

  • Coverage increased (+0.2%) from the base build.
  • Patch coverage: 28 uncovered changes across 2 files (1211 of 1239 lines covered, 97.74%).
  • No coverage regressions found.

Uncovered Changes

File Changed Covered %
payjoin-ffi/src/receive/mod.rs 752 737 98.01%
payjoin-ffi/src/send/mod.rs 387 374 96.64%
Total (4 files) 1239 1211 97.74%

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 20764
Covered Lines: 18215
Line Coverage: 87.72%
Coverage Strength: 279.12 hits per line

💛 - Coveralls

@benalleng

This comment was marked as resolved.

@benalleng
benalleng force-pushed the payjoin-ffi-mutants branch from fdfe9d2 to 13c67db Compare October 5, 2026 17:03
@benalleng
benalleng force-pushed the payjoin-ffi-mutants branch from 988301c to 2fd60ff Compare October 5, 2026 18:02
@benalleng
benalleng force-pushed the payjoin-ffi-mutants branch from fac7f44 to d72ce45 Compare October 6, 2026 12:38
@benalleng
benalleng marked this pull request as ready for review October 6, 2026 12:38
@benalleng benalleng changed the title Payjoin ffi mutants Payjoin ffi test coverage and mutants Oct 6, 2026
@benalleng

Copy link
Copy Markdown
Collaborator Author

Done, I am happy with the +0.2% coverage from the ffi

@caarloshenriq caarloshenriq left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tACK

Ran the ffi tests, coverage, and cargo-mutants locally, all green. Solid coverage: 51 offline, deterministic tests behind _test-utils

@DanGould

Copy link
Copy Markdown
Member

this branch seems fine, but we're closing it for now to take stuff off the plate. Looks like the issue needs a bit of fleshing out. Thanks for review @caarloshenriq but this one was a bit premature

@DanGould DanGould closed this Oct 13, 2026
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.

Should payjoin-ffi be covered by cargo mutants?

4 participants