-
Notifications
You must be signed in to change notification settings - Fork 544
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
Introduce SAMLSSOPersistenceManagerFactory to switch between different persistence implementation #6240
Conversation
dd68425
to
d7d8318
Compare
...re/src/test/java/org/wso2/carbon/identity/core/dao/SAMLSSOPersistenceManagerFactoryTest.java
Outdated
Show resolved
Hide resolved
...y.core/src/main/java/org/wso2/carbon/identity/core/dao/SAMLSSOPersistenceManagerFactory.java
Outdated
Show resolved
Hide resolved
...y.core/src/main/java/org/wso2/carbon/identity/core/dao/SAMLSSOPersistenceManagerFactory.java
Outdated
Show resolved
Hide resolved
...identity.core/src/main/java/org/wso2/carbon/identity/core/SAMLSSOServiceProviderManager.java
Show resolved
Hide resolved
...y.core/src/main/java/org/wso2/carbon/identity/core/dao/SAMLSSOPersistenceManagerFactory.java
Outdated
Show resolved
Hide resolved
2c347ad
to
426f683
Compare
426f683
to
7c68916
Compare
Quality Gate failedFailed conditions |
PR builder started |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #6240 +/- ##
============================================
+ Coverage 45.66% 45.71% +0.04%
- Complexity 14080 14091 +11
============================================
Files 1632 1633 +1
Lines 100764 100857 +93
Branches 17705 17723 +18
============================================
+ Hits 46016 46104 +88
+ Misses 48036 48035 -1
- Partials 6712 6718 +6
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
PR builder completed |
PR builder started |
PR builder completed |
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/12431396705
Proposed changes in this pull request
$subject. Related to,