[dev-v5][Chore] Move test harness to MTP v2 - #5206
Merged
Vincent Baaij (vnbaaij) merged 13 commits intoAug 28, 2026
Merged
Conversation
Vincent Baaij (vnbaaij)
requested a review
from Denis Voituron (dvoituron)
as a code owner
August 28, 2026 07:53
Contributor
There was a problem hiding this comment.
Pull request overview
Moves the repository’s test harness and coverage collection toward Microsoft Testing Platform (MTP) v2, updating test projects and CI/scripts to use the new dotnet test --coverage flow.
Changes:
- Migrate multiple test projects from VSTest/Coverlet packages to
xunit.v3.mtp-v2+Microsoft.Testing.Extensions.CodeCoverage. - Introduce runsettings-based module filtering for coverage and update local/CI coverage collection commands.
- Adjust supporting test utilities (skippable facts, DocApiGen integration assembly loading, runner settings).
Reviewed changes
Copilot reviewed 17 out of 17 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/McpServer/SkippableFactAttribute.cs | Adds caller info forwarding to FactAttribute for better test metadata. |
| tests/McpServer/McpServer.Tests.csproj | Migrates MCP server tests to MTP v2 + code coverage extension; keeps MCP doc JSON generation hook. |
| tests/Integration/Components.IntegrationTests.csproj | Removes xUnit VS adapter reference (project still on xUnit v2). |
| tests/Core/xunit.runner.json | Enables xUnit v3 parallelization in Core tests. |
| tests/Core/coverage.runsettings | Adds runsettings to scope coverage to Core component assembly. |
| tests/Core/Components.Tests.csproj | Migrates Core tests to MTP v2 + code coverage extension; suppresses xUnit warning. |
| tests/Charts/xunit.runner.json | Enables xUnit v3 parallelization in Charts tests. |
| tests/Charts/coverage.runsettings | Adds runsettings to scope coverage to Charts assembly. |
| tests/Charts/Components.Charts.Tests.csproj | Migrates Charts tests to MTP v2 + code coverage extension. |
| tests/_StartCodeCoverage.ps1 | Updates local coverage script to new dotnet test --coverage arguments and output naming. |
| examples/Tools/FluentUI.Demo.DocViewer.Tests/FluentUI.Demo.DocViewer.Tests.csproj | Migrates DocViewer tests to MTP v2 + code coverage extension. |
| examples/Tools/FluentUI.Demo.DocApiGen.Tests/FluentUI.Demo.DocApiGen.Tests.csproj | Migrates DocApiGen unit tests to MTP v2 + code coverage extension. |
| examples/Tools/FluentUI.Demo.DocApiGen.IntegrationTests/FluentUIComponentsIntegrationTests.cs | Switches FluentUI assembly loading approach and updates repo-root discovery to .slnx. |
| examples/Tools/FluentUI.Demo.DocApiGen.IntegrationTests/FluentUI.Demo.DocApiGen.IntegrationTests.csproj | Migrates DocApiGen integration tests to MTP v2 and adds Core project reference. |
| Directory.Packages.props | Adds central versions for MTP/coverage packages; updates xUnit v3 dependency choice. |
| Directory.Build.props | Minor whitespace-only change. |
| .github/workflows/build-core-lib.yml | Updates CI to use dotnet test --coverage instead of Coverlet MSBuild properties. |
Suppressed comments (1)
tests/_StartCodeCoverage.ps1:162
- Same path robustness issue as the Core run: these relative paths depend on the current working directory, but the script does not
Set-Location $scriptDir. UseJoin-Path $scriptDir ...(orPush-Location) so the script works when invoked from anywhere.
& dotnet test .\Charts\Components.Charts.Tests.csproj `
--results-directory $chartsResults `
--configuration Release `
--coverage `
--coverage-output-format cobertura `
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: vnbaaij <1761079+vnbaaij@users.noreply.github.com>
Copilot stopped work on behalf of
Vincent Baaij (vnbaaij) due to an error
August 28, 2026 09:36
|
✅ All tests passed successfully Details on your Workflow / Core Tests page. |
Summary - Unit Tests Code CoverageSummary
CoverageMicrosoft.FluentUI.AspNetCore.Components - 98.3%
Microsoft.FluentUI.AspNetCore.Components.Charts - 100%
|
Vincent Baaij (vnbaaij)
enabled auto-merge (squash)
August 28, 2026 11:40
Denis Voituron (dvoituron)
approved these changes
Aug 28, 2026
Vincent Baaij (vnbaaij)
deleted the
users/vnbaaij/dev-v5/move-tests-to-mtpv2
branch
August 28, 2026 12:20
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
[dev-v5][Chore] Move test harness to MTP v2
This PR migrates the repository’s test projects to Microsoft Testing Platform (MTP) v2. It replaces the existing VSTest/Coverlet setup while preserving TRX test reporting and Cobertura code coverage for local development and CI.
No product or component runtime behavior is changed.
Notes:
Before running the
_StartCodeCoverage.ps1script, you must delete the files in the foldertests\TestResults\*