Enable dotnet test support for MTP

havit · Feb 20, 2025 · 03c5a49 · 03c5a49
1 parent c3ab500
commit 03c5a49
Show file tree

Hide file tree

Showing 13 changed files with 15 additions and 15 deletions.
diff --git a/.github/workflows/dotnet.yml b/.github/workflows/dotnet.yml
@@ -24,6 +24,6 @@ jobs:
     - name: Restore dependencies
       run: dotnet restore
     - name: Build
-      run: dotnet build --no-restore
+      run: dotnet build --no-restore -p:TreatWarningsAsErrors=true
     - name: Test
       run: dotnet test --no-build --verbosity normal
diff --git a/Directory.Build.props b/Directory.Build.props
@@ -16,6 +16,15 @@
 		<!-- aligned NuGet PackageVersion for package-bundles -->
 		<ComponentsPackagesVersion>4.7.11</ComponentsPackagesVersion>
 		<GrpcPackagesVersion>1.6.1</GrpcPackagesVersion>
+
+		<EnableMSTestRunner>true</EnableMSTestRunner>
+		<TestingPlatformDotnetTestSupport>true</TestingPlatformDotnetTestSupport>
+		<TestingPlatformShowTestsFailure>true</TestingPlatformShowTestsFailure>
+		<MSTestAnalysisMode>Recommended</MSTestAnalysisMode>
+
+		<!-- MSTEST0001: Explicitly enable or disable tests parallelization -->
+		<!-- TODO: Evaluate whether or not test projects are parallelizable or not. -->
+		<NoWarn>$(NoWarn);MSTEST0001</NoWarn>
 	</PropertyGroup>
 
 	<Import Project="$([MSBuild]::GetPathOfFileAbove('Directory.Build.props', '$(MSBuildThisFileDirectory)../'))" Condition="Exists($([MSBuild]::GetPathOfFileAbove('Directory.Build.props', '$(MSBuildThisFileDirectory)../')))"/>

diff --git a/...rameworkCore.Tests/Havit.Blazor.Components.Web.Bootstrap.EntifyFrameworkCore.Tests.csproj b/...rameworkCore.Tests/Havit.Blazor.Components.Web.Bootstrap.EntifyFrameworkCore.Tests.csproj
@@ -4,7 +4,6 @@
 		<TargetFrameworks>net9.0;net8.0</TargetFrameworks>
 		<IsPackable>false</IsPackable>
 		<ImplicitUsings>enable</ImplicitUsings>
-		<EnableMSTestRunner>true</EnableMSTestRunner>
 		<OutputType>Exe</OutputType>
 	</PropertyGroup>
 

diff --git a/Havit.Blazor.Components.Web.Bootstrap.Tests/Forms/Internal/DateHelperTests.cs b/Havit.Blazor.Components.Web.Bootstrap.Tests/Forms/Internal/DateHelperTests.cs
@@ -103,15 +103,15 @@ public void DateHelper_GetValueFromDateTimeOffset()
 
 		// DateTime?
 		Assert.AreEqual(dateTimeOffset.DateTime, DateHelper.GetValueFromDateTimeOffset<DateTime?>(dateTimeOffset));
-		Assert.AreEqual(null, DateHelper.GetValueFromDateTimeOffset<DateTime?>(null));
+		Assert.IsNull(DateHelper.GetValueFromDateTimeOffset<DateTime?>(null));
 
 		// DateTimeOffset
 		Assert.AreEqual(dateTimeOffset, DateHelper.GetValueFromDateTimeOffset<DateTimeOffset>(dateTimeOffset));
 		Assert.AreEqual(default(DateTimeOffset), DateHelper.GetValueFromDateTimeOffset<DateTimeOffset>(null));
 
 		// DateTimeOffset?
 		Assert.AreEqual(dateTimeOffset, DateHelper.GetValueFromDateTimeOffset<DateTimeOffset?>(dateTimeOffset));
-		Assert.AreEqual(null, DateHelper.GetValueFromDateTimeOffset<DateTimeOffset?>(null));
+		Assert.IsNull(DateHelper.GetValueFromDateTimeOffset<DateTimeOffset?>(null));
 	}
 
 	private class Fixture

diff --git a/....Blazor.Components.Web.Bootstrap.Tests/Havit.Blazor.Components.Web.Bootstrap.Tests.csproj b/....Blazor.Components.Web.Bootstrap.Tests/Havit.Blazor.Components.Web.Bootstrap.Tests.csproj
@@ -4,7 +4,6 @@
 	  <TargetFrameworks>net9.0;net8.0</TargetFrameworks>
 	  <IsPackable>false</IsPackable>
 	  <ImplicitUsings>enable</ImplicitUsings>
-	  <EnableMSTestRunner>true</EnableMSTestRunner>
 	  <OutputType>Exe</OutputType>
   </PropertyGroup>
 

diff --git a/Havit.Blazor.Components.Web.Tests/DataStores/DictionaryStaticDataStoreTests.cs b/Havit.Blazor.Components.Web.Tests/DataStores/DictionaryStaticDataStoreTests.cs
@@ -167,7 +167,6 @@ public void DictionaryStaticDataStore_ManyParallelCallsShouldNotReloadData()
 	}
 
 	[TestMethod]
-	[ExpectedException(typeof(InvalidOperationException))]
 	public void DictionaryStaticDataStore_SyncWithThrowIfNotLoaded_ShouldRaiseException()
 	{
 		// arrange
@@ -178,7 +177,7 @@ public void DictionaryStaticDataStore_SyncWithThrowIfNotLoaded_ShouldRaiseExcept
 		sut.Setup(s => s.ShouldRefresh()).Returns(false);
 
 		// act
-		var result = sut.Object.GetAll(throwIfNotLoaded: true);
+		Assert.ThrowsExactly<InvalidOperationException>(() => sut.Object.GetAll(throwIfNotLoaded: true));
 
 		// assert
 		// expected exception

diff --git a/Havit.Blazor.Components.Web.Tests/DataStores/StaticDataStoreTests.cs b/Havit.Blazor.Components.Web.Tests/DataStores/StaticDataStoreTests.cs
@@ -143,7 +143,6 @@ public void StaticDataStore_SyncNotLoaded_ShouldReturnDefault()
 	}
 
 	[TestMethod]
-	[ExpectedException(typeof(InvalidOperationException))]
 	public void StaticDataStore_SyncThrowIfNotLoaded_ShouldRaiseException()
 	{
 		// arrange
@@ -153,7 +152,7 @@ public void StaticDataStore_SyncThrowIfNotLoaded_ShouldRaiseException()
 		sut.Setup(s => s.ShouldRefresh()).Returns(false);
 
 		// act
-		var result = sut.Object.GetValue(throwIfNotLoaded: true);
+		Assert.ThrowsExactly<InvalidOperationException>(() => sut.Object.GetValue(throwIfNotLoaded: true));
 
 		// assert
 		// expected exception

diff --git a/Havit.Blazor.Components.Web.Tests/Havit.Blazor.Components.Web.Tests.csproj b/Havit.Blazor.Components.Web.Tests/Havit.Blazor.Components.Web.Tests.csproj
@@ -4,7 +4,6 @@
 		<TargetFrameworks>net9.0;net8.0</TargetFrameworks>
 		<ImplicitUsings>enable</ImplicitUsings>
 		<IsPackable>false</IsPackable>
-		<EnableMSTestRunner>true</EnableMSTestRunner>
 		<OutputType>Exe</OutputType>
 	</PropertyGroup>
 

diff --git a/Havit.Blazor.Components.Web.Tests/SelectorHelpersTests.cs b/Havit.Blazor.Components.Web.Tests/SelectorHelpersTests.cs
@@ -17,7 +17,7 @@ public void SelectorHelpers_GetText()
 	[TestMethod]
 	public void SelectorHelpers_GetValue()
 	{
-		Assert.AreEqual(null, SelectorHelpers.GetValue<int?, string>(i => "X" + i, null)); // use default
+		Assert.IsNull(SelectorHelpers.GetValue<int?, string>(i => "X" + i, null)); // use default
 		Assert.AreEqual("X1", SelectorHelpers.GetValue<int?, string>(i => "X" + i, 1)); // use value selector
 		Assert.AreEqual(1, SelectorHelpers.GetValue<int?, int?>(null, 1)); // use value selector
 	}

diff --git a/Havit.Blazor.Documentation.Tests/Havit.Blazor.Documentation.Tests.csproj b/Havit.Blazor.Documentation.Tests/Havit.Blazor.Documentation.Tests.csproj
@@ -4,7 +4,6 @@
 		<TargetFramework>net9.0</TargetFramework>
 		<ImplicitUsings>enable</ImplicitUsings>
 		<IsPackable>false</IsPackable>
-		<EnableMSTestRunner>true</EnableMSTestRunner>
 		<OutputType>Exe</OutputType>
 	</PropertyGroup>
 

diff --git a/Havit.Blazor.Grpc.Client.Tests/Havit.Blazor.Grpc.Client.Tests.csproj b/Havit.Blazor.Grpc.Client.Tests/Havit.Blazor.Grpc.Client.Tests.csproj
@@ -4,7 +4,6 @@
 		<TargetFrameworks>net9.0;net8.0</TargetFrameworks>
 		<ImplicitUsings>enable</ImplicitUsings>
 		<IsPackable>false</IsPackable>
-		<EnableMSTestRunner>true</EnableMSTestRunner>
 		<OutputType>Exe</OutputType>
 	</PropertyGroup>
 

diff --git a/Havit.Blazor.Grpc.Core.Tests/Havit.Blazor.Grpc.Core.Tests.csproj b/Havit.Blazor.Grpc.Core.Tests/Havit.Blazor.Grpc.Core.Tests.csproj
@@ -4,7 +4,6 @@
 	  <TargetFrameworks>net9.0;net8.0</TargetFrameworks>
 	  <ImplicitUsings>enable</ImplicitUsings>
 	  <IsPackable>false</IsPackable>
-	  <EnableMSTestRunner>true</EnableMSTestRunner>
 	  <OutputType>Exe</OutputType>
   </PropertyGroup>
 

diff --git a/...ngApiStringLocalizers.Tests/Havit.SourceGenerators.StrongApiStringLocalizers.Tests.csproj b/...ngApiStringLocalizers.Tests/Havit.SourceGenerators.StrongApiStringLocalizers.Tests.csproj
@@ -4,7 +4,6 @@
 		<TargetFrameworks>net9.0</TargetFrameworks>
 		<IsPackable>false</IsPackable>
 		<ImplicitUsings>enable</ImplicitUsings>
-		<EnableMSTestRunner>true</EnableMSTestRunner>
 		<OutputType>Exe</OutputType>
 	</PropertyGroup>