CCM-10483: break out source code

Author: harrim91

package.json

@@ -42,7 +42,7 @@
     "lint:fix": "npm run lint:fix --workspaces",
     "start": "npm run start --workspace frontend",
     "test:contracts:clean": "npm --workspace=tests/contracts run pact:clean",
-    "test:contracts:consumers": "npm --workspace=tests/contracts run test:consumers",
+    "test:contracts:consumer": "npm --workspace=tests/contracts run test:consumer",
     "test:contracts:download:consumer": "npm --workspace=tests/contracts run pact:download:consumer",
     "test:contracts:provider": "npm --workspace=tests/contracts run test:provider",
     "test:contracts:provider:ci": "npm --workspace=tests/contracts run test:provider:ci",

tests/contracts/package.json

@@ -15,7 +15,7 @@
     "pact:clean": "./scripts/clean.sh",
     "pact:download:consumer": "./scripts/download-consumer-pacts.sh",
     "pact:upload:consumer": "./scripts/upload-consumer-pacts.sh",
-    "test:consumers": "jest consumer.pact.test.ts",
+    "test:consumer": "jest consumer.pact.test.ts",
     "test:provider": "jest provider.pact.test.ts",
     "test:provider:ci": "./scripts/ci-verify-provider.sh",
     "test:unit": "echo 'No unit tests required'",

tests/contracts/src/auth/events/user-created.event.ts

@@ -0,0 +1,14 @@
+import { randomUUID } from 'node:crypto';
+
+export function createUserCreatedEvent() {
+  return {
+    'detail-type': 'UserCreated',
+    source: 'uk.nhs.notify.auth',
+    time: new Date().toISOString(),
+    version: '1.0',
+    detail: {
+      clientId: randomUUID(),
+      userId: randomUUID(),
+    },
+  };
+}

tests/contracts/src/auth/handlers/template-deleted.handler.ts

@@ -0,0 +1,16 @@
+import { z } from 'zod';
+
+export const $TemplateDeletedEvent = z.object({
+  'detail-type': z.literal('TemplateDeleted'),
+  version: z.literal('1.0'),
+  detail: z.object({
+    id: z.string().uuid(),
+    owner: z.string().uuid(),
+  }),
+});
+
+export async function handleTemplateDeleted(event: unknown): Promise<void> {
+  $TemplateDeletedEvent.parse(event);
+
+  // Handler logic goes here
+}

tests/contracts/src/core/handlers/template-deleted.handler.ts

@@ -0,0 +1,20 @@
+import { z } from 'zod';
+
+// Handlers should define their own minimal input validators
+// Decoupled from implementations in other services
+// So this is slightly different to what's defined in the templates service which produces the event
+// And different to the TemplateDeleted handler in the auth service
+
+export const $TemplateDeletedEvent = z.object({
+  'detail-type': z.literal('TemplateDeleted'),
+  detail: z.object({
+    id: z.string().uuid(),
+    owner: z.string().uuid(),
+  }),
+});
+
+export async function handleTemplateDeleted(event: unknown): Promise<void> {
+  $TemplateDeletedEvent.parse(event);
+
+  // Handler logic goes here
+}

tests/contracts/src/templates/events/template-deleted.event.ts

@@ -0,0 +1,14 @@
+import { randomUUID } from 'node:crypto';
+
+export function createTemplateDeletedEvent() {
+  return {
+    'detail-type': 'TemplateDeleted',
+    source: 'uk.nhs.notify.templates',
+    time: new Date().toISOString(),
+    version: '1.0',
+    detail: {
+      owner: randomUUID(),
+      id: randomUUID(),
+    },
+  };
+}

tests/contracts/src/templates/handlers/user-created.handler.ts

@@ -0,0 +1,17 @@
+import { z } from 'zod';
+
+// I guess this would be defined in the handler source code in the event consumer and imported here
+// Handlers should parse the incoming event before doing anything else
+export const $UserCreatedEvent = z.object({
+  'detail-type': z.literal('UserCreated'),
+  detail: z.object({
+    userId: z.string().uuid(),
+    clientId: z.string().uuid(),
+  }),
+});
+
+export async function handleUserCreatedEvent(event: unknown): Promise<void> {
+  $UserCreatedEvent.parse(event);
+
+  // Handler logic goes here
+}

