HDPI-5188 - Warrant of Restitution update data model#1554
Open
guygrewal77 wants to merge 90 commits intomasterfrom
Open
HDPI-5188 - Warrant of Restitution update data model#1554guygrewal77 wants to merge 90 commits intomasterfrom
guygrewal77 wants to merge 90 commits intomasterfrom
Conversation
…chema, service, and test updates.
…nforcement-Writ-Details # Conflicts: # src/main/java/uk/gov/hmcts/reform/pcs/ccd/service/enforcetheorder/warrant/EnforcementOrderService.java # src/main/resources/testing-support/Create-Case-Make-A-Claim-Basic-Case.json # src/main/resources/testing-support/Create-Enforcement-Writ-Basic-Case.json
…rcementWritEntity, database schema, and associated tests.
…try-with-resources for safe InputStream handling.
…itEntity` and `WritRepository` respectively. Update references across services and tests for consistency.
…nforcement-Writ-Details
…ntOrderService` to delegate specific processing logic for Writ and Warrant to their respective strategies. Update services, mappers, and tests accordingly.
…nforcement-Writ-Details # Conflicts: # src/main/java/uk/gov/hmcts/reform/pcs/ccd/service/enforcetheorder/warrant/EnforcementOrderService.java # src/test/java/uk/gov/hmcts/reform/pcs/ccd/service/enforcetheorder/EnforcementOrderServiceTest.java
…nforcement-Writ-Details
…he codebase, update related methods, mappers, tests, and test resources for consistency and readability.
… imports across services, strategies, and tests. Annotate `saveAndClearDraftData` with `@Transactional`.
…nforcement-Writ-Details
…sion methods and update affected tests.
…pdate related file paths and constants for consistency.
…le paths to align with renamed `testcasegeneration` directory.
…pportHelper` to improve debugging.
…ean up test setup.
…tion` for naming consistency.
…nforcement-Writ-Details
…y across enforcement order services. Streamline null checks in mappers and update associated tests.
…nforcement-Writ-Details # Conflicts: # src/main/java/uk/gov/hmcts/reform/pcs/ccd/repository/enforcetheorder/EnforcementWarrantRepository.java # src/main/java/uk/gov/hmcts/reform/pcs/ccd/service/enforcetheorder/EnforcementOrderService.java # src/main/java/uk/gov/hmcts/reform/pcs/ccd/service/enforcetheorder/mapper/EnforcementWarrantMapper.java # src/test/java/uk/gov/hmcts/reform/pcs/ccd/service/enforcetheorder/EnforcementOrderServiceTest.java # src/test/java/uk/gov/hmcts/reform/pcs/testingsupport/EnforcementDMTest.java
# Conflicts: # src/main/java/uk/gov/hmcts/reform/pcs/ccd/service/enforcetheorder/EnforcementOrderService.java # src/test/java/uk/gov/hmcts/reform/pcs/ccd/service/enforcetheorder/EnforcementOrderServiceTest.java
Update of tests to accomodate the change.
…nforcement-Writ-Details
guygrewal77
removed
enable_e2e_test
guygrewal77
added
enable_e2e_test
tvr-hmcts
approved these changes
Mar 27, 2026
| applyRawWarrantRestDetails(enforcementOrder.getRawWarrantRestDetails(), riskProfileEntity); | ||
| } | ||
| default -> { | ||
| } // Not needed for other enforcement types |
Contributor
There was a problem hiding this comment.
Confirm Eviction will use this I suspect.
guygrewal77
removed
enable_e2e_test
aqibh25
approved these changes
Mar 30, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Jira link
https://tools.hmcts.net/jira/browse/HDPI-5188
Change description
This PR carries out some schema changes to introduce the columns for the warrant of restitution domain model. The corresponding entity classes are implemented and also the functionality to persist the data for the pages implemented so far. Alongside the unit tests have been updated for the changes and refactoring work carried out to use Risk Profile and Document persistence.
Testing done
XUI testing to perform Warrant of Possession and Warrant of Restitution journeys to confirm the data is pre-populated from Warrant into Warrant of Restitution pages like Risk Profile, Vulnerable People.
Checked DB to confirm the data is persisted as expected in the various tables except Statement of Truth which isn't implemented yet. Document persistence was tested on PR env. There currently isn't the description column in the Document table in the DB.
Security Vulnerability Assessment
CVE Suppression: Are there any CVEs present in the codebase (either newly introduced or pre-existing) that are being intentionally suppressed or ignored by this commit?
Checklist