Skip to content

CardView: stabilize columnChooser tests#34362

Open
anna-shakhova wants to merge 1 commit into
DevExpress:mainfrom
anna-shakhova:cardview_column_chooser_tests_main
Open

CardView: stabilize columnChooser tests#34362
anna-shakhova wants to merge 1 commit into
DevExpress:mainfrom
anna-shakhova:cardview_column_chooser_tests_main

Conversation

@anna-shakhova

Copy link
Copy Markdown
Contributor

No description provided.

@anna-shakhova
anna-shakhova force-pushed the cardview_column_chooser_tests_main branch from e5bdecb to d0f0b99 Compare July 17, 2026 14:09
@anna-shakhova
anna-shakhova marked this pull request as ready for review July 17, 2026 14:14
@anna-shakhova
anna-shakhova requested a review from a team as a code owner July 17, 2026 14:14
Copilot AI review requested due to automatic review settings July 17, 2026 14:14

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

Stabilizes CardView column chooser E2E coverage by relying on TestCafe’s built-in assertion waiting behavior and re-enabling a previously skipped/flaky scenario.

Changes:

  • Removed the custom 1s timeout from getCardFieldCaptions so t.expect(...).eql(...) uses the framework default assertion timeout (more resilient under slower CI).
  • Removed the unstable: true meta flag from the “multiple hide/show actions” column chooser test so it runs as part of the normal suite.

Reviewed changes

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

File Description
e2e/testcafe-devextreme/tests/cardView/helpers/cardUtils.ts Removes a short custom timeout so caption-count assertions wait using TestCafe defaults.
e2e/testcafe-devextreme/tests/cardView/columnChooser/functional.ts Re-enables the multiple hide/show column chooser test by removing the unstable meta tag.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants