Call page.goto in e2e test with waitUntil: commit better offline support

e2e/tests/dashboard/behaviours.spec.ts

@@ -104,7 +104,7 @@ test('goals breakdown', async ({ page, request }) => {
     scrollPercentage: 75
   })
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   const goalsTabButton = tabButton(report, 'Goals')
 
@@ -186,7 +186,9 @@ test('goals breakdown', async ({ page, request }) => {
   })
 
   await test.step('listing goals without revenue', async () => {
-    await page.goto('/' + domain + '?f=has_not_done,goal,purchase')
+    await page.goto('/' + domain + '?f=has_not_done,goal,purchase', {
+      waitUntil: 'commit'
+    })
 
     await goalsTabButton.scrollIntoViewIfNeeded()
     await expect(goalsTabButton).toHaveAttribute('data-active', 'true')
@@ -286,7 +288,7 @@ test('props breakdown', async ({ page, request }) => {
 
   await addAllCustomProps({ page, domain })
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   const propsTabButton = tabButton(report, 'Properties')
 
@@ -427,7 +429,7 @@ test('funnels', async ({ page, request }) => {
     })
   }
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   const funnelsTabButton = tabButton(report, 'Funnels')
 

e2e/tests/dashboard/breakdowns.spec.ts

@@ -48,7 +48,7 @@ test('sources breakdown', async ({ page, request }) => {
     ]
   })
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   const report = page.getByTestId('report-sources')
 
@@ -409,7 +409,7 @@ test('pages breakdown', async ({ page, request }) => {
     ]
   })
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   const report = page.getByTestId('report-pages')
 
@@ -523,7 +523,7 @@ test('pages breakdown modal', async ({ page, request }) => {
     events: pageEvents
   })
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   const report = page.getByTestId('report-pages')
 
@@ -683,7 +683,9 @@ test('pages breakdown with a pageview goal filter applied', async ({
   const report = page.getByTestId('report-pages')
 
   await test.step('custom goal filter applied', async () => {
-    await page.goto('/' + domain + '?f=is,goal,create_site')
+    await page.goto('/' + domain + '?f=is,goal,create_site', {
+      waitUntil: 'commit'
+    })
 
     const pagesTabButton = tabButton(report, 'Conversion pages')
     await pagesTabButton.scrollIntoViewIfNeeded()
@@ -718,7 +720,9 @@ test('pages breakdown with a pageview goal filter applied', async ({
   })
 
   await test.step('revenue goal filter applied', async () => {
-    await page.goto('/' + domain + '?f=is,goal,purchase')
+    await page.goto('/' + domain + '?f=is,goal,purchase', {
+      waitUntil: 'commit'
+    })
 
     const pagesTabButton = tabButton(report, 'Conversion pages')
     await pagesTabButton.scrollIntoViewIfNeeded()
@@ -789,7 +793,7 @@ test('locations breakdown', async ({ page, request }) => {
     ]
   })
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   const report = page.getByTestId('report-locations')
 
@@ -950,7 +954,7 @@ test('devices breakdown', async ({ page, request }) => {
     ]
   })
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   const report = page.getByTestId('report-devices')
 

e2e/tests/dashboard/filtering.spec.ts

@@ -30,7 +30,7 @@ test.describe('page filtering tests', () => {
       ]
     })
 
-    await page.goto('/' + domain)
+    await page.goto('/' + domain, { waitUntil: 'commit' })
 
     const pageFilterRow = filterRow(page, 'page')
     const pageInput = page.getByPlaceholder('Select a Page')
@@ -84,7 +84,7 @@ test.describe('page filtering tests', () => {
       ]
     })
 
-    await page.goto('/' + domain)
+    await page.goto('/' + domain, { waitUntil: 'commit' })
 
     const pageFilterRow = filterRow(page, 'page')
     const pageInput = page.getByPlaceholder('Select a Page')
@@ -199,7 +199,7 @@ test.describe('page filtering tests', () => {
       ]
     })
 
-    await page.goto('/' + domain)
+    await page.goto('/' + domain, { waitUntil: 'commit' })
 
     const entryPageFilterRow = filterRow(page, 'entry_page')
     const entryPageInput = page.getByPlaceholder('Select an Entry Page')
@@ -236,7 +236,7 @@ test.describe('page filtering tests', () => {
       ]
     })
 
-    await page.goto('/' + domain)
+    await page.goto('/' + domain, { waitUntil: 'commit' })
 
     const exitPageFilterRow = filterRow(page, 'exit_page')
     const exitPageInput = page.getByPlaceholder('Select an Exit Page')
@@ -273,7 +273,7 @@ test.describe('hostname filtering tests', () => {
       ]
     })
 
-    await page.goto('/' + domain)
+    await page.goto('/' + domain, { waitUntil: 'commit' })
 
     const hostnameFilterRow = filterRow(page, 'hostname')
     const hostnameInput = page.getByPlaceholder('Select a Hostname')
@@ -310,7 +310,7 @@ test.describe('acquisition filtering tests', () => {
       ]
     })
 
-    await page.goto('/' + domain)
+    await page.goto('/' + domain, { waitUntil: 'commit' })
 
     await test.step('filtering by source', async () => {
       const sourceFilterRow = filterRow(page, 'source')
@@ -415,7 +415,7 @@ test.describe('acquisition filtering tests', () => {
       ]
     })
 
-    await page.goto('/' + domain)
+    await page.goto('/' + domain, { waitUntil: 'commit' })
 
     await test.step('filtering by UTM medium', async () => {
       const utmMediumFilterRow = filterRow(page, 'utm_medium')
@@ -574,7 +574,7 @@ test.describe('location filtering tests', () => {
       ]
     })
 
-    await page.goto('/' + domain)
+    await page.goto('/' + domain, { waitUntil: 'commit' })
 
     await test.step('filtering by country', async () => {
       const countryFilterRow = filterRow(page, 'country')
@@ -658,7 +658,7 @@ test.describe('screen size filtering tests', () => {
       ]
     })
 
-    await page.goto('/' + domain)
+    await page.goto('/' + domain, { waitUntil: 'commit' })
 
     const screenSizeFilterRow = filterRow(page, 'screen')
     const screenSizeInput = page.getByPlaceholder('Select a Screen size')
@@ -698,7 +698,7 @@ test.describe('browser filtering tests', () => {
       ]
     })
 
-    await page.goto('/' + domain)
+    await page.goto('/' + domain, { waitUntil: 'commit' })
 
     await test.step('filtering by browser type', async () => {
       const browserFilterRow = filterRow(page, 'browser')
@@ -769,7 +769,7 @@ test.describe('operating system filtering tests', () => {
       ]
     })
 
-    await page.goto('/' + domain)
+    await page.goto('/' + domain, { waitUntil: 'commit' })
 
     await test.step('filtering by operating system type', async () => {
       const operatingSystemFilterRow = filterRow(page, 'os')
@@ -840,7 +840,7 @@ test.describe('goal filtering tests', () => {
     await addPageviewGoal({ page, domain, pathname: '/page1' })
     await addPageviewGoal({ page, domain, pathname: '/page2' })
 
-    await page.goto('/' + domain)
+    await page.goto('/' + domain, { waitUntil: 'commit' })
 
     const goalFilterRow = filterRow(page, 'goal')
     const goalInput = goalFilterRow.getByPlaceholder('Select a Goal')
@@ -915,7 +915,7 @@ test.describe('property filtering tests', () => {
       ]
     })
 
-    await page.goto('/' + domain)
+    await page.goto('/' + domain, { waitUntil: 'commit' })
 
     const propFilterRow = filterRow(page, 'props')
     const propNameInput = propFilterRow.getByPlaceholder('Property')

e2e/tests/dashboard/general.spec.ts

@@ -11,7 +11,7 @@ test('dashboard renders for logged in user', async ({ page, request }) => {
   const { domain } = await setupSite({ page, request })
   await populateStats({ request, domain, events: [{ name: 'pageview' }] })
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   await expect(page).toHaveTitle(/Plausible/)
 
@@ -24,7 +24,7 @@ test('dashboard renders for anonymous viewer', async ({ page, request }) => {
   await populateStats({ request, domain, events: [{ name: 'pageview' }] })
   await logout(page)
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   await expect(page).toHaveTitle(/Plausible/)
 
@@ -44,15 +44,15 @@ test('dashboard renders via shared link', async ({ page, request }) => {
   await logout(page)
 
   await test.step('public link', async () => {
-    await page.goto(link)
+    await page.goto(link, { waitUntil: 'commit' })
 
     await expect(page.getByRole('button', { name: domain })).toBeVisible()
 
     await expect(page.locator('#visitors')).toHaveText('1')
   })
 
   await test.step('password protected link', async () => {
-    await page.goto(passwordLink)
+    await page.goto(passwordLink, { waitUntil: 'commit' })
 
     await page.locator('input#password').fill('secret')
 
@@ -81,7 +81,7 @@ test('dashboard renders with imported data', async ({ page, request }) => {
     ]
   })
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   await test.step('with imported data included', async () => {
     await expect(page.locator('#visitors')).toHaveText('4')
@@ -109,11 +109,11 @@ test('tab selection user preferences are preserved across reloads', async ({
   const { domain } = await setupSite({ page, request })
   await populateStats({ request, domain, events: [{ name: 'pageview' }] })
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   await page.getByRole('button', { name: 'Entry pages' }).click()
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   let currentTab = await page.evaluate(
     (domain) => localStorage.getItem('pageTab__' + domain),
@@ -124,7 +124,7 @@ test('tab selection user preferences are preserved across reloads', async ({
 
   await page.getByRole('button', { name: 'Exit pages' }).click()
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   currentTab = await page.evaluate(
     (domain) => localStorage.getItem('pageTab__' + domain),
@@ -142,7 +142,7 @@ test('back navigation closes the modal', async ({ page, request, baseURL }) => {
     events: [{ name: 'pageview' }]
   })
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   await page.getByRole('button', { name: 'Filter' }).click()
 

e2e/tests/dashboard/segments.spec.ts

@@ -83,7 +83,7 @@ const createPersonalSegment = async (page: Page, name: string) => {
 test('saving a segment', async ({ page, request }) => {
   const { domain } = await setupSiteAndStats({ page, request })
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   await test.step('creating personal segment using defaults', async () => {
     await addSourceFilter(page, 'Facebook')
@@ -220,7 +220,7 @@ test('creating a segment from a combination of segment and a filter is not allow
 }) => {
   const { domain } = await setupSiteAndStats({ page, request })
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   await addSourceFilter(page, 'Google')
   await createPersonalSegment(page, 'Traffic from Google')
@@ -249,7 +249,7 @@ test('creating a segment from a combination of segment and a filter is not allow
 test('editing an existing segment', async ({ page, request }) => {
   const { domain } = await setupSiteAndStats({ page, request })
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   await addSourceFilter(page, 'Google')
   await createPersonalSegment(page, 'Traffic from Google')
@@ -294,7 +294,7 @@ test('editing an existing segment', async ({ page, request }) => {
 test('saving edited segment as new', async ({ page, request }) => {
   const { domain } = await setupSiteAndStats({ page, request })
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   await addSourceFilter(page, 'Google')
   await createPersonalSegment(page, 'Traffic from Google')
@@ -348,7 +348,7 @@ test('saving edited segment as new', async ({ page, request }) => {
 test('deleting segment', async ({ page, request }) => {
   const { domain } = await setupSiteAndStats({ page, request })
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   await addSourceFilter(page, 'Google')
   await createPersonalSegment(page, 'Traffic from Google')
@@ -377,7 +377,7 @@ test('deleting segment', async ({ page, request }) => {
 test('closing edited segment without saving', async ({ page, request }) => {
   const { domain } = await setupSiteAndStats({ page, request })
 
-  await page.goto('/' + domain)
+  await page.goto('/' + domain, { waitUntil: 'commit' })
 
   await addSourceFilter(page, 'Google')
   await createPersonalSegment(page, 'Traffic from Google')

e2e/tests/dashboard/team-setup.spec.ts

@@ -7,7 +7,7 @@ test('submitting team name via Enter key does not crash', async ({
   request
 }) => {
   await setupSite({ page, request })
-  await page.goto('/team/setup')
+  await page.goto('/team/setup', { waitUntil: 'commit' })
 
   await expectLiveViewConnected(page)