test: use retryable assertion for scrollY (#26298)

nuxt · Mar 16, 2024 · c7d0567 · c7d0567
1 parent 2007990
commit c7d0567
Showing 1 changed file with 47 additions and 55 deletions.
diff --git a/test/basic.test.ts b/test/basic.test.ts
@@ -717,61 +717,53 @@ describe('nuxt links', () => {
     await page.close()
   })
 
-  it('expect scroll to top on routes with same component',
-    async () => {
-      // #22402
-      const page = await createPage('/big-page-1', {
-        viewport: {
-          width: 1000,
-          height: 1000
-        }
-      })
-      await page.waitForFunction(() => window.useNuxtApp?.()._route.fullPath === '/big-page-1')
-
-      await page.locator('#big-page-2').scrollIntoViewIfNeeded()
-      expect(await page.evaluate(() => window.scrollY)).toBeGreaterThan(0)
-      await page.locator('#big-page-2').click()
-      await page.waitForFunction(path => window.useNuxtApp?.()._route.fullPath === path, '/big-page-2')
-      expect(await page.evaluate(() => window.scrollY)).toBe(0)
-
-      await page.locator('#big-page-1').scrollIntoViewIfNeeded()
-      expect(await page.evaluate(() => window.scrollY)).toBeGreaterThan(0)
-      await page.locator('#big-page-1').click()
-      await page.waitForFunction(path => window.useNuxtApp?.()._route.fullPath === path, '/big-page-1')
-      expect(await page.evaluate(() => window.scrollY)).toBe(0)
-      await page.close()
-    },
-    // Flaky behavior when using Webpack
-    { retry: isWebpack ? 10 : 0 }
-  )
-
-  it('expect scroll to top on nested pages',
-    async () => {
-      // #20523
-      const page = await createPage('/nested/foo/test', {
-        viewport: {
-          width: 1000,
-          height: 1000
-        }
-      })
-      await page.waitForFunction(path => window.useNuxtApp?.()._route.fullPath === path, '/nested/foo/test')
-
-      await page.locator('#user-test').scrollIntoViewIfNeeded()
-      expect(await page.evaluate(() => window.scrollY)).toBeGreaterThan(0)
-      await page.locator('#user-test').click()
-      await page.waitForFunction(path => window.useNuxtApp?.()._route.fullPath === path, '/nested/foo/user-test')
-      expect(await page.evaluate(() => window.scrollY)).toBe(0)
-
-      await page.locator('#test').scrollIntoViewIfNeeded()
-      expect(await page.evaluate(() => window.scrollY)).toBeGreaterThan(0)
-      await page.locator('#test').click()
-      await page.waitForFunction(path => window.useNuxtApp?.()._route.fullPath === path, '/nested/foo/test')
-      expect(await page.evaluate(() => window.scrollY)).toBe(0)
-      await page.close()
-    },
-    // Flaky behavior when using Webpack
-    { retry: isWebpack ? 10 : 0 }
-  )
+  it('expect scroll to top on routes with same component', async () => {
+    // #22402
+    const page = await createPage('/big-page-1', {
+      viewport: {
+        width: 1000,
+        height: 1000
+      }
+    })
+    await page.waitForFunction(() => window.useNuxtApp?.()._route.fullPath === '/big-page-1')
+
+    await page.locator('#big-page-2').scrollIntoViewIfNeeded()
+    await page.waitForFunction(() => window.scrollY > 0)
+    await page.locator('#big-page-2').click()
+    await page.waitForFunction(path => window.useNuxtApp?.()._route.fullPath === path, '/big-page-2')
+    await page.waitForFunction(() => window.scrollY === 0)
+
+    await page.locator('#big-page-1').scrollIntoViewIfNeeded()
+    await page.waitForFunction(() => window.scrollY > 0)
+    await page.locator('#big-page-1').click()
+    await page.waitForFunction(path => window.useNuxtApp?.()._route.fullPath === path, '/big-page-1')
+    await page.waitForFunction(() => window.scrollY === 0)
+    await page.close()
+  })
+
+  it('expect scroll to top on nested pages', async () => {
+    // #20523
+    const page = await createPage('/nested/foo/test', {
+      viewport: {
+        width: 1000,
+        height: 1000
+      }
+    })
+    await page.waitForFunction(path => window.useNuxtApp?.()._route.fullPath === path, '/nested/foo/test')
+
+    await page.locator('#user-test').scrollIntoViewIfNeeded()
+    expect(await page.evaluate(() => window.scrollY)).toBeGreaterThan(0)
+    await page.locator('#user-test').click()
+    await page.waitForFunction(path => window.useNuxtApp?.()._route.fullPath === path, '/nested/foo/user-test')
+    expect(await page.evaluate(() => window.scrollY)).toBe(0)
+
+    await page.locator('#test').scrollIntoViewIfNeeded()
+    expect(await page.evaluate(() => window.scrollY)).toBeGreaterThan(0)
+    await page.locator('#test').click()
+    await page.waitForFunction(path => window.useNuxtApp?.()._route.fullPath === path, '/nested/foo/test')
+    expect(await page.evaluate(() => window.scrollY)).toBe(0)
+    await page.close()
+  })
 })
 
 describe('head tags', () => {