Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MSTest SDK list extensions per profile #43715

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

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
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.

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