Skip to content

CI: make visual test passing rules stricter#34177

Draft
EugeniyKiyashko wants to merge 6 commits into
DevExpress:26_1from
EugeniyKiyashko:26_1_tests_stability
Draft

CI: make visual test passing rules stricter#34177
EugeniyKiyashko wants to merge 6 commits into
DevExpress:26_1from
EugeniyKiyashko:26_1_tests_stability

Conversation

@EugeniyKiyashko

Copy link
Copy Markdown
Contributor

No description provided.

@EugeniyKiyashko EugeniyKiyashko self-assigned this Jun 30, 2026
@EugeniyKiyashko EugeniyKiyashko requested a review from a team as a code owner June 30, 2026 09:17
@EugeniyKiyashko EugeniyKiyashko requested a review from a team as a code owner June 30, 2026 17:58
Copilot AI review requested due to automatic review settings July 1, 2026 09:15
@EugeniyKiyashko EugeniyKiyashko requested a review from a team as a code owner July 1, 2026 09:15

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Tightens CI visual/E2E TestCafe “pass” criteria by disabling automatic recovery mechanisms (quarantine/retries) and explicitly marking a couple of flaky tests as unstable.

Changes:

  • Disabled TestCafe quarantine mode in the visual-tests GitHub workflow jobs.
  • Disabled failed-test retries in the e2e/testcafe-devextreme runner.
  • Marked specific Scheduler/DateRangeBox tests as unstable and adjusted one fixture’s page-reload behavior.

Reviewed changes

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

File Description
e2e/testcafe-devextreme/tests/scheduler/common/dragAndDrop/appointmentCollector.ts Marks a flaky drag-and-drop test as unstable.
e2e/testcafe-devextreme/tests/editors/dateRangeBox/keyboard.ts Enables page reloads for the fixture and marks a flaky tab-navigation test as unstable.
e2e/testcafe-devextreme/runner.ts Disables failed-test retries and simplifies retry logic.
.github/workflows/visual-tests-demos.yml Attempts to disable TestCafe quarantine mode in CI runs.

Comment thread .github/workflows/visual-tests-demos.yml
Comment thread .github/workflows/visual-tests-demos.yml
Comment thread .github/workflows/visual-tests-demos.yml
Comment thread e2e/testcafe-devextreme/runner.ts
Copilot AI review requested due to automatic review settings July 1, 2026 12:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 7 out of 10 changed files in this pull request and generated 2 comments.

Comment thread apps/demos/utils/visual-tests/testcafe-runner.ts
Comment thread e2e/testcafe-devextreme/runner.ts
@EugeniyKiyashko EugeniyKiyashko marked this pull request as draft July 1, 2026 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants