Skip to content

CIMD-3818 Create Summons application manually to validate duplicate URN#74

Open
mkarakaya55 wants to merge 3 commits into
team/CIMD-3817from
CIMD-3818-duplicate-urn-summons-validation
Open

CIMD-3818 Create Summons application manually to validate duplicate URN#74
mkarakaya55 wants to merge 3 commits into
team/CIMD-3817from
CIMD-3818-duplicate-urn-summons-validation

Conversation

@mkarakaya55
Copy link
Copy Markdown

Jira link

CIMD-3818

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

…ding summons exists

Add OR condition to the DUPLICATED_PROSECUTION guard in receiveSjpProsecution()
and receiveCCCase() so that any new case or summons submission is rejected when
applicationIdToDefendantIdsMap is non-empty (i.e. a pending summons already claims
the same URN). Four parameterised unit tests added following TDD red-green cycle;
full suite: 505 tests, 0 failures.

Constitution Principles touched:
  II  (CQRS Three-Layer Discipline): command side only — no new event emitted,
       listener and processor layers confirmed unaffected.
  VIII (Test-Driven Development): failing tests written and confirmed failing before
       production guard conditions were added.

Principles confirmed unaffected:
  I   (Contract First): no new RAML or JSON schema changes.
  III (CPP Framework Idioms): aggregate-only change, existing patterns preserved.
  VI  (Schema-Subscription Symmetry): no new event, no subscription change.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@mkarakaya55 mkarakaya55 requested a review from a team as a code owner May 19, 2026 22:33
@mkarakaya55 mkarakaya55 requested review from ConnorOKane-Kainos, MohamedFarouk-HMCTS and sahil-arora-hmcts and removed request for a team May 19, 2026 22:33
mkarakaya55 and others added 2 commits May 20, 2026 10:49
…ding summons exists

Add OR condition to the DUPLICATED_PROSECUTION guard in receiveSjpProsecution()
and receiveCCCase() so that any new case or summons submission is rejected when
applicationIdToDefendantIdsMap is non-empty (i.e. a pending summons already claims
the same URN). Four parameterised unit tests added following TDD red-green cycle;
full suite: 505 tests, 0 failures.

Constitution Principles touched:
  II  (CQRS Three-Layer Discipline): command side only — no new event emitted,
       listener and processor layers confirmed unaffected.
  VIII (Test-Driven Development): failing tests written and confirmed failing before
       production guard conditions were added.

Principles confirmed unaffected:
  I   (Contract First): no new RAML or JSON schema changes.
  III (CPP Framework Idioms): aggregate-only change, existing patterns preserved.
  VI  (Schema-Subscription Symmetry): no new event, no subscription change.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@cpp-github-management
Copy link
Copy Markdown

Passed

@mkarakaya55 mkarakaya55 enabled auto-merge May 21, 2026 13:16
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.

1 participant