chore(samples): migrate search-vuejs from Cypress to Playwright by alexprudhomme · Pull Request #7474 · coveo/ui-kit

alexprudhomme · 2026-04-21T15:27:35Z

Problem

The samples/atomic/search-vuejs sample uses Cypress for E2E testing and points to an org that no longer works. The tests are also not run in CI.

Solution

Migrate E2E tests from Cypress to Playwright
Switch the sample to use the searchuisamples org and simplify the layout to match other search samples
Add the Playwright tests to the CI pipeline

Reviewable by commit

changeset-bot · 2026-04-21T15:27:48Z

⚠️ No Changeset found

Latest commit: ee87dbd

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

svcsnykcoveo · 2026-04-21T15:27:59Z

✅ Snyk checks have passed. No issues have been found so far.

Status	Scan Engine	Critical	High	Medium	Low	Total (0)
✅	Open Source Security	0	0	0	0	0 issues
✅	Licenses	0	0	0	0	0 issues
✅	Code Security	0	0	0	0	0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

chromatic-com · 2026-04-21T15:52:55Z

Tip

All tests passed and all changes approved!

🟢 UI Tests: 356 tests unchanged
🟢 UI Review: 356 stories published -- no changes
Storybook Publish: 356 stories published

pkg-pr-new · 2026-04-21T15:53:40Z

coveo-atomic-pkg-new-template

@coveo/atomic

npm i https://pkg.pr.new/@coveo/atomic@7474

@coveo/atomic-hosted-page

npm i https://pkg.pr.new/@coveo/atomic-hosted-page@7474

@coveo/atomic-legacy

npm i https://pkg.pr.new/@coveo/atomic-legacy@7474

@coveo/atomic-react

npm i https://pkg.pr.new/@coveo/atomic-react@7474

@coveo/auth

npm i https://pkg.pr.new/@coveo/auth@7474

@coveo/bueno

npm i https://pkg.pr.new/@coveo/bueno@7474

@coveo/create-atomic

npm i https://pkg.pr.new/@coveo/create-atomic@7474

@coveo/create-atomic-component

npm i https://pkg.pr.new/@coveo/create-atomic-component@7474

@coveo/create-atomic-component-project

npm i https://pkg.pr.new/@coveo/create-atomic-component-project@7474

@coveo/create-atomic-result-component

npm i https://pkg.pr.new/@coveo/create-atomic-result-component@7474

@coveo/create-atomic-rollup-plugin

npm i https://pkg.pr.new/@coveo/create-atomic-rollup-plugin@7474

@coveo/headless

npm i https://pkg.pr.new/@coveo/headless@7474

@coveo/headless-react

npm i https://pkg.pr.new/@coveo/headless-react@7474

@coveo/shopify

npm i https://pkg.pr.new/@coveo/shopify@7474

commit: ee87dbd

github-actions · 2026-04-21T15:55:41Z

🔗 Scratch Orgs ready to test this PR:

✅ LWS Enabled: Open Org
✅ LWS Disabled: Open Org

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot

Pull request overview

This PR migrates the samples/atomic/search-vuejs sample E2E coverage from Cypress to Playwright, updates the sample configuration to use the searchuisamples org, and wires the new Playwright suite into CI.

Changes:

Added Playwright config + smoke test for the Atomic Vue sample and removed Cypress tests/config.
Updated the Vue sample’s Atomic configuration and simplified facets/results UI to align with other search samples.
Added a dedicated CI job and composite action to run the Vue Playwright tests.

Reviewed changes

Copilot reviewed 17 out of 18 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
`samples/atomic/search-vuejs/tests/smoke.spec.ts`	New Playwright smoke test for core search flows and console error assertions.
`samples/atomic/search-vuejs/playwright.config.ts`	Playwright runner config and dev-server bootstrapping for the sample.
`samples/atomic/search-vuejs/package.json`	Adds Playwright scripts/dependency; removes Cypress dependency.
`samples/atomic/search-vuejs/src/components/SearchPage.vue`	Switches org/token; simplifies `<atomic-search-interface>` attributes.
`samples/atomic/search-vuejs/src/components/SearchPageFacets.vue`	Updates facets to match the new sample org and align with other samples.
`samples/atomic/search-vuejs/src/components/TopBar.vue`	Removes price sorting options from the sample top bar.
`samples/atomic/search-vuejs/src/components/ResultsList.vue`	Switches result list display mode (grid → list).
`samples/atomic/search-vuejs/src/templates/result-template.html`	Simplifies result template and switches excerpt/date rendering.
`samples/atomic/search-vuejs/.gitignore`	Ignores Playwright output directories/caches.
`samples/atomic/search-vuejs/cypress.config.ts`	Removes Cypress configuration.
`samples/atomic/search-vuejs/cypress/e2e/smoke-test.cypress.cy.ts`	Removes Cypress smoke test.
`samples/atomic/search-vuejs/cypress/support/e2e.ts`	Removes Cypress support bootstrap file.
`samples/atomic/search-vuejs/cypress/support/commands.ts`	Removes Cypress commands placeholder.
`samples/atomic/search-vuejs/cypress/fixtures/example.json`	Removes Cypress fixture placeholder.
`samples/atomic/search-vuejs/cypress/tsconfig.json`	Removes Cypress TS config.
`pnpm-lock.yaml`	Locks Playwright dependency; removes Cypress lock entry for this sample.
`.github/workflows/ci.yml`	Adds a CI job to run Atomic Vue Playwright tests and gates PR/merge-queue validity on it.
`.github/actions/playwright-atomic-search-vuejs/action.yml`	New composite action to run the Vue Playwright suite and upload reports on failure.

Files not reviewed (1)

pnpm-lock.yaml: Language not supported

Copilot · 2026-04-21T20:55:06Z

+    const consoleErrors: string[] = [];
+    page.on('console', (msg) => {
+      if (msg.type() === 'error') {
+        consoleErrors.push(msg.text());
+      }
+    });


The test only collects console messages of type error, but Playwright reports uncaught runtime exceptions via the pageerror event. As written, the smoke test can pass even if the app throws an unhandled exception. Consider also listening to page.on('pageerror', ...) and failing the test (or adding it to the same filtered error list) so the migration preserves Cypress' "uncaught exception" coverage.

Copilot · 2026-04-21T20:55:07Z

+inputs:
+  github-token:
+    description: The GitHub token for accurately detecting a build vs a re-run build
+    required: false
+    default: ''


This composite action declares a github-token input but never uses it. To avoid confusion for callers, either remove the unused input or wire it into whatever rerun/build detection logic it’s intended for (matching how other actions in this repo handle it).

Suggested change

inputs:

github-token:

description: The GitHub token for accurately detecting a build vs a re-run build

required: false

default: ''

alexprudhomme temporarily deployed to pr-workflows April 21, 2026 15:27 — with GitHub Actions Inactive

alexprudhomme had a problem deploying to PR Artifacts April 21, 2026 15:27 — with GitHub Actions Failure

coveo-code-areas-pr-checks Bot added area:atomic area:node-dependencies team:dxui labels Apr 21, 2026

alexprudhomme force-pushed the KIT-5651-move-search-vuejs-to-playwright branch from 0ec33df to cf775d2 Compare April 21, 2026 15:45

alexprudhomme temporarily deployed to pr-workflows April 21, 2026 15:46 — with GitHub Actions Inactive

alexprudhomme temporarily deployed to PR Artifacts April 21, 2026 15:46 — with GitHub Actions Inactive

coveo-code-areas-pr-checks Bot added area:headless area:quantic area:root team:knowledge-ui-kit-reviewers team:salesforce-integration labels Apr 21, 2026

alexprudhomme temporarily deployed to PR Artifacts April 21, 2026 15:55 — with GitHub Actions Inactive

alexprudhomme force-pushed the KIT-5651-move-search-vuejs-to-playwright branch from cf775d2 to e4d7ce9 Compare April 21, 2026 15:59

alexprudhomme temporarily deployed to pr-workflows April 21, 2026 15:59 — with GitHub Actions Inactive

alexprudhomme temporarily deployed to PR Artifacts April 21, 2026 15:59 — with GitHub Actions Inactive

coveo-code-areas-pr-checks Bot removed area:root area:quantic labels Apr 21, 2026

alexprudhomme temporarily deployed to pr-workflows April 21, 2026 18:34 — with GitHub Actions Inactive

alexprudhomme temporarily deployed to PR Artifacts April 21, 2026 18:34 — with GitHub Actions Inactive

alexprudhomme temporarily deployed to PR Artifacts April 21, 2026 20:28 — with GitHub Actions Inactive

alexprudhomme temporarily deployed to pr-workflows April 21, 2026 20:28 — with GitHub Actions Inactive

coveo-code-areas-pr-checks Bot added the area:github label Apr 21, 2026

fix(samples): use searchuisamples org in search-vuejs sample

e3b6624

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

alexprudhomme force-pushed the KIT-5651-move-search-vuejs-to-playwright branch from f76926e to 006d234 Compare April 21, 2026 20:33

alexprudhomme temporarily deployed to pr-workflows April 21, 2026 20:34 — with GitHub Actions Inactive

alexprudhomme temporarily deployed to PR Artifacts April 21, 2026 20:34 — with GitHub Actions Inactive

ci: add Playwright e2e tests for search-vuejs to CI pipeline

ee87dbd

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

alexprudhomme force-pushed the KIT-5651-move-search-vuejs-to-playwright branch from 006d234 to ee87dbd Compare April 21, 2026 20:36

alexprudhomme temporarily deployed to pr-workflows April 21, 2026 20:36 — with GitHub Actions Inactive

alexprudhomme temporarily deployed to PR Artifacts April 21, 2026 20:36 — with GitHub Actions Inactive

alexprudhomme marked this pull request as ready for review April 21, 2026 20:50

alexprudhomme requested review from a team, Copilot, fbeaudoincoveo, louis-bompart and y-lakhdar April 21, 2026 20:50

Copilot started reviewing on behalf of alexprudhomme April 21, 2026 20:51 View session

alexprudhomme enabled auto-merge April 21, 2026 20:54

Copilot AI reviewed Apr 21, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore(samples): migrate search-vuejs from Cypress to Playwright#7474

chore(samples): migrate search-vuejs from Cypress to Playwright#7474
alexprudhomme wants to merge 3 commits intomainfrom
KIT-5651-move-search-vuejs-to-playwright

alexprudhomme commented Apr 21, 2026 •

edited by atlassian Bot

Loading

Uh oh!

changeset-bot Bot commented Apr 21, 2026 •

edited

Loading

Uh oh!

svcsnykcoveo commented Apr 21, 2026 •

edited

Loading

Uh oh!

chromatic-com Bot commented Apr 21, 2026 •

edited

Loading

All tests passed and all changes approved!

Uh oh!

pkg-pr-new Bot commented Apr 21, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented Apr 21, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Apr 21, 2026

Uh oh!

Copilot AI Apr 21, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

alexprudhomme commented Apr 21, 2026 • edited by atlassian Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Problem

Solution

Uh oh!

changeset-bot Bot commented Apr 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

⚠️ No Changeset found

Uh oh!

svcsnykcoveo commented Apr 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Snyk checks have passed. No issues have been found so far.

Uh oh!

chromatic-com Bot commented Apr 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

All tests passed and all changes approved!

Uh oh!

pkg-pr-new Bot commented Apr 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented Apr 21, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Apr 21, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 21, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

alexprudhomme commented Apr 21, 2026 •

edited by atlassian Bot

Loading

changeset-bot Bot commented Apr 21, 2026 •

edited

Loading

svcsnykcoveo commented Apr 21, 2026 •

edited

Loading

chromatic-com Bot commented Apr 21, 2026 •

edited

Loading

pkg-pr-new Bot commented Apr 21, 2026 •

edited

Loading