wso2 / carbon-identity-framework

Apache License 2.0
120 stars 541 forks source link

Add In-Memory Claim Management #6136

Closed amanda-ariyaratne closed 4 days ago

amanda-ariyaratne commented 4 days ago

Reverts wso2/carbon-identity-framework#6127

sonarcloud[bot] commented 4 days ago

Quality Gate Passed Quality Gate passed

Issues
59 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.6% Duplication on New Code

See analysis details on SonarQube Cloud

codecov[bot] commented 4 days ago

Codecov Report

Attention: Patch coverage is 84.69602% with 73 lines in your changes missing coverage. Please review.

Project coverage is 41.03%. Comparing base (0fc102d) to head (eab9155). Report is 13 commits behind head on master.

Files with missing lines Patch % Lines
...laim/metadata/mgt/UnifiedClaimMetadataManager.java 88.64% 12 Missing and 9 partials :warning:
.../claim/metadata/mgt/DefaultClaimMetadataStore.java 0.00% 17 Missing :warning:
...tadata/mgt/ClaimMetadataManagementServiceImpl.java 74.60% 12 Missing and 4 partials :warning:
...ty/claim/metadata/mgt/util/ClaimMetadataUtils.java 75.00% 6 Missing and 4 partials :warning:
...identity/claim/metadata/mgt/dao/LocalClaimDAO.java 80.95% 3 Missing and 1 partial :warning:
...etadata/mgt/SystemDefaultClaimMetadataManager.java 97.33% 1 Missing and 1 partial :warning:
...rbon/identity/claim/metadata/mgt/dao/ClaimDAO.java 88.23% 2 Missing :warning:
...laim/metadata/mgt/DBBasedClaimMetadataManager.java 98.30% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #6136 +/- ## ============================================ + Coverage 40.75% 41.03% +0.28% - Complexity 14633 14754 +121 ============================================ Files 1772 1777 +5 Lines 119759 118558 -1201 Branches 19523 20005 +482 ============================================ - Hits 48808 48656 -152 + Misses 63595 62586 -1009 + Partials 7356 7316 -40 ``` | [Flag](https://app.codecov.io/gh/wso2/carbon-identity-framework/pull/6136/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=wso2) | Coverage Δ | | |---|---|---| | [unit](https://app.codecov.io/gh/wso2/carbon-identity-framework/pull/6136/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=wso2) | `25.61% <84.69%> (+0.41%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=wso2#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


🚨 Try these New Features:

amanda-ariyaratne commented 4 days ago

Integration tests are expected to fail by this PR. They will be fixed with https://github.com/wso2/product-is/pull/21601