Skip to content

Follow-up on test platforms docs refactoring#51978

Open
Evangelink wants to merge 8 commits intomainfrom
fix/pr-51854-review-feedback
Open

Follow-up on test platforms docs refactoring#51978
Evangelink wants to merge 8 commits intomainfrom
fix/pr-51854-review-feedback

Conversation

Copilot AI review requested due to automatic review settings March 3, 2026 14:10
@dotnetrepoman dotnetrepoman bot added this to the March 2026 milestone Mar 3, 2026
@Evangelink Evangelink enabled auto-merge (squash) March 3, 2026 14:10
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Follow-up documentation refinements to the .NET testing platform content, focusing on clarifying platform selection guidance and updating Microsoft.Testing.Platform (MTP) reference pages after the earlier refactor in #51854.

Changes:

  • Update platform selection guidance (including new scenarios and updated dotnet test notes).
  • Refresh MTP docs (retry API call shape, TRX report details, exit code list, and various wording/title improvements).
  • Adjust MTP architecture docs to reflect updated guidance and API surface descriptions.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
docs/core/testing/test-platforms-overview.md Updates guidance on choosing VSTest vs MTP, including new scenarios and revised mixing/dotnet-test messaging.
docs/core/testing/microsoft-testing-platform-troubleshooting.md Updates the documented exit code list (marks exit code 5 as no longer produced).
docs/core/testing/microsoft-testing-platform-test-reports.md Adds guidance about manual TRX provider registration order and updates the default TRX filename format description.
docs/core/testing/microsoft-testing-platform-retry.md Updates manual registration example to use builder.AddRetryProvider().
docs/core/testing/microsoft-testing-platform-features.md Refines scenario navigation text (telemetry phrasing).
docs/core/testing/microsoft-testing-platform-fakes.md Updates article title metadata for clarity.
docs/core/testing/microsoft-testing-platform-extensions-vstest-bridge.md Rewords the intro to better position the bridge extension’s purpose.
docs/core/testing/microsoft-testing-platform-architecture-test-framework.md Updates architecture guidance and type descriptions for ITestFramework and related APIs.
docs/core/testing/microsoft-testing-platform-architecture-extensions.md Clarifies the page title and section naming for extension points.
Comments suppressed due to low confidence (1)

docs/core/testing/test-platforms-overview.md:50

  • The dotnet test row for MTP omits the key pre-.NET 10 behavior (VSTest mode via TestingPlatformDotnetTestSupport plus the extra -- separator), which is still documented elsewhere (for example, migrating-vstest-microsoft-testing-platform.md and unit-testing-with-dotnet-test.md). Consider adding a brief “In .NET 9 SDK and earlier…” note here (or explicitly defer to the linked unit-testing-with-dotnet-test.md) to avoid misleading readers on older SDKs.
| `dotnet test` behavior | Default VSTest mode. VSTest arguments and behavior apply. | Native MTP mode is available in .NET 10 SDK and later. |

Evangelink and others added 2 commits March 3, 2026 16:17
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Evangelink and others added 4 commits March 4, 2026 10:17
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