-
Notifications
You must be signed in to change notification settings - Fork 378
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
Improve user info response to return roles without internal domain appended #2674
Improve user info response to return roles without internal domain appended #2674
Conversation
e4e88d6
to
2dcb821
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2674 +/- ##
============================================
- Coverage 56.18% 51.79% -4.40%
+ Complexity 8242 6433 -1809
============================================
Files 631 631
Lines 46794 52699 +5905
Branches 9043 10277 +1234
============================================
+ Hits 26291 27294 +1003
- Misses 16728 21530 +4802
- Partials 3775 3875 +100
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
49a33b9
to
63e5358
Compare
PR builder started |
PR builder completed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving the pull request based on the successful pr build https://github.com/wso2/product-is/actions/runs/12810813485
e2a2fd6
to
18e1094
Compare
ea35ff5
to
8559bcf
Compare
The added test cases cover the flow. But the test classes written in the oauth endpoint module. Even the required case is validated, the code coverage is not counted as the code is covered from the a different mdoule. |
Proposed changes in this pull request
Add the below config to enable this improvement.
Related Issues