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

Rename Package MsTest.Sdk to Microsoft.NET.Sdk.Test #2693

Open
Varorbc opened this issue Apr 11, 2024 · 4 comments
Open

Rename Package MsTest.Sdk to Microsoft.NET.Sdk.Test #2693

Varorbc opened this issue Apr 11, 2024 · 4 comments

Comments

@Varorbc
Copy link
Contributor

Varorbc commented Apr 11, 2024

Summary

https://www.nuget.org/packages/MSTest.Sdk

Microsoft provides most of the package name is the full name, I hope the package name is also the full name, so that users feel is a system, Not an alternative.Just like the following names

Microsoft.NET.Sdk
Microsoft.NET.Sdk.Web
Microsoft.NET.Sdk.Worker
Microsoft.NET.Sdk.BlazorWebAssembly

Background and Motivation

Proposed Feature

Alternative Designs

@Varorbc Varorbc changed the title Rename Package MsTest.Sdk to Microsoft.Test.sdk Rename Package MsTest.Sdk to Microsoft.NET.Sdk.Test Apr 12, 2024
@MarcoRossignoli
Copy link
Contributor

I think there's some confusion around the concept of MSBuild SDK, MSTest.Sdk and dotnet SDK (dotnet build etc...)

MSBuild SDK: This is an MSBuild feature that let you ship an MSBuild(targets/props) as custom MSBuild sdk like explained here https://learn.microsoft.com/en-us/visualstudio/msbuild/how-to-use-project-sdk?view=vs-2022
MSTest.Sdk: is what we shipped here https://learn.microsoft.com/en-us/dotnet/core/testing/unit-testing-mstest-sdk so it's the MSTest custom MSBuild sdk.
dotnet SDKs: some MSBuild sdks are shipped inside the official dotnet SDK and are documented here https://learn.microsoft.com/en-us/dotnet/core/project-sdk/overview

So use a generic full name Microsoft.NET.Sdk.Testing is confusing because

  • every different test adapter has got different versioning and different set of packages. What if you want to use xUnit or nUnit or something else? The sdk name is too generic.
  • the MSTest.Sdk is not inserted inside the sdk but it's resolved through nuget and so doesn't work like the other "inserted" MSBuild sdk but you have always to specify the version inside the csproj or global.json.

@Nirmal4G
Copy link

comment from dotnet/sdk#25797 (comment)

I thought Microsoft was good at naming. You guys disappoint me. You have loads of names at your disposal. Come on! Pick a good one that is simple and is a one word change to the standard SDK Name.

Some Examples:

  • Microsoft.Test.Sdk
  • Microsoft.Testing.Sdk
  • Microsoft.NET.Test.Sdk
  • Microsoft.NET.Sdk.Test
  • Microsoft.NET.Sdk.Testing
  • Microsoft.NET.Testing.Sdk

Pick one from above that seems intuitive to users who are already familiar with existing packages. Please do note that Microsoft.NET.Test.Sdk package can only be used as a package reference. So, why not convert to an SDK, ship in-band with the .NET SDK and deprecate the package reference functionality after v18?

@MarcoRossignoli
Copy link
Contributor

@Nirmal4G thanks for the suggestion, here I've explained because at the moment use some of that names could be problematic for future evolution #2693 (comment)

I think that MSBuild SDK for testing make sense if at the end the usage allows an "empty" csproj like https://learn.microsoft.com/en-us/dotnet/core/testing/unit-testing-mstest-sdk otherwise simply move 1 reference from the user project file to the sdk won't help a lot with the transparent evolution of testing frameworks.

@martincostello
Copy link

As it stands, the suggestions in this comment #2693 (comment) don't seem to make it specific that the SDK is specifically for MSTest. From the blog posts so far, it doesn't seem like the new test SDK is intended to support NUnit, xunit etc. as they aren't neccessarily compatible with things like native AoT.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants