-
Notifications
You must be signed in to change notification settings - Fork 150
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix realm normalization coverage #1233
base: main
Are you sure you want to change the base?
Conversation
This is a squash of the commits from multiple repositories: Original PR by sonOfRa: #818 Adaptations by thomasdarimont: https://github.com/thomasdarimont/keycloak-config-cli/tree/feature/realm-normalization Fixes #799 Co-authored-by: Simon Levermann <simon.levermann@codecentric.de> Signed-off-by: Thomas Darimont <thomas.darimont@googlemail.com>
Quality Gate passedIssues Measures |
Hi @thomasdarimont, I’ve added additional test cases to ensure comprehensive coverage for the recent implementation. Please review them at your convenience and verify their correctness. The workflow has now been successfully completed. |
@thomasdarimont can you rebase this onto your branch for a cleaner integration? |
What this PR does / why we need it:
Which issue this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged): This is a fix for the open ticket #1207.The change increases the coverage for the realm Normalization feature.
Special notes for your reviewer:
PR Readiness Checklist:
Complete these before marking the PR as
ready to review
:CHANGELOG.md
release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR