test: fix flaky drawer open and column sort timing issues (#15677)

### What

Fixed three flaky list view e2e tests: "should persist per-page limit in
list drawer", "should sort", and "should sort without resetting column
preferences".


<img width="1200" height="224" alt="Screenshot 2026-02-18 at 2 47 24 PM"
src="https://github.com/user-attachments/assets/ac4e2147-8925-4626-ab1f-8e33ab6c1a3e"
/>

### Why

The `drawer--is-open` and `sort-column--active` classes weren't applied
immediately after clicking buttons. Tests were checking visibility
before React finished re-rendering, causing intermittent timeouts in CI.

### How

Added explicit waits for drawer and sort button states before running
assertions. The drawer now waits for `state: 'visible'` after clicking
"Select posts", and the sortColumn helper waits for the active state on
sort buttons before checking the URL.

Author: PatrikKozak

test/__helpers/e2e/columns/sortColumn.ts

@@ -25,14 +25,18 @@ export const sortColumn = async (
   const downChevron = columnHeading.locator('button.sort-column__desc')
 
   if (options.targetState === 'asc') {
+    await upChevron.waitFor({ state: 'visible' })
     await upChevron.click()
-    await expect(columnHeading.locator('button.sort-column__asc.sort-column--active')).toBeVisible()
+    const activeButton = columnHeading.locator('button.sort-column__asc.sort-column--active')
+    await activeButton.waitFor({ state: 'visible' })
+    await expect(activeButton).toBeVisible()
     await page.waitForURL(() => page.url().includes(`sort=${options.fieldPath}`))
   } else if (options.targetState === 'desc') {
+    await downChevron.waitFor({ state: 'visible' })
     await downChevron.click()
-    await expect(
-      columnHeading.locator('button.sort-column__desc.sort-column--active'),
-    ).toBeVisible()
+    const activeButton = columnHeading.locator('button.sort-column__desc.sort-column--active')
+    await activeButton.waitFor({ state: 'visible' })
+    await expect(activeButton).toBeVisible()
     await page.waitForURL(() => page.url().includes(`sort=-${options.fieldPath}`))
   }
 }

test/__helpers/e2e/goToNextPage.ts

@@ -15,7 +15,9 @@ export const goToNextPage = async (
   } = { targetPage: 2, affectsURL: true },
 ) => {
   const pageControls = (options.scope || page).locator('.paginator')
-  await pageControls.locator('button').nth(1).click()
+  const nextButton = pageControls.locator('button').nth(1)
+  await nextButton.waitFor({ state: 'visible' })
+  await nextButton.click()
 
   if (options.affectsURL) {
     const regex = new RegExp(`page=${options.targetPage}(?:&|$)`)
@@ -40,7 +42,9 @@ export const goToPreviousPage = async (
   },
 ) => {
   const pageControls = (options.scope || page).locator('.paginator')
-  await pageControls.locator('button').nth(0).click()
+  const prevButton = pageControls.locator('button').nth(0)
+  await prevButton.waitFor({ state: 'visible' })
+  await prevButton.click()
 
   if (options.affectsURL) {
     const regex = new RegExp(`page=${options.targetPage}(?:&|$)`)

test/admin/e2e/list-view/e2e.spec.ts

@@ -1628,9 +1628,12 @@ describe('List View', () => {
       await page.goto(withListViewUrl.list)
 
       // Open the list drawer via the "Select posts" button
-      await page.locator('button:has-text("Select posts")').click()
+      const selectButton = page.locator('button:has-text("Select posts")')
+      await selectButton.waitFor({ state: 'visible' })
+      await selectButton.click()
 
       const listDrawer = page.locator('.list-drawer.drawer--is-open')
+      await listDrawer.waitFor({ state: 'visible' })
       await expect(listDrawer).toBeVisible()
 
       await expect(page.locator('.list-drawer .per-page')).toContainText('Per Page: 10')
@@ -1647,7 +1650,9 @@ describe('List View', () => {
       await expect(listDrawer).toBeHidden()
 
       // Reopen the drawer
-      await page.locator('button:has-text("Select posts")').click()
+      await selectButton.waitFor({ state: 'visible' })
+      await selectButton.click()
+      await listDrawer.waitFor({ state: 'visible' })
       await expect(listDrawer).toBeVisible()
 
       await expect(page.locator('.list-drawer .per-page')).toContainText('Per Page: 5')
@@ -1670,6 +1675,7 @@ describe('List View', () => {
 
     test('should sort', async () => {
       await page.reload()
+      await expect(page.locator(tableRowLocator)).toHaveCount(2)
 
       await sortColumn(page, { fieldPath: 'number', targetState: 'asc' })
 
@@ -1783,9 +1789,15 @@ describe('List View', () => {
       })
 
       await page.goto(postsUrl.list)
+      await expect(page.locator(tableRowLocator).first()).toBeVisible()
 
       // sort by title
-      await page.locator('#heading-title button.sort-column__asc').click()
+      const sortButton = page.locator('#heading-title button.sort-column__asc')
+      await sortButton.waitFor({ state: 'visible' })
+      await sortButton.click()
+      await page
+        .locator('#heading-title button.sort-column__asc.sort-column--active')
+        .waitFor({ state: 'visible' })
       await page.waitForURL(/sort=title/)
 
       // enable a column that is _not_ part of this collection's default columns