Skip to content

Conversation

mariam-abdulla
Copy link
Member

@mariam-abdulla mariam-abdulla commented Sep 8, 2025

This pull request updates the documentation for the dotnet test command to clarify how to enable the Microsoft Testing Platform (MTP) test runner. The instructions now reference global.json instead of the deprecated dotnet.config, and the example configurations have been updated accordingly.

Documentation updates for test runner configuration:

  • Updated the description to specify that enabling MTP is now done by setting the test runner in the global.json file, rather than using a dotnet.config file.
  • Replaced all example configurations from INI-style dotnet.config to JSON-style global.json format.
  • Updated warning messages to reference opting in via global.json instead of dotnet.config.

Internal previews

📄 File 🔗 Preview link
docs/core/testing/unit-testing-with-dotnet-test.md Testing with 'dotnet test'
docs/core/tools/dotnet-test.md dotnet test command
docs/core/whats-new/dotnet-10/sdk.md What's new in the SDK and tooling for .NET 10

@mariam-abdulla mariam-abdulla requested review from adegeo and a team as code owners September 8, 2025 07:05
@dotnetrepoman dotnetrepoman bot added this to the September 2025 milestone Sep 8, 2025
@dotnet-policy-service dotnet-policy-service bot added dotnet-fundamentals/svc dotnet-cli/subsvc community-contribution Indicates PR is created by someone from the .NET community. labels Sep 8, 2025
Copy link
Member

@Youssef1313 Youssef1313 left a comment

Choose a reason for hiding this comment

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

LGTM, but blocking merge until dotnet/sdk#50673 is merged.

@Youssef1313 Youssef1313 added 🚧 Hold for related PR Indicates a PR can only be merged when other related PRs are merged (see comments for links) and removed community-contribution Indicates PR is created by someone from the .NET community. labels Sep 8, 2025
Copy link
Member

@Youssef1313 Youssef1313 left a comment

Choose a reason for hiding this comment

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

@mariam-abdulla Please make sure to update the other pages that mention dotnet.config.

  • docs/core/whats-new/dotnet-10/sdk.md
  • docs/core/testing/unit-testing-with-dotnet-test.md

@mariam-abdulla
Copy link
Member Author

@mariam-abdulla Please make sure to update the other pages that mention dotnet.config.

  • docs/core/whats-new/dotnet-10/sdk.md
  • docs/core/testing/unit-testing-with-dotnet-test.md

PR updated

@Youssef1313 Youssef1313 removed the 🚧 Hold for related PR Indicates a PR can only be merged when other related PRs are merged (see comments for links) label Sep 11, 2025
```ini
[dotnet.test.runner]
name = "Microsoft.Testing.Platform"
To enable this mode, add the following configuration to your `global.json` file:
Copy link
Member

Choose a reason for hiding this comment

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

The other PR is merged.

One small question. Should we actually briefly mention that during early previews and RC1 this was using dotnet.config, which we dropped in favor of global.json in RC2?

I want to avoid confusion when people are trying to use RC1 (because RC2 isn't released yet).

@gewarren Is there generally some policy docs follow when there is a breaking change between RC1 and RC2 when RC2 isn't yet released?

Copy link
Member

@Youssef1313 Youssef1313 left a comment

Choose a reason for hiding this comment

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

LGTM but needs an answer on the open question, as this is a breaking change between RC1 (released already) and RC2 (not yet released).

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