Skip to content

Lookup: Fix loading state flaky test#33469

Merged
r-farkhutdinov merged 3 commits into
DevExpress:26_1from
r-farkhutdinov:26_1_lookup_fix_flaky_test
Apr 30, 2026
Merged

Lookup: Fix loading state flaky test#33469
r-farkhutdinov merged 3 commits into
DevExpress:26_1from
r-farkhutdinov:26_1_lookup_fix_flaky_test

Conversation

@r-farkhutdinov
Copy link
Copy Markdown
Contributor

No description provided.

Co-authored-by: Copilot <copilot@github.com>
@r-farkhutdinov r-farkhutdinov self-assigned this Apr 30, 2026
@r-farkhutdinov r-farkhutdinov requested a review from a team as a code owner April 30, 2026 09:38
Copilot AI review requested due to automatic review settings April 30, 2026 09:38
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Stabilizes the TestCafe screenshot test for dxLookup popup height in the loading state by removing the unstable marker and attempting to eliminate animation-driven flakiness.

Changes:

  • Removed unstable: true metadata from the “loading state” screenshot test.
  • Injected a stylesheet rule to pause CSS animations before taking the loading-state screenshot.

Comment thread e2e/testcafe-devextreme/tests/editors/lookup/common.ts Outdated
Ruslan Farkhutdinov and others added 2 commits April 30, 2026 12:43
Copilot AI review requested due to automatic review settings April 30, 2026 11:25
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 1 out of 1 changed files in this pull request and generated no new comments.

@r-farkhutdinov r-farkhutdinov merged commit 2fb4ba4 into DevExpress:26_1 Apr 30, 2026
129 checks passed
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.

3 participants