Skip to content

test(infinite-scroll): stabilize top scroll test#31049

Open
ManthanBhatt wants to merge 1 commit intoionic-team:mainfrom
ManthanBhatt:fix-infinite-scroll-top-test
Open

test(infinite-scroll): stabilize top scroll test#31049
ManthanBhatt wants to merge 1 commit intoionic-team:mainfrom
ManthanBhatt:fix-infinite-scroll-top-test

Conversation

@ManthanBhatt
Copy link
Copy Markdown

Issue number: resolves #30423


What is the current behavior?

The top-positioned infinite scroll test is flaky on CI with Safari and is currently skipped for WebKit. The test assumes a specific initial scroll state, which makes it timing-sensitive.

What is the new behavior?

The test now uses a deterministic scroll sequence by scrolling to the bottom first, waiting for the content to actually be offset, and then scrolling back to the top before waiting for ionInfiniteComplete.

  • remove the WebKit skip
  • ensure the test starts from a known scrolled state
  • keep the existing item-count assertion after loading more content

Does this introduce a breaking change?

  • Yes
  • No

Other information

  • Verified locally with npx playwright test src/components/infinite-scroll/test/top/infinite-scroll.e2e.ts --project="Mobile Chrome"
  • Verified locally with npx playwright test src/components/infinite-scroll/test/top/infinite-scroll.e2e.ts --project="Mobile Safari"
  • npx eslint src/components/infinite-scroll/test/top/infinite-scroll.e2e.ts passed

@ManthanBhatt ManthanBhatt requested a review from a team as a code owner March 30, 2026 14:35
@ManthanBhatt ManthanBhatt requested a review from BenOsodrac March 30, 2026 14:35
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 30, 2026

@ManthanBhatt is attempting to deploy a commit to the Ionic Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions bot added the package: core @ionic/core package label Mar 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: core @ionic/core package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: Infinite scroll - scrolling to the top test is flaky on CI with Safari

1 participant