Skip to content
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

Add Pre Update Password action management related files #6281

Merged
merged 19 commits into from
Jan 25, 2025

Conversation

ashanthamara
Copy link
Contributor

@ashanthamara ashanthamara commented Jan 13, 2025

Proposed changes in this pull request

  1. Implement necessary classes required to perform action management related operations for pre update password action

Related PRs:

Related Issue

Copy link

codecov bot commented Jan 13, 2025

Codecov Report

Attention: Patch coverage is 84.93151% with 33 lines in your changes missing coverage. Please review.

Project coverage is 46.41%. Comparing base (f4edfa8) to head (2e2c319).
Report is 23 commits behind head on master.

Files with missing lines Patch % Lines
...ernal/PreUpdatePasswordActionServiceComponent.java 0.00% 21 Missing ⚠️
...ement/PreUpdatePasswordActionDTOModelResolver.java 91.17% 0 Missing and 9 partials ⚠️
...ment/service/impl/ActionManagementServiceImpl.java 81.81% 1 Missing and 1 partial ⚠️
...n/management/dao/impl/ActionManagementDAOImpl.java 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #6281      +/-   ##
============================================
+ Coverage     46.33%   46.41%   +0.08%     
- Complexity    14681    14735      +54     
============================================
  Files          1707     1713       +6     
  Lines        103362   103578     +216     
  Branches      18508    18528      +20     
============================================
+ Hits          47895    48078     +183     
- Misses        48627    48649      +22     
- Partials       6840     6851      +11     
Flag Coverage Δ
unit 30.01% <84.93%> (+0.12%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ashanthamara ashanthamara marked this pull request as ready for review January 25, 2025 09:11
malithie
malithie previously approved these changes Jan 25, 2025
malithie
malithie previously approved these changes Jan 25, 2025
@ashanthamara
Copy link
Contributor Author

Not running the Integration Test Runner, as this only introduces new component. Will run the PR with the api-server PR.

@ashanthamara ashanthamara merged commit a752ab1 into wso2:master Jan 25, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants