Skip to content

Improve tests and authentication success object construction. - #20

Merged
tvdijen merged 1 commit into
simplesamlphp:masterfrom
ioigoume:parse-all-authentication-success-elements
Nov 21, 2025
Merged

tvdijen merged 1 commit into
simplesamlphp:masterfrom
ioigoume:parse-all-authentication-success-elements

Conversation

@ioigoume

Copy link
Copy Markdown
Contributor

No description provided.

@ioigoume
ioigoume force-pushed the parse-all-authentication-success-elements branch from a0247aa to 6035ba2 Compare November 21, 2025 10:48
@ioigoume
ioigoume force-pushed the parse-all-authentication-success-elements branch from 6035ba2 to c5e73d8 Compare November 21, 2025 10:50
@codecov

codecov Bot commented Nov 21, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 60.86957% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 95.75%. Comparing base (b026d07) to head (c5e73d8).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@              Coverage Diff              @@
##              master      #20      +/-   ##
=============================================
- Coverage     100.00%   95.75%   -4.25%     
- Complexity        43       54      +11     
=============================================
  Files             10       10              
  Lines            190      212      +22     
=============================================
+ Hits             190      203      +13     
- Misses             0        9       +9     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ioigoume
ioigoume requested review from pradtke and tvdijen November 21, 2025 10:55
@ioigoume ioigoume self-assigned this Nov 21, 2025
@tvdijen
tvdijen merged commit 4d85173 into simplesamlphp:master Nov 21, 2025
22 checks passed
@ioigoume
ioigoume deleted the parse-all-authentication-success-elements branch December 1, 2025 10:47
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Mar 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants