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/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);
   });

tests/helpers/validate-json-schema.ts

@@ -19,7 +19,7 @@ export default async function validateApiResponse(
   method: string,
   pathVar: string,
   status: number,
-  body: any
+  body: any,
 ): Promise<ValidationResult> {
   const openapiDoc = await import(paths);
 
@@ -36,7 +36,7 @@ export default async function validateApiResponse(
     operation.responses[status] || operation.responses.default;
   if (!responseSchema) {
     throw new Error(
-      `No schema defined for status ${status} at ${method.toUpperCase()} ${pathVar}`
+      `No schema defined for status ${status} at ${method.toUpperCase()} ${pathVar}`,
     );
   }
 

tests/mtls/mtls-test.spec.ts

@@ -1,7 +1,7 @@
-import { test, expect, request, APIRequestContext } from "@playwright/test";
+import { APIRequestContext, expect, request, test } from "@playwright/test";
 
 // Assume you have your constants in a config file
-const PROXY_URL = process.env.PROXY_URL;
+const { PROXY_URL } = process.env;
 
 test("should fail when connecting without client certificate", async () => {
   let apiContext: APIRequestContext | null = null;
@@ -15,15 +15,15 @@ test("should fail when connecting without client certificate", async () => {
     // Check if request succeeded or failed
     if (response.ok()) {
       throw new Error(
-        `Expected connection failure, but got success with status ${response.status()}`
+        `Expected connection failure, but got success with status ${response.status()}`,
       );
     }
     // Assert on the actual error code returned by the gateway
     // For mTLS, often 401, 403, or 502 depending on infra config
     expect(response.ok()).toBeFalsy();
-  } catch (err: any) {
+  } catch (error: any) {
     // If the request truly fails at the TLS layer, Playwright will throw instead
-    expect(err.message).toMatch(/SSL|certificate|ECONNRESET|socket/i);
+    expect(error.message).toMatch(/SSL|certificate|ECONNRESET|socket/i);
   } finally {
     if (apiContext) {
       await apiContext.dispose();