Modified tests

Author: varsha-vanbatte-nhs

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

@@ -38,8 +38,44 @@ test.describe("API Gateway Tests to Verify Get Letter PDF Endpoint", () => {
     );
 
     expect(response.status()).toBe(200);
-    const responseBody = await response.text();
-    expect(responseBody).toContain("PDF");
+    // const responseBody = await response.text();
+    // expect(responseBody).toContain("PDF");
+
+    async function fetchAndValidatePdf(url: string) {
+      const res = await request.get(url, { headers });
+      return {
+        status: res.status(),
+        headers: res.headers(),
+        buffer: await res.body().catch(() => null),
+        text: await res.text().catch(() => null),
+      };
+    }
+
+    const pdfUrl = await response.url();
+    const parsed = new URL(pdfUrl);
+    const expiresParam = parsed.searchParams.get("X-Amz-Expires");
+
+    expect(expiresParam).toBeTruthy();
+    const expiresSeconds = Number(expiresParam);
+    expect(Number.isFinite(expiresSeconds)).toBeTruthy();
+
+    // Validate the URL works immediately
+    const url = await fetchAndValidatePdf(pdfUrl);
+    expect(url.status).toBe(200);
+    expect(url.headers["content-type"] || "").toContain("application/pdf");
+
+    const waitMs = Math.max(expiresSeconds * 1000 + 2000, 0);
+    if (waitMs > 0) {
+      // Cap wait to avoid extremely long waits
+      const MAX_WAIT_MS = 120_000; // 2 minutes cap
+      const effectiveWait = Math.min(waitMs, MAX_WAIT_MS);
+      await new Promise((resolve) => {
+        setTimeout(resolve, effectiveWait);
+      });
+
+      const after = await fetchAndValidatePdf(pdfUrl);
+      expect([400, 403]).toContain(after.status);
+    }
   });
 
   test(`Get /letters/{id}/data returns 404 if no resource is found for id`, async ({

tests/config/playwright.base.config.ts

@@ -8,7 +8,7 @@ const envMaxInstances = Number.parseInt(process.env.WORKERS_MAX_INST!) || 10;
  */
 export const config: PlaywrightTestConfig = {
   /* Maximum time one test can run for. */
-  timeout: 60 * 1000,
+  timeout: 80 * 1000,
   workers: envMaxInstances,
   expect: {
     /**