Skip to content

Test CI - #34954

Open
EugeniyKiyashko wants to merge 3 commits into
DevExpress:mainfrom
EugeniyKiyashko:test_CI_26_2
Open

Test CI#34954
EugeniyKiyashko wants to merge 3 commits into
DevExpress:mainfrom
EugeniyKiyashko:test_CI_26_2

Conversation

@EugeniyKiyashko

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI lite review requested due to automatic review settings August 27, 2026 13:45
@EugeniyKiyashko
EugeniyKiyashko requested a review from a team as a code owner August 27, 2026 13:45

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

Updates the TestCafe CI workflow to allow style-dependent tests to be included even when style files haven’t changed, by honoring a PR label override. This improves CI flexibility for cases where a full test sweep is needed without modifying style paths.

Changes:

  • Extend styles-changed job output to become true when either style-related paths change or the PR has the force all tests label.

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

Copilot AI review requested due to automatic review settings August 27, 2026 14:45

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

Suppressed comments (1)

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

.github/workflows/testcafe_tests.yml:24

  • The styles-changed job output now references github.event.pull_request.labels, but this workflow also runs on merge_group and workflow_dispatch events where github.event.pull_request may be undefined, which can cause expression evaluation failures. Guard the label check with github.event_name == 'pull_request' (or similar) so non-PR runs don’t break.
      styles-changed: ${{ steps.changes.outputs.styles == 'true' || contains(github.event.pull_request.labels.*.name, 'force all tests') }}

Copilot AI review requested due to automatic review settings August 27, 2026 19: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 3 out of 3 changed files in this pull request and generated 1 comment.

).eql('true');
}

await t.hover(Selector('.dx-scheduler-date-table-other-month'));
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