Fix E2E test Filedownload issue

frontend-react/e2e/helpers/utils.ts

@@ -139,3 +139,18 @@ export function toDateWithTime(date: string, time?: string) {
     }
     return toDateTime;
 }
+
+export function removeDateTime(filename: string) {
+    // Example string: "co.yml-beb0c9d9-ca1f-4af3-853e-0aba61541f66-20240829191221.hl7"
+    // Find the last hyphen and the last dot in the string
+    const lastHyphenIndex = filename.lastIndexOf("-");
+    const lastDotIndex = filename.lastIndexOf(".");
+
+    // If both indices are found, implying a properly formatted file extension,
+    // remove the timestamp
+    if (lastHyphenIndex !== -1 && lastDotIndex !== -1 && lastHyphenIndex < lastDotIndex) {
+        return filename.slice(0, lastHyphenIndex);
+    }
+
+    return filename;
+}

frontend-react/e2e/spec/all/authenticated/daily-data-page-user-flow.spec.ts → frontend-react/e2e/spec/chromium-only/authenticated/daily-data-page-user-flow.spec.ts

@@ -4,6 +4,7 @@
 import fs from "node:fs";
 import {
     expectTableColumnValues,
+    removeDateTime,
     tableColumnDateTimeInRange,
     tableDataCellValue,
     tableRows,
@@ -79,10 +80,6 @@
         test.describe("admin user", () => {
             test.use({ storageState: "e2e/.auth/admin.json" });
 
-            test.beforeAll(({ browserName }) => {
-                test.skip(browserName !== "chromium");
-            });
-
             test.describe(`${TEST_ORG_IGNORE} org - ${TEST_ORG_UP_RECEIVER_UP} receiver`, () => {
                 test.describe("onLoad", () => {
                     test("has correct title", async ({ dailyDataPage }) => {
@@ -102,6 +99,7 @@
                     });
 
                     test("table has pagination", async ({ dailyDataPage }) => {
+                        await dailyDataPage.page.locator(".usa-table tbody").waitFor({ state: "visible" });
                         await expect(dailyDataPage.page.locator('[aria-label="Pagination"]')).toBeAttached();
                     });
 
@@ -292,9 +290,11 @@
                         expect(await tableDataCellValue(dailyDataPage.page, 0, 0)).toEqual(reportId);
                     });
 
-                    test("returns match for Filename", async ({ dailyDataPage }) => {
+                    test.skip("returns match for Filename", async ({ dailyDataPage }) => {
+                        // Filename search is currently broken and being tracked
+                        // in ticket #15644
                         const fileName = await tableDataCellValue(dailyDataPage.page, 0, 4);
-                        await searchInput(dailyDataPage.page).fill(fileName);
+                        await searchInput(dailyDataPage.page).fill(removeDateTime(fileName));
                         await searchButton(dailyDataPage.page).click();
                         await dailyDataPage.page.locator(".usa-table tbody").waitFor({ state: "visible" });
 
@@ -365,7 +365,7 @@
                     });
                 });
 
-                test.describe.skip("on 'Filename' click", () => {
+                test.describe("on 'Filename' click", () => {
                     test.beforeEach(async ({ dailyDataPage }) => {
                         await dailyDataPage.page.locator(".usa-table tbody").waitFor({ state: "visible" });
                         await dailyDataPage.page.locator("#receiver-dropdown").selectOption(TEST_ORG_UP_RECEIVER_UP);
@@ -387,7 +387,7 @@
                         const download = await downloadProm;
 
                         // assert filename
-                        expect(download.suggestedFilename()).toEqual(expect.stringContaining(fileName));
+                        expect(removeDateTime(download.suggestedFilename())).toEqual(removeDateTime(fileName));
 
                         // get and assert stats
                         expect((await fs.promises.stat(await download.path())).size).toBeGreaterThan(200);
@@ -414,8 +414,8 @@
                             );
 
                             if (responsePromise) {
                                 await expect(dailyDataPage.page).toHaveURL(`${URL_REPORT_DETAILS}/${reportId}`);
                                 await expect(dailyDataPage.page).toHaveTitle(/Daily Data - ReportStream/);
                                 await expect(dailyDataPage.page.locator("h1").getByText(reportId)).toBeVisible();
 
                                 // Facility table headers

frontend-react/e2e/spec/chromium-only/public-pages-link-check.spec.ts

@@ -55,7 +55,9 @@
         // Set test timeout to be 1 minute instead of 30 seconds
         test.setTimeout(60000);
         for (const path of urlPaths) {
-            await page.goto(path);
+            await page.goto(path, {
+                waitUntil: "networkidle",
+            });
             const baseUrl = new URL(page.url()).origin;
 
             const allATags = await page.getByRole("link", { includeHidden: true }).elementHandles();