try to set the right base url

MuckRock · Nov 16, 2023 · 3c4282d · 3c4282d
1 parent 39a7f85
commit 3c4282d
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 8 deletions.
diff --git a/.github/workflows/playwright.yml b/.github/workflows/playwright.yml
@@ -1,11 +1,14 @@
 name: Playwright Tests
 
 on:
+  push:
+    branches: [master]
   pull_request:
     branches: [master]
 
 env:
   URL: "https://deploy-preview-${{ github.event.number }}.muckcloud.com"
+  PLAYWRIGHT_TEST_BASE_URL: "https://deploy-preview-${{ github.event.number }}.muckcloud.com"
 
 jobs:
   wait:
@@ -29,6 +32,7 @@ jobs:
       - uses: actions/setup-node@v3
         with:
           node-version: 18
+
       - name: Install dependencies
         run: npm ci
 

diff --git a/netlify.toml b/netlify.toml
diff --git a/tests/anonymous/viewer/document.spec.js b/tests/anonymous/viewer/document.spec.js
@@ -9,6 +9,8 @@ test("test", async ({ page }) => {
     "/documents/20000007-finalseasonal_allergies_pollen_and_mold_2023__en";
   await page.goto(url);
 
+  expect(new URL(page.url()).pathname).toBe(url);
+
   await expect(page.locator(".sidebar").getByRole("heading")).toHaveText(title);
 
   await page.getByRole("link", { name: "Original Document (PDF) »" }).click();
@@ -17,9 +19,7 @@ test("test", async ({ page }) => {
 
   await page.getByRole("link", { name: "p. 1" }).click();
 
-  const currentUrl = new URL(page.url());
-
-  expect(currentUrl.hash).toEqual("#document/p1");
+  expect(new URL(page.url()).hash).toEqual("#document/p1");
 
   await page
     .locator("div")