Skip to content

sscsft-152 Add Confidentiality Confirmed Event flow#5099

Open
hemantasharma1129 wants to merge 1 commit intomasterfrom
sscsft-152-confidentiality-confirmed-event
Open

sscsft-152 Add Confidentiality Confirmed Event flow#5099
hemantasharma1129 wants to merge 1 commit intomasterfrom
sscsft-152-confidentiality-confirmed-event

Conversation

@hemantasharma1129
Copy link
Contributor

SSCSFT-152

Change description

  1. Created new Case event and case vent fields for Confidentiality Confirmed.
  2. Created a new Mid Event handler to validate the flow.
  3. Created new Submit handler to reset InterlocReview state.

Testing done

  1. Performed Manual Testing from the portal by manipulating data as the ticket depends upon SSCSFT-146.
  2. Following Tests - Unit, Integration and Functional tests are written to cover various scenarios.

Use cases:

  1. Show/Hide the new event Confidentiality Confirmed based on entitlements.
  2. Via Midevent handler validation, if OtherParty's confidentiality required is Undetermined show error message.
  3. Via Submit Handler, on successful submission, reset the InterlocReview state to NULL.

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?

  • Yes
  • No

Checklist

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • Does this PR introduce a breaking change

Copy link
Contributor

@raghera raghera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good stuff in general. A couple of comments, let me know if you would like to discuss.

Copy link
Contributor

@raghera raghera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good Hemanta!

@hemantasharma1129 hemantasharma1129 force-pushed the sscsft-152-confidentiality-confirmed-event branch from 3415bd3 to 21e935c Compare March 6, 2026 12:03
@github-actions
Copy link

github-actions bot commented Mar 6, 2026

CCD diff report

AuthorisationCaseField/AuthorisationCaseField-cm-nonprod.json

CRUDAccessProfileCaseFieldID
+CRUD
+caseworker-sscs-clerk
+confidentialityConfirmedLabel
+CRUD
+caseworker-sscs-superuser
+confidentialityConfirmedLabel
+CRUD
+caseworker-sscs
+confidentialityConfirmedLabel

CaseEvent/CaseEvent-cm-nonprod.json

DescriptionPublishEventEnablingConditionPostConditionStatePreConditionState(s)NameIDCallBackURLAboutToSubmitEvent
+Confidentiality Confirmed
+N
+appeal.benefitType.code="childSupport"
+withDwp
+awaitConfidentialityRequirements
+Confidentiality Confirmed
+confidentialityConfirmed
+${CCD_DEF_TRIBUNALS_API_URL}/ccdAboutToSubmit

CaseEventToFields/CaseEventToFields-cm-nonprod.json

CallBackURLMidEventPageLabelPublishPageIDPageFieldDisplayOrderPageDisplayOrderDisplayContextCaseFieldIDCaseEventID
+${CCD_DEF_TRIBUNALS_API_URL}/ccdMidEvent
+
+N
+1.0
+1
+1
+READONLY
+confidentialityConfirmedLabel
+confidentialityConfirmed

@github-actions
Copy link

github-actions bot commented Mar 6, 2026

Integration Tests results

   85 files  +1     85 suites  +1   11m 17s ⏱️ +8s
1 492 tests +1  1 484 ✅ +1  8 💤 ±0  0 ❌ ±0 
1 495 runs  +1  1 487 ✅ +1  8 💤 ±0  0 ❌ ±0 

Results for commit 9b9d556. ± Comparison against base commit 0c5eac7.

This pull request removes 12 and adds 13 tests. Note that renamed tests count towards both.
uk.gov.hmcts.reform.sscs.bulkscan.controllers.SscsBulkScanExceptionRecordCallback ‑ [1] url=http://localhost:44949/transform-exception-record/, isAuto=false
uk.gov.hmcts.reform.sscs.bulkscan.controllers.SscsBulkScanExceptionRecordCallback ‑ [2] url=http://localhost:44949/transform-scanned-data/, isAuto=true
uk.gov.hmcts.reform.sscs.tyanotifications.config.BundledNotificationsConfigTestIt ‑ given_bundledLetters_should_notHaveTemplate(DIRECTION_ISSUED) [6]
uk.gov.hmcts.reform.sscs.tyanotifications.config.BundledNotificationsConfigTestIt ‑ given_bundledLetters_should_notHaveTemplate(DIRECTION_ISSUED_WELSH) [1]
uk.gov.hmcts.reform.sscs.tyanotifications.config.BundledNotificationsConfigTestIt ‑ given_bundledLetters_should_notHaveTemplate(LIBERTY_TO_APPLY_GRANTED) [7]
uk.gov.hmcts.reform.sscs.tyanotifications.config.BundledNotificationsConfigTestIt ‑ given_bundledLetters_should_notHaveTemplate(LIBERTY_TO_APPLY_REFUSED) [0]
uk.gov.hmcts.reform.sscs.tyanotifications.config.BundledNotificationsConfigTestIt ‑ given_bundledLetters_should_notHaveTemplate(PROCESS_AUDIO_VIDEO) [8]
uk.gov.hmcts.reform.sscs.tyanotifications.config.BundledNotificationsConfigTestIt ‑ given_bundledLetters_should_notHaveTemplate(PROCESS_AUDIO_VIDEO_WELSH) [9]
uk.gov.hmcts.reform.sscs.tyanotifications.config.BundledNotificationsConfigTestIt ‑ given_bundledLetters_should_notHaveTemplate(SET_ASIDE_GRANTED) [4]
uk.gov.hmcts.reform.sscs.tyanotifications.config.BundledNotificationsConfigTestIt ‑ given_bundledLetters_should_notHaveTemplate(SET_ASIDE_REFUSED) [3]
…
uk.gov.hmcts.reform.sscs.bulkscan.controllers.SscsBulkScanExceptionRecordCallback ‑ [1] url=http://localhost:42481/transform-exception-record/, isAuto=false
uk.gov.hmcts.reform.sscs.bulkscan.controllers.SscsBulkScanExceptionRecordCallback ‑ [2] url=http://localhost:42481/transform-scanned-data/, isAuto=true
uk.gov.hmcts.reform.sscs.callback.ConfidentialityConfirmedIt ‑ callToMidEventHandler_returnsErrorWhenConfidentialityMissing
uk.gov.hmcts.reform.sscs.tyanotifications.config.BundledNotificationsConfigTestIt ‑ given_bundledLetters_should_notHaveTemplate(DIRECTION_ISSUED) [1]
uk.gov.hmcts.reform.sscs.tyanotifications.config.BundledNotificationsConfigTestIt ‑ given_bundledLetters_should_notHaveTemplate(DIRECTION_ISSUED_WELSH) [7]
uk.gov.hmcts.reform.sscs.tyanotifications.config.BundledNotificationsConfigTestIt ‑ given_bundledLetters_should_notHaveTemplate(LIBERTY_TO_APPLY_GRANTED) [0]
uk.gov.hmcts.reform.sscs.tyanotifications.config.BundledNotificationsConfigTestIt ‑ given_bundledLetters_should_notHaveTemplate(LIBERTY_TO_APPLY_REFUSED) [8]
uk.gov.hmcts.reform.sscs.tyanotifications.config.BundledNotificationsConfigTestIt ‑ given_bundledLetters_should_notHaveTemplate(PROCESS_AUDIO_VIDEO) [9]
uk.gov.hmcts.reform.sscs.tyanotifications.config.BundledNotificationsConfigTestIt ‑ given_bundledLetters_should_notHaveTemplate(PROCESS_AUDIO_VIDEO_WELSH) [4]
uk.gov.hmcts.reform.sscs.tyanotifications.config.BundledNotificationsConfigTestIt ‑ given_bundledLetters_should_notHaveTemplate(SET_ASIDE_GRANTED) [3]
…

♻️ This comment has been updated with latest results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants