Skip to content

Add PHP 8.5 to CI matrix, fix test deprecation#5

Merged
maksimovic merged 3 commits intomasterfrom
php85
Mar 17, 2026
Merged

Add PHP 8.5 to CI matrix, fix test deprecation#5
maksimovic merged 3 commits intomasterfrom
php85

Conversation

@maksimovic
Copy link
Owner

  • Add PHP 8.5 to CI test matrix
  • Fix PHPUnit deprecation warning in testVerifyMalformedJWS_RS256_to_HS256_with_explicit_alg (expectation of E_NOTICE is deprecated in PHPUnit 9+)
  • Update README to list 8.5 support
  • All 74 tests pass on PHP 8.5

Note: phpseclib v2 emits deprecation notices on PHP 8.4/8.5 (chr() out of range, xml_set_object deprecated) — these are upstream issues in phpseclib, not in this package.

@codecov
Copy link

codecov bot commented Mar 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.89%. Comparing base (567ca95) to head (972425b).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master       #5      +/-   ##
============================================
+ Coverage     94.45%   96.89%   +2.43%     
  Complexity      187      187              
============================================
  Files             6        6              
  Lines           451      451              
============================================
+ Hits            426      437      +11     
+ Misses           25       14      -11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Adds 28 new test methods covering: JWE construction from JWT instances,
dir algorithm with A256CBC-HS512, RSA-OAEP decrypt paths, JWK kid
propagation, phpseclib RSA object inputs, unsupported algorithm errors
(A256KW, ECDH-ES variants), invalid auth tag detection, and round-trip
encryption of long payloads. JWE method coverage: 42% -> 58%, line
coverage: 88% -> 94%. Total line coverage: 94% -> 97%.
@maksimovic maksimovic merged commit 45e1986 into master Mar 17, 2026
11 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.

1 participant