fix: test

rilldata · Feb 4, 2025 · bc95a36 · bc95a36
1 parent 4d38d00
commit bc95a36
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 16 deletions.
diff --git a/web-local/tests/explores/explores.spec.ts b/web-local/tests/explores/explores.spec.ts
@@ -74,9 +74,6 @@ test.describe("explores", () => {
       await page.getByRole("menuitem", { name: "All Time" }).click();
     });
 
-    // Turn on comparison
-    await page.getByRole("button", { name: "Comparing" }).click();
-
     // Check the total records are 100k
     await expect(page.getByText("Total records 100k")).toBeVisible();
 

diff --git a/web-local/tests/explores/leaderboard-and-dim-table-sort.spec.ts b/web-local/tests/explores/leaderboard-and-dim-table-sort.spec.ts
@@ -19,6 +19,17 @@ test.describe("leaderboard and dimension table sorting", () => {
   test("leaderboard and dimension table sorting", async ({ page }) => {
     await page.waitForTimeout(1000);
 
+    const timeRangeMenu = page.getByRole("button", {
+      name: "Select time range",
+    });
+
+    async function openTimeRangeMenu() {
+      await timeRangeMenu.click();
+      await page
+        .getByRole("menu", { name: "Select time range" })
+        .waitFor({ state: "visible" });
+    }
+
     await openTimeRangeMenu();
     await page.getByRole("menuitem", { name: "All Time" }).click();
 
@@ -40,16 +51,6 @@ test.describe("leaderboard and dimension table sorting", () => {
       page.getByRole("row", { name: "null 32.9k" }),
     );
 
-    const timeRangeMenu = page.getByRole("button", {
-      name: "Select time range",
-    });
-
-    async function openTimeRangeMenu() {
-      await timeRangeMenu.click();
-      await page
-        .getByRole("menu", { name: "Select time range" })
-        .waitFor({ state: "visible" });
-    }
 
     await assertAAboveB(
       page.getByRole("row", { name: "Microsoft 10.4k" }),

diff --git a/web-local/tests/utils/dataSpecifcHelpers.ts b/web-local/tests/utils/dataSpecifcHelpers.ts
@@ -45,11 +45,11 @@ export async function assertAdBidsDashboard(page: Page) {
         values: [
           "facebook.com",
           "msn.com",
-          "google.com",
+          "news.google.com",
+          "sports.yahoo.com",
           "news.yahoo.com",
+          "google.com",
           "instagram.com",
-          "sports.yahoo.com",
-          "news.google.com",
         ],
       },
     ]),