[aleksei-semikozov]

No description provided.

[Copilot]

Pull request overview

Introduces Playwright-based end-to-end testing infrastructure to the monorepo: a new e2e/playwright package for component-level tests (including screenshot/trace support and CI matrix execution), and migrates wrapper E2E tests from TestCafe to Playwright with updated CI artifact collection.

Changes:

• Added e2e/playwright Nx project with Playwright runner, static server, fixtures/helpers, models, and initial tests.
• Migrated e2e/wrappers E2E tests from TestCafe (*.test.js + custom runner) to Playwright (*.spec.ts + Playwright webServer).
• Updated pnpm workspace/lockfile and GitHub Actions workflows to run and lint the new Playwright suites and upload Playwright reports/traces on failure.

Reviewed changes

Copilot reviewed 62 out of 66 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
pnpm-workspace.yaml	Adds @playwright/test to the shared catalog.
pnpm-lock.yaml	Locks Playwright dependencies and adds the new e2e/playwright importer.
e2e/wrappers/tests/textbox.test.js	Removes legacy TestCafe wrapper E2E test.
e2e/wrappers/tests/textbox.spec.ts	Adds Playwright wrapper E2E test replacement.
e2e/wrappers/tests/select-box-nested-validator.test.js	Removes legacy TestCafe wrapper E2E test.
e2e/wrappers/tests/select-box-nested-validator.spec.ts	Adds Playwright wrapper E2E test replacement.
e2e/wrappers/tests/inputs-list-in-form.test.js	Removes legacy TestCafe wrapper E2E test.
e2e/wrappers/tests/inputs-list-in-form.spec.ts	Adds Playwright wrapper E2E test replacement.
e2e/wrappers/tests/gantt-template-state-update.test.js	Removes legacy TestCafe wrapper E2E test.
e2e/wrappers/tests/gantt-template-state-update.spec.ts	Adds Playwright wrapper E2E test replacement with framework skip.
e2e/wrappers/tests/chat-template-rerender.test.js	Removes legacy TestCafe wrapper E2E test.
e2e/wrappers/tests/chat-template-rerender.spec.ts	Adds Playwright wrapper E2E test replacement with framework skip.
e2e/wrappers/tests/button.test.js	Removes legacy TestCafe wrapper E2E test.
e2e/wrappers/tests/button.spec.ts	Adds Playwright wrapper E2E test replacement.
e2e/wrappers/test-helpers.js	Removes TestCafe-specific test helper utilities.
e2e/wrappers/serve.mjs	Adds an Express server to serve built wrapper apps for Playwright webServer.
e2e/wrappers/runner.js	Removes the TestCafe runner implementation.
e2e/wrappers/README.md	Documents local/CI wrapper Playwright E2E workflow and artifact usage.
e2e/wrappers/playwright.config.ts	Adds Playwright config for wrappers E2E (per-framework ports, webServer, artifacts).
e2e/wrappers/package.json	Switches wrapper tests from TestCafe runner to Playwright + cross-env; drops TestCafe/minimist.
e2e/wrappers/fixtures.ts	Adds Playwright fixtures for wrapper tests (framework option).
e2e/wrappers/docker/run.sh	Adds Docker runner to mirror CI environment and optionally run Playwright UI.
e2e/wrappers/docker/Dockerfile	Adds Docker image definition for wrapper Playwright E2E runs.
e2e/wrappers/.gitignore	Ignores Playwright report/result directories in wrappers E2E package.
e2e/playwright/tsconfig.json	Adds TypeScript config for the new component Playwright project.
e2e/playwright/tests/navigation/button/common.spec.ts	Adds initial component Playwright tests for Button (incl. screenshot coverage).
e2e/playwright/tests/editors/checkBox/common.spec.ts	Adds initial component Playwright tests for CheckBox.
e2e/playwright/tests/container.html	Adds a dedicated container page that loads built DevExtreme artifacts/themes for tests.
e2e/playwright/serve.mjs	Adds a static server serving the repo as docroot for component tests.
e2e/playwright/runner.mjs	Adds a CLI runner mapping TestCafe-like flags to Playwright (shards, grep, workers, theme).
e2e/playwright/README.md	Documents component Playwright usage, screenshots/etalons, themes, CI diagnostics.
e2e/playwright/project.json	Registers devextreme-playwright-tests as an Nx project with lint/test targets.
e2e/playwright/playwright.config.ts	Adds Playwright config (workers, webServer, snapshot pathing, CI screenshot budgets, timezone).
e2e/playwright/package.json	Adds the new Playwright project package manifest and scripts.
e2e/playwright/models/types.ts	Introduces widget name/option typing scaffolding for model layer.
e2e/playwright/models/internal/widget.ts	Adds a base widget model for interacting with DevExtreme widgets via jQuery/instance APIs.
e2e/playwright/models/checkBox.ts	Adds CheckBox model helpers (checked/indeterminate state).
e2e/playwright/models/button.ts	Adds Button model helpers (text/selected state).
e2e/playwright/helpers/themeUtils.ts	Adds theme name utilities for screenshot naming/theme switching.
e2e/playwright/helpers/testPageUtils.ts	Adds utilities to open/reset/clear the test container page and theme state.
e2e/playwright/helpers/screenshots.ts	Adds screenshot assertion helpers aligned with existing etalon naming conventions.
e2e/playwright/helpers/mockDate.ts	Adds date mocking helpers based on Playwright clock.
e2e/playwright/helpers/domUtils.ts	Adds DOM helper utilities used by tests for building layouts and tweaking attributes/styles.
e2e/playwright/helpers/createWidget.ts	Adds helper to instantiate DevExtreme jQuery widgets from tests.
e2e/playwright/helpers/const.ts	Adds shared constants for server/theme/browser size and env handling.
e2e/playwright/helpers/apiMock.ts	Adds Playwright-based API mocking helper (route-based fulfillment).
e2e/playwright/fixtures.ts	Adds Playwright fixtures that pre-open and reset the container page per test.
e2e/playwright/eslint.config.mjs	Adds ESLint config for the new TypeScript-based Playwright package.
e2e/playwright/docker/run.sh	Adds Docker runner for component Playwright tests (incl. UI mode).
e2e/playwright/docker/Dockerfile	Adds Docker image definition for component Playwright E2E runs.
e2e/playwright/.gitignore	Ignores Playwright artifacts directory for component tests.
apps/vue/test.js	Removes legacy TestCafe playground test stub.
apps/vue/runner.js	Removes legacy TestCafe runner for Vue playground.
apps/vue/project.json	Cleans up inputs now that test.js is removed.
apps/vue/package.json	Removes the legacy test script for Vue playground.
apps/react/test.js	Removes legacy TestCafe playground test stub.
apps/react/runner.js	Removes legacy TestCafe runner for React playground.
apps/react/project.json	Cleans up inputs now that test.js is removed.
apps/react/package.json	Removes the legacy test script for React playground.
.github/workflows/wrapper_tests_e2e.yml	Updates wrapper E2E CI to run Playwright and upload report/trace artifacts on failure.
.github/workflows/testcafe_tests_playwright.yml	Adds a dedicated CI workflow to run component Playwright tests in a shard/theme matrix.
.github/workflows/lint.yml	Adds a lint step for the new e2e/playwright package.
.github/workflows/default_workflow.yml	Adds devextreme-playwright-tests to the default Nx test/lint workflow project list.

Files not reviewed (1)

• pnpm-lock.yaml: Generated file

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Pull request overview

Copilot reviewed 62 out of 66 changed files in this pull request and generated 3 comments.

Files not reviewed (1)

• pnpm-lock.yaml: Generated file

Suppressed comments (4)

Previously missed (4) — in code that hasn't changed since the last review.

e2e/playwright/README.md:4

• This sentence is misleading: even when tests use channel: 'chrome' at runtime, installing @playwright/test will still download Playwright's bundled browsers unless PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD=1 is set. Clarifying this avoids unexpected install time/size regressions.

End-to-end tests for DevExtreme components. They run on [Playwright](https://playwright.dev/) and
use the system Google Chrome (`channel: 'chrome'`), so no browser download is needed.

e2e/wrappers/README.md:5

• This sentence is misleading: using channel: 'chrome' avoids Playwright-managed browsers at runtime, but installing @playwright/test still downloads Playwright's bundled browsers unless PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD=1 is set. Clarify to prevent unexpected install-time downloads.
  .github/workflows/testcafe_tests_playwright.yml:234
• @playwright/test pulls in playwright, which downloads bundled browsers during pnpm install unless PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD=1 is set. Since the suite uses system Chrome (channel: 'chrome'), consider setting this env var on the install step to keep CI installs smaller/faster.

    - name: Install dependencies
      run: pnpm install --frozen-lockfile

.github/workflows/wrapper_tests_e2e.yml:171

• @playwright/test installs playwright, which downloads bundled browsers during pnpm install unless PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD=1 is set. Since these wrapper E2E tests run with channel: 'chrome', consider setting the env var on the install step to reduce CI install time/size.

      - name: Install dependencies
        run: pnpm install --frozen-lockfile

[Copilot]

Pull request overview

Copilot reviewed 62 out of 66 changed files in this pull request and generated 4 comments.

Files not reviewed (1)

• pnpm-lock.yaml: Generated file

Suppressed comments (1)

.github/workflows/playwright_tests.yml:234

• Same as the build job: add PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD=1 during pnpm install so CI doesn’t download Playwright browsers that aren’t used when running against system Chrome (channel: 'chrome').

    - name: Install dependencies
      run: pnpm install --frozen-lockfile

[Copilot]

Copilot review overview

🔵 Needs a closer look

The createWidget helper has unresolved date serialization, callback dependency, nullability, and option-factory typing issues.

Review effort: Lite
Findings: 3 · 1

Open (4)

• Missing default Fluent etalon for generic light test
• This test is tagged for generic.light, material.blue.light, and material.blue.light.compact,…
• With the inherited strictNullChecks setting, JSON.stringify returns string | undefined, so…
• Allow async option factories in the public type

Files not reviewed (1)

• pnpm-lock.yaml: Generated file

[Copilot]

Copilot review overview

🟡 Changes recommended

Unresolved CI theme handling, clock initialization, function serialization, and Gantt asset issues remain.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 4 · 2

Open (6)

• Pass theme only to themed component jobs · New
• Missing default Fluent etalon for generic light test
• This test is tagged for generic.light, material.blue.light, and material.blue.light.compact,…
• With the inherited strictNullChecks setting, JSON.stringify returns string | undefined, so…
• Install Playwright clock before setting fixed time · New
• Allow async option factories in the public type

Files not reviewed (1)

• pnpm-lock.yaml: Generated file

[Copilot]

Copilot review overview

🔵 Needs a closer look

Unresolved Date handling, clock setup, and screenshot baseline issues block approval; documentation also has an artifact-name mismatch.

Review effort: Lite
Findings: 4 · 2

Open (6)

• Pass theme only to themed component jobs
• Missing default Fluent etalon for generic light test
• This test is tagged for generic.light, material.blue.light, and material.blue.light.compact,…
• With the inherited strictNullChecks setting, JSON.stringify returns string | undefined, so…
• Install Playwright clock before setting fixed time
• Allow async option factories in the public type

Files not reviewed (1)

• pnpm-lock.yaml: Generated file

Previously missed (1)

In code that hasn't changed since last review

Documented artifact names do not match uploaded workflow artifacts

e2e/​playwright/​README.md:110

The documented download names do not match the artifacts produced by playwright_tests.yml: the workflow uploads playwright-report and playwright-traces, not the *-common names shown here. Following these commands after a failed run returns no matching artifact.

[Copilot]

Copilot review overview

🔵 Needs a closer look

Unresolved moderate helper, asset, and linting issues must be addressed before approval.

Review effort: Lite
Findings: 4 · 1

Open (5)

• Pass theme only to themed component jobs
• Missing default Fluent etalon for generic light test
• This test is tagged for generic.light, material.blue.light, and material.blue.light.compact,…
• With the inherited strictNullChecks setting, JSON.stringify returns string | undefined, so…
• Install Playwright clock before setting fixed time

Resolved since last review (1)

• Allow async option factories in the public type

Files not reviewed (1)

• pnpm-lock.yaml: Generated file

Previously missed (2)

In code that hasn't changed since last review

Mock helper incorrectly JSON-encodes string response bodies

e2e/​playwright/​helpers/​apiMock.ts:33

This helper always JSON-stringifies the mocked body, so a string response such as the XML payloads used by existing RequestMock fixtures (e2e/testcafe-devextreme/tests/common/pivotGrid/apiMocks/OLAP_api.mock.ts:29) is returned with quotes and cannot be parsed as XML. Preserve string bodies verbatim (and let the caller's content-type header apply) while JSON-encoding object bodies.

Documented artifact names do not match uploaded artifacts

e2e/​playwright/​README.md:110

The CI artifacts are uploaded under playwright-report and playwright-traces in the workflow, not playwright-report-common and playwright-traces-common. These documented gh run download commands therefore fail to find the artifacts; use the actual merged artifact names.

[Copilot]

Copilot review overview

🟡 Changes recommended

Unresolved CI artifact handling, widget serialization/function issues, and type-check cache invalidation remain.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1

Open (1)

• Unsupported blob output variable causes report filename collisions · New

Resolved since last review (5)

• Pass theme only to themed component jobs
• Missing default Fluent etalon for generic light test
• This test is tagged for generic.light, material.blue.light, and material.blue.light.compact,…
• With the inherited strictNullChecks setting, JSON.stringify returns string | undefined, so…
• Install Playwright clock before setting fixed time

Files not reviewed (1)

• pnpm-lock.yaml: Generated file

Previously missed (1)

In code that hasn't changed since last review

Date serialization converts Date values to strings

e2e/​playwright/​helpers/​createWidget.ts:30

JSON.stringify invokes Date.prototype.toJSON before calling this replacer, so raw is already an ISO string here and the DATE_MARK branch is never reached for Date properties. The helper consequently sends strings instead of Date instances, contradicting the serialization contract and breaking widget callbacks/options that depend on the value being a Date; use a serialization approach that observes Dates before toJSON runs.