Use .NET Test SDK instead of regular one

- This adds props and targets from "Microsoft.NET.Test.Sdk". - Detect if using NuGet's Central Package versioning. - Fix some mistakes in the NuGet project files.
nirin · Dec 2, 2021 · c25ee51 · c25ee51
1 parent 78c5c6e
commit c25ee51
Show file tree

Hide file tree

Showing 9 changed files with 10 additions and 37 deletions.
diff --git a/Sources/NuGet.Build.Sdk/Extensions/Directory.Packages.props b/Sources/NuGet.Build.Sdk/Extensions/Directory.Packages.props
@@ -18,6 +18,7 @@
 	<Import Project="$(DirectoryPackagesProps)" Condition="Exists('$(DirectoryPackagesProps)')"/>
 
 	<PropertyGroup Condition="Exists('$(DirectoryPackagesProps)')">
+		<ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
 		<CentralPackageVersionsFileImported>true</CentralPackageVersionsFileImported>
 	</PropertyGroup>
 

diff --git a/Sources/NuGet.Build.Sdk/NuGet.Build.Sdk.nuproj b/Sources/NuGet.Build.Sdk/NuGet.Build.Sdk.nuproj
@@ -11,7 +11,7 @@
 		<PackageTags>$(PackageTags);nuget;build;pack;package;restore;dependency;reference</PackageTags>
 		<Summary>An MSBuild SDK package that provides package reference support and dependency management to the projects.</Summary>
 		<Description>$(Summary) It adds a `Restore` target which facilitates downloading and referencing shared libraries from a remote server.</Description>
-		<Description>$(Summary) It also adds a `Pack` target which facilitates creation of such packages using project outputs with pre-determined layouts.</Description>
+		<Description>$(Description) It also adds a `Pack` target which facilitates creation of such packages using project outputs with pre-determined layouts.</Description>
 		<Description>$(Description) It can be used either directly on your project (_via the `<Sdk/>` element_) or in-directly (_via other SDKs which refer to it internally_).</Description>
 	</PropertyGroup>
 

diff --git a/Sources/NuGet.Packaging.Sdk/NuGet.Packaging.Sdk.nuproj b/Sources/NuGet.Packaging.Sdk/NuGet.Packaging.Sdk.nuproj
@@ -8,7 +8,7 @@
 		<Title>NuGet Packaging Sdk</Title>
 		<PackageId>NuGet.Packaging.Sdk</PackageId>
 		<PackageVersion Condition="'$(PackageVersion)' == ''">1.0.0</PackageVersion>
-		<PackageTags>$(PackageTags);nuget;build;packaging;project;sysytem;nuproj;pkgproj;sfxproj</PackageTags>
+		<PackageTags>$(PackageTags);nuget;build;packaging;project;system;nuproj;pkgproj;sfxproj</PackageTags>
 		<Summary>An MSBuild SDK package that provides Project System support for NuGet package authoring.</Summary>
 		<Description>$(Summary) Having a separate Project System makes it possible to author complex NuGet packages, otherwise not possible with Pack targets.</Description>
 		<Description>$(Description) One such need is to create a completely customized package with a custom layout that is different from the standard layout.</Description>

diff --git a/Tests/Directory.Build.props b/Tests/Directory.Build.props
@@ -2,36 +2,12 @@
 
 	<Import Project="..\$(MSBuildThisFile)"/>
 
-	<PropertyGroup Label="Build">
-		<NoWarn>NU1507</NoWarn>
-		<ManagePackageVersionsCentrally>True</ManagePackageVersionsCentrally>
-		<RestoreUseStaticGraphEvaluation>True</RestoreUseStaticGraphEvaluation>
-	</PropertyGroup>
-
-	<PropertyGroup Label="Compiler">
-		<Features>Strict</Features>
-		<Nullable>Enable</Nullable>
-		<LangVersion>Latest</LangVersion>
-	</PropertyGroup>
-
 	<Choose>
 		<When Condition="$(MSBuildProjectName.EndsWith('Tests'))">
