test auto fixes

Author: nhsd-david-wass

tests/component-tests/apiGateway-tests/get-letter-status.spec.ts

@@ -30,7 +30,7 @@ test.describe("API Gateway Tests to Verify Get Letter Status Endpoint", () => {
       `${baseUrl}/${SUPPLIER_LETTERS}/${letter.id}`,
       {
         headers,
-      }
+      },
     );
 
     const responseBody = await response.json();

tests/component-tests/apiGateway-tests/testCases/create-mi.ts

@@ -1,70 +1,70 @@
-
-
-
 export type MiRequestBody = {
   data: {
     type: string;
     attributes: {
-        groupId: string;
-        lineItem: string;
-        quantity: number;
-        specificationId: string;
-        stockRemaining: number;
-        timestamp: string;
+      groupId: string;
+      lineItem: string;
+      quantity: number;
+      specificationId: string;
+      stockRemaining: number;
+      timestamp: string;
     };
   };
 };
 
-export function miValidRequest() : MiRequestBody{
+export function miValidRequest(): MiRequestBody {
   let requestBody: MiRequestBody;
 
   requestBody = {
     data: {
-       attributes: {
-        groupId: 'group123',
-        lineItem: 'envelope-business-standard',
+      attributes: {
+        groupId: "group123",
+        lineItem: "envelope-business-standard",
         quantity: 10,
-        specificationId: 'Test-Spec-Id',
+        specificationId: "Test-Spec-Id",
         stockRemaining: 100,
         timestamp: new Date().toISOString(),
       },
-      type: 'ManagementInformation',
-  }};
+      type: "ManagementInformation",
+    },
+  };
   return requestBody;
 }
 
-export function miInvalidRequest() : MiRequestBody{
+export function miInvalidRequest(): MiRequestBody {
   let requestBody: MiRequestBody;
 
   requestBody = {
     data: {
-       attributes: {
-        groupId: 'group123',
-        lineItem: 'envelope-business-standard',
+      attributes: {
+        groupId: "group123",
+        lineItem: "envelope-business-standard",
         quantity: 10,
-        specificationId: 'Test-Spec-Id',
+        specificationId: "Test-Spec-Id",
         stockRemaining: 100,
         timestamp: new Date().toISOString(),
       },
-      type: '?',
-  }};
+      type: "?",
+    },
+  };
   return requestBody;
 }
 
-export function miInvalidDateRequest() : MiRequestBody{
+export function miInvalidDateRequest(): MiRequestBody {
   let requestBody: MiRequestBody;
 
   requestBody = {
     data: {
-       attributes: {
-        groupId: 'group123',
-        lineItem: 'envelope-business-standard',
+      attributes: {
+        groupId: "group123",
+        lineItem: "envelope-business-standard",
         quantity: 10,
-        specificationId: 'Test-Spec-Id',
+        specificationId: "Test-Spec-Id",
         stockRemaining: 100,
-        timestamp: '2021-10-28T',
+        timestamp: "2021-10-28T",
       },
-      type: 'ManagementInformation',
-  }};
+      type: "ManagementInformation",
+    },
+  };
   return requestBody;
 }

tests/component-tests/apiGateway-tests/testCases/update-letter-status.ts

@@ -41,7 +41,7 @@ export function patchRequestHeaders(): RequestHeaders {
 
 export function patchValidRequestBody(
   id: string,
-  status: string
+  status: string,
 ): PatchMessageRequestBody {
   let requestBody: PatchMessageRequestBody;
 
@@ -59,7 +59,7 @@ export function patchValidRequestBody(
 
 export function patchFailureRequestBody(
   id: string,
-  status: string
+  status: string,
 ): PatchMessageRequestBody {
   let requestBody: PatchMessageRequestBody;
 

tests/component-tests/apiGateway-tests/update-letter-status.spec.ts

@@ -45,7 +45,7 @@ test.describe("API Gateway Tests to Verify Patch Status Endpoint", () => {
       {
         headers,
         data: body,
-      }
+      },
     );
 
     expect(response.status()).toBe(202);
@@ -72,7 +72,7 @@ test.describe("API Gateway Tests to Verify Patch Status Endpoint", () => {
       {
         headers,
         data: body,
-      }
+      },
     );
 
     expect(response.status()).toBe(202);
@@ -92,7 +92,7 @@ test.describe("API Gateway Tests to Verify Patch Status Endpoint", () => {
       {
         headers,
         data: body,
-      }
+      },
     );
 
     const responseBody = await response.json();
@@ -113,7 +113,7 @@ test.describe("API Gateway Tests to Verify Patch Status Endpoint", () => {
       {
         headers,
         data: body,
-      }
+      },
     );
 
     const responseBody = await response.json();
@@ -133,7 +133,7 @@ test.describe("API Gateway Tests to Verify Patch Status Endpoint", () => {
       {
         headers,
         data: body,
-      }
+      },
     );
 
     const responseBody = await response.json();

tests/config/reporters.ts

@@ -1,5 +1,5 @@
 import type { ReporterDescription } from "@playwright/test";
-import path from "path";
+import path from "node:path";
 
 const resultsDir = process.env.RESULTS_DIR || "results";
 const reportsDir = process.env.REPORTS_DIR || "reports";

tests/config/sandbox.config.ts

@@ -1,7 +1,7 @@
 import type { PlaywrightTestConfig } from "@playwright/test";
+import path from "node:path";
 import { config as baseConfig } from "./playwright.base.config";
 import { getReporters } from "./reporters";
-import path from "path";
 
 const localConfig: PlaywrightTestConfig = {
   /* Reporter to use. See https://playwright.dev/docs/test-reporters */

tests/helpers/generate-fetch-test-data.ts

@@ -33,7 +33,7 @@ export interface SupplierApiLetters {
 
 export async function createTestData(
   supplierId: string,
-  count?: number
+  count?: number,
 ): Promise<void> {
   await runCreateLetter({
     filter: "nhs-notify-supplier-api-letter-test-data-utility",
@@ -50,7 +50,7 @@ export async function createTestData(
 export const getLettersBySupplier = async (
   supplierId: string,
   status: string,
-  limit: number
+  limit: number,
 ) => {
   const supplierStatus = `${supplierId}#${status}`;
   const params = {
@@ -78,13 +78,13 @@ export const deleteLettersBySupplier = async (id: string) => {
       TableName: LETTERSTABLENAME,
       Key: { supplierId: SUPPLIERID, id },
       ReturnValues: "ALL_OLD",
-    })
+    }),
   );
   return resp.Attributes;
 };
 
 export async function checkSupplierExists(
-  supplierId: string
+  supplierId: string,
 ): Promise<boolean> {
   try {
     const params = {

tests/helpers/generate_fetch_testData.ts

@@ -1,53 +1,63 @@
-import { envName, LETTERSTABLENAME, SUPPLIERID } from "../constants/api_constants";
+import { DynamoDBClient } from "@aws-sdk/client-dynamodb";
+import {
+  DeleteCommand,
+  DynamoDBDocumentClient,
+  QueryCommand,
+} from "@aws-sdk/lib-dynamodb";
+import {
+  LETTERSTABLENAME,
+  SUPPLIERID,
+  envName,
+} from "../constants/api_constants";
 import { runCreateLetter } from "./pnpmHelpers";
-import { DynamoDBClient } from '@aws-sdk/client-dynamodb';
-import { DeleteCommand, DynamoDBDocumentClient, QueryCommand } from '@aws-sdk/lib-dynamodb';
 
 const ddb = new DynamoDBClient({});
 const docClient = DynamoDBDocumentClient.from(ddb);
 
 export interface SupplierApiLetters {
-    supplierId: string,
-    specificationId: string,
-    supplierStatus: string,
-    createdAt: string,
-    supplierStatusSk: string,
-    updatedAt: string,
-    groupId: string,
-    reasonCode: string,
-    id: string,
-    url: string,
-    ttl: string,
-    reasonText: string,
-    status: string,
-    source: string
-};
+  supplierId: string;
+  specificationId: string;
+  supplierStatus: string;
+  createdAt: string;
+  supplierStatusSk: string;
+  updatedAt: string;
+  groupId: string;
+  reasonCode: string;
+  id: string;
+  url: string;
+  ttl: string;
+  reasonText: string;
+  status: string;
+  source: string;
+}
 
 export async function createTestData(supplierId: string): Promise<void> {
-
-    await runCreateLetter({
-    filter: 'nhs-notify-supplier-api-data-generator',
-    supplierId: supplierId,
+  await runCreateLetter({
+    filter: "nhs-notify-supplier-api-data-generator",
+    supplierId,
     environment: envName,
-    awsAccountId: '820178564574',
-    groupId: 'TestGroupID',
-    specificationId: 'TestSpecificationID',
-    status: 'PENDING',
+    awsAccountId: "820178564574",
+    groupId: "TestGroupID",
+    specificationId: "TestSpecificationID",
+    status: "PENDING",
     count: 1,
   });
-};
-
-export const getLettersBySupplier = async(supplierId: string, status: string, limit: number) => {
+}
 
-    const supplierStatus = `${supplierId}#${status}`;
-      const params = {
+export const getLettersBySupplier = async (
+  supplierId: string,
+  status: string,
+  limit: number,
+) => {
+  const supplierStatus = `${supplierId}#${status}`;
+  const params = {
     TableName: LETTERSTABLENAME,
-    IndexName: 'supplierStatus-index',
-    KeyConditionExpression: 'supplierStatus = :supplierStatus',
+    IndexName: "supplierStatus-index",
+    KeyConditionExpression: "supplierStatus = :supplierStatus",
     ProjectionExpression:
-      'id, specificationId, groupId, reasonCode, reasonText',
+      "id, specificationId, groupId, reasonCode, reasonText",
     ExpressionAttributeValues: {
-      ':supplierStatus': supplierStatus,
+      ":supplierStatus": supplierStatus,
     },
     Limit: limit,
   };
@@ -59,13 +69,13 @@ export const getLettersBySupplier = async(supplierId: string, status: string, li
   return Items as SupplierApiLetters[];
 };
 
-export const deleteLettersBySupplier = async(id: string) => {
+export const deleteLettersBySupplier = async (id: string) => {
   const resp = await docClient.send(
     new DeleteCommand({
       TableName: LETTERSTABLENAME,
       Key: { supplierId: SUPPLIERID, id },
-      ReturnValues: 'ALL_OLD',
-    })
-  )
+      ReturnValues: "ALL_OLD",
+    }),
+  );
   return resp.Attributes;
-}
+};

tests/helpers/pnpm-helpers.ts

@@ -29,7 +29,7 @@ export async function runCreateLetter(options: {
 
   const workspaceRoot = path.resolve(
     __dirname,
-    "../../scripts/utilities/letter-test-data"
+    "../../scripts/utilities/letter-test-data",
   );
   const cmd = process.platform === "win32" ? "npm.cmd" : "npm";
   const root = path.resolve(workspaceRoot);
@@ -73,7 +73,7 @@ export async function runCreateLetter(options: {
     });
 
     child.on("close", (code) =>
-      code === 0 ? resolve() : reject(new Error(`pnpm exited with ${code}`))
+      code === 0 ? resolve() : reject(new Error(`pnpm exited with ${code}`)),
     );
     child.on("error", reject);
   });
@@ -87,11 +87,11 @@ export async function createSupplierData(options: {
   environment: string;
   status: string;
 }) {
-  const { apimId, environment, name, status, supplierId, filter } = options;
+  const { apimId, environment, filter, name, status, supplierId } = options;
 
   const workspaceRoot = path.resolve(
     __dirname,
-    "../../scripts/utilities/supplier-data"
+    "../../scripts/utilities/supplier-data",
   );
   const cmd = process.platform === "win32" ? "npm.cmd" : "npm";
   const root = path.resolve(workspaceRoot);
@@ -131,7 +131,7 @@ export async function createSupplierData(options: {
     });
 
     child.on("close", (code) =>
-      code === 0 ? resolve() : reject(new Error(`pnpm exited with ${code}`))
+      code === 0 ? resolve() : reject(new Error(`pnpm exited with ${code}`)),
     );
     child.on("error", reject);
   });