tests/contracts/tests/auth/consumer/template-deleted.consumer.pact.test.ts

@@ -4,20 +4,9 @@ import {
   Matchers,
   asynchronousBodyHandler,
 } from '@pact-foundation/pact';
-import { z } from 'zod';
+import { $TemplateDeletedEvent } from '../../../src/auth/handlers/template-deleted.handler';
 
-// I guess this would be defined in the handler source code in the event consumer and imported here
-// Handlers should parse the incoming event before doing anything else
-const $TemplateDeletedEvent = z.object({
-  'detail-type': z.literal('TemplateDeleted'),
-  version: z.literal('1.0'),
-  detail: z.object({
-    id: z.string().uuid(),
-    owner: z.string().uuid(),
-  }),
-});
-
-// Simulate consumer handler that processes the incoming event
+// Stub of handler that processes the incoming event
 // Only check the validation - don't run actual handler logic
 async function handleTemplateDeleted(event: unknown): Promise<void> {
   $TemplateDeletedEvent.parse(event);
@@ -34,7 +23,7 @@ describe('Pact Message Consumer - TemplateDeleted Event', () => {
 
   it('should validate the template deleted event structure and handler logic', async () => {
     await messagePact
-      .given('A template has been deleted')
+      .given('a template has been deleted')
       .expectsToReceive('TemplateDeleted')
       .withContent({
         'detail-type': 'TemplateDeleted',

tests/contracts/tests/auth/provider/user-created.provider.pact.test.ts

@@ -1,20 +1,7 @@
 import fs from 'node:fs';
 import path from 'node:path';
 import { MessageProviderPact } from '@pact-foundation/pact';
-
-// This would be the actual function that produces the event payload in the producer source code
-function createUserCreatedEvent() {
-  return {
-    'detail-type': 'UserCreated',
-    source: 'uk.nhs.notify.auth',
-    time: new Date().toISOString(),
-    version: '1.0',
-    detail: {
-      clientId: '3aef04e4-4491-42ae-b24a-95c66b80cbc9',
-      userId: 'f58efb29-27a0-4da3-a502-ddd387771c1e',
-    },
-  };
-}
+import { createUserCreatedEvent } from '../../../src/auth/events/user-created.event';
 
 describe('Pact Message Provider - UserCreated Event', () => {
   const pactDir = path.resolve(__dirname, 'pacts');

tests/contracts/tests/core/consumer/template-deleted.consumer.pact.test.ts

@@ -4,19 +4,9 @@ import {
   Matchers,
   asynchronousBodyHandler,
 } from '@pact-foundation/pact';
-import { z } from 'zod';
+import { $TemplateDeletedEvent } from '../../../src/core/handlers/template-deleted.handler';
 
-// I guess this would be defined in the handler source code in the event consumer and imported here
-// Handlers should parse the incoming event before doing anything else
-const $TemplateDeletedEvent = z.object({
-  'detail-type': z.literal('TemplateDeleted'),
-  detail: z.object({
-    id: z.string().uuid(),
-    owner: z.string().uuid(),
-  }),
-});
-
-// Simulate consumer handler that processes the incoming event
+// Stub of handler that processes the incoming event
 // Only check the validation - don't run actual handler logic
 async function handleTemplateDeleted(event: unknown): Promise<void> {
   $TemplateDeletedEvent.parse(event);
@@ -33,7 +23,7 @@ describe('Pact Message Consumer - TemplateDeleted Event', () => {
 
   it('should validate the template deleted event structure and handler logic', async () => {
     await messagePact
-      .given('A template has been deleted')
+      .given('a template has been deleted')
       .expectsToReceive('TemplateDeleted')
       .withContent({
         'detail-type': 'TemplateDeleted',