Skip to content

Conversation

@Evangelink
Copy link
Member

@Evangelink Evangelink commented Nov 22, 2024

Summary

Fixes #43710


Internal previews

📄 File 🔗 Preview link
docs/core/testing/unit-testing-mstest-sdk.md MSTest SDK overview

@Evangelink Evangelink requested review from a team and IEvangelist as code owners November 22, 2024 20:37
@dotnetrepoman dotnetrepoman bot added this to the November 2024 milestone Nov 22, 2024
@Evangelink
Copy link
Member Author

Ready for review @IEvangelist

Copy link
Contributor

@adegeo adegeo left a comment

Choose a reason for hiding this comment

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

I cleaned up the table, but the preview of this doesn't really look great. Also, there needs to be an introduction to the table, otherwise it's just kind of plopped there after a code block and it doesn't really make sense.


image

What about listing them in bullets?

  • None - No extensions are enabled.

  • Default - Enables the recommended extensions for this version of MSTest.SDK. This is the default when the property isn't set explicitly.

    Enables the following extensions:

    • Code Coverage
    • Trx
  • AllMicrosoft - Enable all extensions shipped by Microsoft (including extensions with a restrictive license).

    Enables the following extensions:

    • Code Coverage / Crash Dump / Fakes (MSTest.Sdk 3.7.0+)
    • Hang Dump / Hot Reload / Retry
    • Trx

@Evangelink
Copy link
Member Author

@adegeo Thanks for the suggestion, I have reworked it. Please review.

@adegeo adegeo merged commit 3972bd0 into dotnet:main Dec 2, 2024
8 checks passed
@Evangelink Evangelink deleted the mstest-sdk-profile branch December 5, 2024 12:29
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.

List of Extensions included when setting the Flag "AllMicrosoft" is missing or not accessible.

4 participants