tests

Author: Namitha-Prabhu

tests/constants/api_constants.ts

@@ -1,8 +1,17 @@
 export const SUPPLIER_LETTERS = 'letters';
-export const SUPPLIER_API_URL_SANDBOX = 'https://internal-dev-sandbox.api.service.nhs.uk/nhs-notify-supplier';
 export const AWS_REGION = 'eu-west-2';
 export const envName = process.env.PR_NUMBER ?? 'main';
 export const API_NAME = `nhs-${envName}-supapi`;
 export const LETTERSTABLENAME = `nhs-${envName}-supapi-letters`;
 export const SUPPLIERID = 'TestSupplier1';
 export const MI_ENDPOINT = 'mi';
+
+const formattedPr = process.env.PR_NUMBER
+  ? process.env.PR_NUMBER.replace(/^pr(\d+)$/i, "PR-$1")
+  : null;
+
+// Build sandbox environment name
+const sandboxEnv = formattedPr
+  ? `nhs-notify-supplier-${formattedPr}`
+  : "nhs-notify-supplier";
+export const SUPPLIER_API_URL_SANDBOX = `https://internal-dev-sandbox.api.service.nhs.uk/${sandboxEnv}`;

tests/sandbox/testCases/updateLetterStatus_testCases.ts

@@ -16,8 +16,8 @@ export type ApiSandboxUpdateLetterStatusTestData = {
 
 export const apiSandboxUpdateLetterStatusTestData: ApiSandboxUpdateLetterStatusTestData[] = [
   {
-    testCase: '200 response if record is updated with status PENDING',
-    id: '2WL5eYSWGzCHlGmzNxuqVusPxDg',
+    testCase: '202 response if record is updated with status PENDING',
+    id: '24L5eYSWGzCHlGmzNxuqVusPxDg',
     header: sandBoxHeader,
     body: {
       data: {
@@ -28,47 +28,25 @@ export const apiSandboxUpdateLetterStatusTestData: ApiSandboxUpdateLetterStatusT
         },
       }
     },
-    expectedStatus: 200,
-        expectedResponse: {
-      data: {
-        type: 'Letter',
-        id: '2WL5eYSWGzCHlGmzNxuqVusPxDg',
-        attributes: {
-          status: 'PENDING',
-          specificationId:'2WL5eYSWGzCHlGmzNxuqVusPxDg',
-        },
-      }
-    },
+    expectedStatus: 202,
   },
 
   {
-    testCase: '200 response if record is updated with status REJECTED',
-    id: '2WL5eYSWGzCHlGmzNxuqVusPxDg',
+    testCase: '202 response if record is updated with status REJECTED',
+    id: '24L5eYSWGzCHlGmzNxuqVusPxDg',
     header: sandBoxHeader,
     body: {
       data: {
         type: 'Letter',
         id: '2WL5eYSWGzCHlGmzNxuqVusPxDg',
         attributes: {
           status: 'REJECTED',
-          reasonCode: 'R01',
-          reasonText: 'failed validation',
-        },
-      }
-    },
-    expectedStatus: 200,
-        expectedResponse: {
-      data: {
-        type: 'Letter',
-        id: '2WL5eYSWGzCHlGmzNxuqVusPxDg',
-        attributes: {
-          reasonCode: 'R01',
-          reasonText: 'failed validation',
-          status: 'REJECTED',
-          specificationId:'2WL5eYSWGzCHlGmzNxuqVusPxDg',
+          reasonCode: 'R07',
+          reasonText: 'No such address',
         },
       }
     },
+    expectedStatus: 202,
   },
   {
     testCase: '404 response if no resource is found for the given id',
@@ -112,10 +90,10 @@ export const apiSandboxUpdateLetterStatusTestData: ApiSandboxUpdateLetterStatusT
     },
     expectedStatus: 400,
     expectedResponse: {
-      message: 'request.body.data.attributes.status should be equal to one of the allowed values: PENDING, ACCEPTED, REJECTED, PRINTED, ENCLOSED, CANCELLED, DISPATCHED, DELIVERED, FAILED, RETURNED, DESTROYED, FORWARDED',
+      message: 'request.body.data.attributes.status should be equal to one of the allowed values: PENDING, ACCEPTED, REJECTED, PRINTED, ENCLOSED, CANCELLED, DISPATCHED, DELIVERED, FAILED, RETURNED, FORWARDED',
       errors: [{
           path: '.body.data.attributes.status',
-          message: 'should be equal to one of the allowed values: PENDING, ACCEPTED, REJECTED, PRINTED, ENCLOSED, CANCELLED, DISPATCHED, DELIVERED, FAILED, RETURNED, DESTROYED, FORWARDED',
+          message: 'should be equal to one of the allowed values: PENDING, ACCEPTED, REJECTED, PRINTED, ENCLOSED, CANCELLED, DISPATCHED, DELIVERED, FAILED, RETURNED, FORWARDED',
           errorCode: 'enum.openapi.validation'
       }]
     },

tests/sandbox/testCases/updateMultipleStatus_testCases.ts

@@ -23,7 +23,7 @@ type postRequest = {
 
 export const apiSandboxMultipleLetterStatusTestData: ApiSandboxUpdateLetterStatusTestData[] =
 [{
-    testCase: '200 response if records are updated',
+    testCase: '202 response if records are updated',
     header: sandBoxHeader,
     body:{
         data :
@@ -106,7 +106,7 @@ export const apiSandboxMultipleLetterStatusTestData: ApiSandboxUpdateLetterStatu
             type: 'Letter'
         }
     ]},
-    expectedStatus: 200
+    expectedStatus: 202
 },
 {
     testCase: '404 response if invalid request is passed',

tests/sandbox/updateLetterStatus.spec.ts

@@ -7,16 +7,18 @@ test.describe('Sandbox Tests To Update Letter Status', () =>
 {
   apiSandboxUpdateLetterStatusTestData.forEach(({ testCase, header, id, body, expectedStatus, expectedResponse }) => {
     test(`Patch /Letters endpoint returns ${testCase}`, async ({ request }) => {
-
         const response = await request.patch(`${SUPPLIER_API_URL_SANDBOX}/${SUPPLIER_LETTERS}/${id}` ,{
                 headers: header,
                 data: body
           });
 
-        const res = await response.json();
-        expect(response.status()).toBe(expectedStatus);
-        expect(res).toEqual(expectedResponse);
+        console.log("URL", `${SUPPLIER_API_URL_SANDBOX}/${SUPPLIER_LETTERS}/${id}`);
 
+        expect(response.status()).toBe(expectedStatus);
+        if (response.status() != 202){
+          const res = await response.json();
+          expect(res).toEqual(expectedResponse);
+        }
     });
   });
 });