Skip to content

test: improve infection coverage#55

Merged
Max Stegmeyer (mstegmeyer) merged 4 commits intomainfrom
test/improve-infection-coverage
Mar 30, 2026
Merged

test: improve infection coverage#55
Max Stegmeyer (mstegmeyer) merged 4 commits intomainfrom
test/improve-infection-coverage

Conversation

@mstegmeyer
Copy link
Copy Markdown
Contributor

@mstegmeyer Max Stegmeyer (mstegmeyer) commented Mar 30, 2026

Summary

  • upgrade the remaining workflows from actions/checkout@v2 to @v4 to remove the Node 20 deprecation warning
  • add targeted mutation-killing tests around DynamoDB shop persistence, dual-signature fallback behavior, context resolver fallbacks, authenticated client token validation, payment responses, and storefront shop resolution

Why

The Infection job on main was reporting multiple escaped mutants plus the checkout deprecation warning. These changes remove the covered survivors and raise the mutation threshold to match the improved score.

Validation

  • vendor/bin/phpunit
  • devenv shell -- vendor/bin/infection --min-msi=99 --min-covered-msi=100 --threads=4

Comment thread src/Adapter/DynamoDB/DynamoDBRepository.php Outdated
@mstegmeyer Max Stegmeyer (mstegmeyer) merged commit faf860a into main Mar 30, 2026
12 checks passed
@mstegmeyer Max Stegmeyer (mstegmeyer) deleted the test/improve-infection-coverage branch March 30, 2026 11:57
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.

2 participants