Skip to content

DataGrid - AI Assistant: regenerate e2e tests#33808

Merged
Alyar666 merged 8 commits into
DevExpress:26_1from
Alyar666:ai_assistant_e2e_regenerate_26_1
Jun 16, 2026
Merged

DataGrid - AI Assistant: regenerate e2e tests#33808
Alyar666 merged 8 commits into
DevExpress:26_1from
Alyar666:ai_assistant_e2e_regenerate_26_1

Conversation

@Alyar666

@Alyar666 Alyar666 commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

What does the PR change?

Adds e2e TestCafe tests for the DataGrid AI Assistant — the Regenerate button — visibility after pre-execution failures, absence after command execution, resend-and-replace behavior and in-flight disabling (plan §1.12, §3.8.2).

Part of splitting the AI Assistant e2e suite into per-area pull requests.

Copilot AI review requested due to automatic review settings June 2, 2026 22:30
@Alyar666 Alyar666 requested a review from a team as a code owner June 2, 2026 22:30
@Alyar666 Alyar666 self-assigned this Jun 2, 2026

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

Adds/extends the DataGrid AI Assistant (chat) TestCafe page models and introduces a new e2e suite covering Regenerate-button behavior across failure/success/abort scenarios, as part of splitting the AI Assistant e2e coverage into per-area PRs.

Changes:

  • Added a new regenerate.functional.ts TestCafe suite validating Regenerate visibility, replacement behavior, and in-flight state handling.
  • Extended devextreme-testcafe-models DataGrid/AI Assistant chat POMs with additional selectors/accessors (messages, input, suggestions, abort-confirm dialog).
  • Introduced shared ai-assistant test helpers (testHelpers.ts) for common constants and client-side utilities.

Reviewed changes

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

Show a summary per file
File Description
packages/testcafe-models/dataGrid/index.ts Adds DataGrid POM API helper for reading datasource sort and a helper to focus the AI Assistant button.
packages/testcafe-models/dataGrid/aiAssistantChat.ts Expands AI Assistant chat POM to expose input, message grouping, suggestions, and abort-confirm dialog accessors.
packages/testcafe-models/chat.ts Adds a reusable getMessageBubbles() accessor and reuses it in getMessage().
e2e/testcafe-devextreme/tests/dataGrid/common/aiAssistant/testHelpers.ts Adds shared constants and ClientFunctions for AI-assistant DataGrid tests.
e2e/testcafe-devextreme/tests/dataGrid/common/aiAssistant/regenerate.functional.ts New e2e tests for DataGrid AI Assistant Regenerate scenarios and sequential regeneration behavior.

Comment thread packages/testcafe-models/dataGrid/aiAssistantChat.ts
Comment thread packages/testcafe-models/dataGrid/aiAssistantChat.ts
Copilot AI review requested due to automatic review settings June 5, 2026 00:43

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

Comment thread packages/testcafe-models/dataGrid/aiAssistantChat.ts
Comment thread e2e/testcafe-devextreme/tests/dataGrid/common/aiAssistant/testHelpers.ts Outdated
@Alyar666 Alyar666 force-pushed the ai_assistant_e2e_regenerate_26_1 branch from 73b42b7 to e092d3d Compare June 5, 2026 08:12
Copilot AI review requested due to automatic review settings June 5, 2026 08:53

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

Copilot AI review requested due to automatic review settings June 8, 2026 09:42

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

Copilot AI review requested due to automatic review settings June 8, 2026 10:38

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

Comment thread e2e/testcafe-devextreme/tests/dataGrid/common/aiAssistant/testHelpers.ts Outdated
Alyar and others added 7 commits June 16, 2026 13:07
the Regenerate button — visibility after pre-execution failures, absence after command execution, resend-and-replace behavior and in-flight disabling (plan §1.12, §3.8.2)

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Add grid-state assertions (apiColumnOption) to every regenerate test so each
case confirms whether the command applied or not, and verify 1.12.8 resends the
same prompt with a freshly-built context. Switch 1.12.4 to getErrorMessages for
consistency. Move the mock factory into the test file and drop POM/testHelpers
members unused by this PR.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Replace the per-file tagged response mock with the shared queue helper and
HANG/FAIL sentinels.
Collapse the per-section fixtures into a single named fixture; keep the
// === §X === markers as in-file section dividers.
…licit regenerate responses

Restore scoped getWrapper; drop await inside t.expect for retry; give the
sequential-regenerate test a failure response per expected call instead of
relying on the unexpected-call fallback (Copilot review).
Copilot AI review requested due to automatic review settings June 16, 2026 09:58
@Alyar666 Alyar666 force-pushed the ai_assistant_e2e_regenerate_26_1 branch from 5476529 to 167a2a3 Compare June 16, 2026 09:58

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

@Alyar666 Alyar666 merged commit 1b1f552 into DevExpress:26_1 Jun 16, 2026
169 of 172 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