-			<PropertyGroup>
-				<IsPackable>false</IsPackable>
-			</PropertyGroup>
-
 			<ItemGroup>
 				<ProjectReference Include="..\MSBuild.TestFramework\MSBuild.TestFramework.csproj" />
 			</ItemGroup>
-
-			<ItemGroup>
-				<PackageReference Include="MSTest.TestAdapter" />
-				<PackageReference Include="MSTest.TestFramework" />
-				<PackageReference Include="Microsoft.NET.Test.Sdk" />
-			</ItemGroup>
 		</When>
-
-		<Otherwise/>
 	</Choose>
 
 </Project>
diff --git a/Tests/Directory.Build.targets b/Tests/Directory.Build.targets
@@ -1,8 +1,7 @@
 <Project>
 
 	<PropertyGroup Condition="'$(IsTestProject)' == 'true' AND '$(TargetFrameworkIdentifier)' == '.NETCoreApp'">
-		<GenerateProgramFile>false</GenerateProgramFile>
-		<GeneratedProgramFile>Program$(DefaultLanguageSourceExtension)</GeneratedProgramFile>
+		<EnableDefaultProgramFile>false</EnableDefaultProgramFile>
 	</PropertyGroup>
 
 	<Import Project="..\$(MSBuildThisFile)"/>

diff --git a/Tests/Directory.Packages.props b/Tests/Directory.Packages.props
@@ -1,7 +1,10 @@
 <Project>
 
 	<PropertyGroup Label="Core">
+		<!-- .NET Standard -->
 		<NETStandardImplicitPackageVersion>2.0.3</NETStandardImplicitPackageVersion>
+		<!-- MSTest -->
+		<MSTestImplicitPackageVersion>2.2.8</MSTestImplicitPackageVersion>
 	</PropertyGroup>
 
 	<ItemGroup Label="Core">
@@ -11,11 +14,4 @@
 		<PackageVersion Version="17.0.0" Include="Microsoft.Build.Utilities.Core" />
 	</ItemGroup>
 
-	<ItemGroup Label="Test">
-		<!-- MSTest -->
-		<PackageVersion Version="2.2.8"  Include="MSTest.TestAdapter" />
-		<PackageVersion Version="2.2.8"  Include="MSTest.TestFramework" />
-		<PackageVersion Version="17.0.0" Include="Microsoft.NET.Test.Sdk" />
-	</ItemGroup>
-
 </Project>
diff --git a/Tests/MSBuild.Core.Tests/MSBuild.Core.Tests.csproj b/Tests/MSBuild.Core.Tests/MSBuild.Core.Tests.csproj
@@ -1,4 +1,4 @@
-<Project Sdk="MSBuild.NET.Sdk">
+<Project Sdk="MSBuild.NET.Test.Sdk">
 
 	<PropertyGroup>
 		<TargetFrameworks>net6.0;net472</TargetFrameworks>

diff --git a/Tests/MSBuild.NET.Tests/MSBuild.NET.Tests.csproj b/Tests/MSBuild.NET.Tests/MSBuild.NET.Tests.csproj
@@ -1,4 +1,4 @@
-<Project Sdk="MSBuild.NET.Sdk">
+<Project Sdk="MSBuild.NET.Test.Sdk">
 
 	<PropertyGroup>
 		<TargetFrameworks>net6.0;net472</TargetFrameworks>

diff --git a/global.json b/global.json
@@ -6,6 +6,7 @@
 		"MSBuild.Solution.Sdk": "1.0.0-dev.*",
 
 		"MSBuild.NET.Sdk": "1.0.0-dev.*",
+		"MSBuild.NET.Test.Sdk": "1.0.0-dev.*",
 		"MSBuild.NET.Legacy.Sdk": "1.0.0-dev.*",
 
 		"NuGet.Build.Sdk": "1.0.0-dev.*",