[testing] Add xaml and resizetizer tests to run on helix#33567
Merged
[testing] Add xaml and resizetizer tests to run on helix#33567
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds two unit test projects (Controls.Xaml.UnitTests and Resizetizer.UnitTests) to the Helix test infrastructure while temporarily disabling their execution in the CI pipeline. The changes prepare these tests to run on Helix by including them in the test project configuration, but comment out their actual execution steps pending further validation.
Changes:
- Added
Controls.Xaml.UnitTestsandResizetizer.UnitTeststo the list of XUnit projects ineng/helix.proj - Commented out CI pipeline steps that execute and publish results for both test projects in
eng/pipelines/arcade/stage-unit-tests.yml
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| eng/helix.proj | Added two test projects to XUnit project list for Helix test execution |
| eng/pipelines/arcade/stage-unit-tests.yml | Commented out execution and result publishing steps for the two test projects |
3 tasks
0951f65 to
7ef7f2b
Compare
Copilot AI
added a commit
that referenced
this pull request
Jan 28, 2026
- Add run-helix-tests skill for submitting unit tests to Helix - Submit-HelixTests.ps1: Submit tests to Helix queues - Get-HelixJobStatus.ps1: Monitor job completion - Get-HelixWorkItemLog.ps1: Fetch work item console logs - Update eng/helix.proj to include Xaml and Resizetizer unit tests - Add HelixCorrelationPayload for SourceGen DLL (required by XAML tests) - Update eng/pipelines/arcade/stage-unit-tests.yml to run XAML/Resizetizer tests on Helix instead of inline - Clean up duplicate documentation in helix-device-tests.instructions.md Extracted from PR #33567 - infrastructure changes only, no test-specific fixes included. Co-authored-by: rmarinho <1235097+rmarinho@users.noreply.github.com>
This was referenced Jan 28, 2026
Open
5fdb547 to
5b6bdff
Compare
Member
Author
|
/rebase |
5b6bdff to
49a3c14
Compare
StephaneDelcroix
previously approved these changes
Jan 29, 2026
src/Controls/tests/Xaml.UnitTests/MSBuild/_Directory.Build.Helix.props
Outdated
Show resolved
Hide resolved
src/Controls/tests/Xaml.UnitTests/MSBuild/_Directory.Build.Helix.targets
Outdated
Show resolved
Hide resolved
src/Controls/tests/Xaml.UnitTests/Controls.Xaml.UnitTests.csproj
Outdated
Show resolved
Hide resolved
This PR enables running XAML and Resizetizer unit tests on Helix infrastructure. Changes: - Add XAML and Resizetizer test projects to Helix configuration (eng/helix.proj) - Add Helix correlation payloads for SourceGen DLL and MSBuild test files - Create Helix-specific Directory.Build files for MSBuild tests - Update test infrastructure to find files in Helix payload paths - Fix CA1307/CA1865 analyzer conflicts in string.LastIndexOf calls - Add run-helix-tests skill for local Helix test submission Test results on Helix: - XAML Unit Tests: 1814 tests passing - Resizetizer Unit Tests: 590 tests passing
49a3c14 to
79d0c18
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of Change
Work to run more of our unit tests on Helix.
This pull request updates the unit test project configuration and related pipeline steps. The main changes are to ensure that the
Controls.Xaml.UnitTestsandResizetizer.UnitTestsprojects are included in the build/test process, but their execution in the CI pipeline is currently commented out.Test project configuration:
Controls.Xaml.UnitTests.csprojand re-enabledResizetizer.UnitTests.csprojin the list of XUnit projects ineng/helix.proj, ensuring these test projects are recognized and available for test runs.Pipeline changes:
eng/pipelines/arcade/stage-unit-tests.ymlthat run and publish results for bothControls.Xaml.UnitTestsandResizetizer.UnitTests, effectively disabling their execution in the CI pipeline for now.Works fine from VS
