Skip to content

Add more MSTest properties #42544

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

Closed
6 tasks done
gewarren opened this issue Sep 9, 2024 · 2 comments · Fixed by #45767
Closed
6 tasks done

Add more MSTest properties #42544

gewarren opened this issue Sep 9, 2024 · 2 comments · Fixed by #45767
Assignees
Labels
📌 seQUESTered Identifies that an issue has been imported into Quest. resolved-by-customer

Comments

@gewarren
Copy link
Contributor

gewarren commented Sep 9, 2024

  • UseWinUI to set to true when testing WinUI UI components. (MSTest)
  • EnableMSTestV2CopyResources (default value is true) that allows to copy resources dll to the output (MSTest)
  • xUnit is finishing its onboarding so there will be an equivalent property [to EnableNUnitRunner]. I have double checked and xUnit v3+ will be using UseMicrosoftTestingPlatformRunner
  • I would also add DisableTestingPlatformServerCapability that removes the project capability used by VS and VS Code Test Explorers to know which platform protocol to use (VSTest or Microsoft.Testing.Platform). Note that at the moment, the Microsoft.Testing.Platform is not yet available publicly but will be soon.
  • GenerateTestingPlatformConfigurationFile is only available when IsTestingPlatformApplication is true and is used to allow the copy and rename of the config file in the output folder.
  • I am wondering if it would make sense to group the properties by platform, VSTest and Microsoft.Testing.Platform, to make it easy to understand which properties are available in which mode.

Originally posted by @Evangelink in #42404 (comment)


Document Details

Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.


Associated WorkItem - 417525

@gewarren
Copy link
Contributor Author

@Youssef1313 Any interest in taking this one?

@Youssef1313
Copy link
Member

@gewarren Sorry I didn't notice this one earlier but came across it by chance today.

  • I personally don't feel like UseWinUI is something to be documented for MSTest.

  • I also don't know under what scenario a user will want to set EnableMSTestV2CopyResources to false. So can't say much about this one.

  • UseMicrosoftTestingPlatformRunner is briefly mentioned here. But I think we could expand it yes.

  • For DisableTestingPlatformServerCapability, my understanding is that it's mostly used as a temporary workaround for Test Explorer issues. So I'm not really sure whether or not it should be documented.

  • So far I never seen a scenario where a user will want to set GenerateTestingPlatformConfigurationFile to false, but it could still be reasonable to document.

  • For grouping properties by testing platform (VSTest/MTP), I totally agree. I have already seen cases where there was confusion about GenerateProgramFile property (which is VSTest-specific), and some users were expecting it to have an effect for MTP.

@dotnetrepoman dotnetrepoman bot added the 🗺️ mapQUEST Only used as a way to mark an issue as updated for quest. RepoMan should instantly remove it. label Apr 3, 2025
@gewarren gewarren added the 🗺️ reQUEST Triggers an issue to be imported into Quest. label Apr 3, 2025
@dotnet-policy-service dotnet-policy-service bot removed 🗺️ mapQUEST Only used as a way to mark an issue as updated for quest. RepoMan should instantly remove it. ⌚ Not Triaged Not triaged labels Apr 3, 2025
@sequestor sequestor bot added 📌 seQUESTered Identifies that an issue has been imported into Quest. and removed 🗺️ reQUEST Triggers an issue to be imported into Quest. labels Apr 4, 2025
@gewarren gewarren moved this from 🔖 Ready to 👀 In review in dotnet/docs April 2025 sprint project Apr 9, 2025
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in dotnet/docs April 2025 sprint project Apr 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📌 seQUESTered Identifies that an issue has been imported into Quest. resolved-by-customer
Projects
Development

Successfully merging a pull request may close this issue.

4 participants