Skip to content

Conversation

@aymeric-giraudet
Copy link
Member

@aymeric-giraudet aymeric-giraudet commented Feb 3, 2026

Summary

  • Migrates the main e2e test suite from WebdriverIO to Playwright
  • Migrates Next.js package e2e tests (both App Router and Pages Router) from WebdriverIO to Playwright
  • Keeps WebdriverIO only for IE11 tests (via Sauce Labs) since Playwright doesn't support IE11
  • Updates CircleCI config to run Playwright tests for all e2e jobs except IE11

Changes

  • Added Playwright configuration and fixtures with helper methods for InstantSearch interactions
  • Rewrote all e2e specs in Playwright syntax
  • Updated port from 5000 to 3456 to avoid macOS AirPlay conflict
  • Added SPA routing rewrites in website/serve.json for e-commerce examples
  • Added .claude/rules/e2e.md documentation for the e2e testing setup

Test plan

  • Run yarn website:examples && yarn test:e2e locally
  • Verify all Playwright tests pass in CI (Chromium + Firefox)
  • Verify IE11 tests still pass via Sauce Labs
  • Verify Next.js e2e tests pass for both App Router and Pages Router packages

🤖 Generated with Claude Code

@codesandbox-ci
Copy link

codesandbox-ci bot commented Feb 3, 2026

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 9a4c661:

Sandbox Source
example-instantsearch-getting-started Configuration
example-react-instantsearch-getting-started Configuration
example-react-instantsearch-next-app-dir-example Configuration
example-react-instantsearch-next-routing-example Configuration
example-vue-instantsearch-getting-started Configuration

@aymeric-giraudet aymeric-giraudet force-pushed the test/migrate-to-playwright branch 3 times, most recently from a133088 to a701ad7 Compare February 3, 2026 17:09
@aymeric-giraudet aymeric-giraudet force-pushed the test/migrate-to-playwright branch from a701ad7 to ae17885 Compare February 4, 2026 10:42
@aymeric-giraudet aymeric-giraudet marked this pull request as ready for review February 5, 2026 15:41
@aymeric-giraudet aymeric-giraudet requested review from a team, FabienMotte and shaejaz and removed request for a team February 9, 2026 10:18
Copy link
Contributor

@shaejaz shaejaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good!

"@wdio/static-server-service": "5.16.5",
"dotenv": "8.2.0",
"serve": "14.2.4",
"ts-node": "8.4.1",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we still need this? it's deleted in the other package

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants