Migrate from VSTest to MTP for test run

.github/workflows/dotnet.yml

@@ -63,15 +63,18 @@ jobs:
       - name: Build
         run: dotnet build -c ${{ matrix.configuration }} --no-restore
       - name: Test
-        run: dotnet test -c ${{ matrix.configuration }} --no-build --no-restore --verbosity normal --collect:"XPlat Code Coverage" -p:TestTfmsInParallel=true -- /Parallel
+        # --report-gh is not working
+        run: dotnet test -c ${{ matrix.configuration }} --no-build --no-restore --verbosity normal --coverage --coverage-output-format cobertura -p:TestTfmsInParallel=true
       - name: Upload coverage reports to Codecov
         uses: codecov/codecov-action@fb8b3582c8e4def4969c97caa2f19720cb33a72f # v7.0.0
         with:
+          files: ./TestResults/*.cobertura.xml
           flags: ${{ matrix.os }}
           token: ${{ secrets.CODECOV_TOKEN }}
       - name: Upload coverage reports to Codecov
         uses: codecov/codecov-action@fb8b3582c8e4def4969c97caa2f19720cb33a72f # v7.0.0
         with:
+          files: ./TestResults/*.cobertura.xml
           flags: ${{ matrix.configuration }}
           token: ${{ secrets.CODECOV_TOKEN }}
 
@@ -152,7 +155,8 @@ jobs:
       - name: Setup .NET (8.0.x, global.json)
         uses: actions/setup-dotnet@a98b56852c35b8e3190ac28c8c2271da59106c68 # v6.0.0
         with:
-          dotnet-version: 8.0.x
+          dotnet-version: |
+            8.0.x
           global-json-file: global.json
       # No need to define min ver version globally now since only uploading release
       # when version and tag is defined.

global.json

@@ -3,5 +3,8 @@
     "version": "10.0.400",
     "rollForward": "latestPatch",
     "allowPrerelease": false
+  },
+  "test": {
+    "runner": "Microsoft.Testing.Platform"
   }
 }

src/Sep.Test/Sep.Test.csproj

@@ -1,10 +1,13 @@
-<Project Sdk="Microsoft.NET.Sdk">
+<Project Sdk="MSTest.Sdk/4.3.3">
 
   <PropertyGroup>
     <RootNamespace>nietras.SeparatedValues.Test</RootNamespace>
     <AllowUnsafeBlocks>true</AllowUnsafeBlocks>
     <IsPackable>false</IsPackable>
     <NoWarn>$(NoWarn);CA2007</NoWarn>
+
+    <OutputType>Exe</OutputType>
+    <EnableMSTestRunner>true</EnableMSTestRunner>
   </PropertyGroup>
 
   <ItemGroup>
@@ -15,12 +18,4 @@
     <AssemblyAttribute Include="System.Diagnostics.CodeAnalysis.ExcludeFromCodeCoverageAttribute" />
   </ItemGroup>
 
-  <ItemGroup>
-    <PackageReference Include="coverlet.collector" Version="10.0.1">
-      <PrivateAssets>all</PrivateAssets>
-      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
-    </PackageReference>
-    <PackageReference Include="MSTest" Version="4.3.3" />
-  </ItemGroup>
-
 </Project>

src/Sep.XyzTest/Sep.XyzTest.csproj

@@ -1,10 +1,13 @@
-<Project Sdk="Microsoft.NET.Sdk">
+<Project Sdk="MSTest.Sdk/4.3.3">
 
   <PropertyGroup>
     <RootNamespace>nietras.SeparatedValues.XyzTest</RootNamespace>
     <AllowUnsafeBlocks>true</AllowUnsafeBlocks>
     <IsPackable>false</IsPackable>
     <NoWarn>$(NoWarn);CA2007</NoWarn>
+
+    <OutputType>Exe</OutputType>
+    <EnableMSTestRunner>true</EnableMSTestRunner>
   </PropertyGroup>
 
   <ItemGroup>
@@ -16,12 +19,7 @@
   </ItemGroup>
 
   <ItemGroup>
-    <PackageReference Include="coverlet.collector" Version="10.0.1">
-      <PrivateAssets>all</PrivateAssets>
-      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
-    </PackageReference>
     <PackageReference Include="PublicApiGenerator" Version="11.5.4" />
-    <PackageReference Include="MSTest" Version="4.3.3" />
     <PackageReference Include="CsvHelper" Version="33.1.0" />
     <PackageReference Include="Sylvan.Common" Version="0.4.3" />
     <PackageReference Include="Sylvan.Data" Version="0.2.17" />

test-parsers.ps1

@@ -1,7 +1,16 @@
 #!/usr/bin/env pwsh
-Try {
+$ErrorActionPreference = "Stop"
+$PSNativeCommandUseErrorActionPreference = $true
+
+try {
     dotnet build .\src\Sep.Test\Sep.Test.csproj --nologo -c Debug --no-restore
+    if ($LASTEXITCODE -ne 0) {
+        throw "Debug build failed with exit code $LASTEXITCODE."
+    }
     dotnet build .\src\Sep.Test\Sep.Test.csproj --nologo -c Release --no-restore
+    if ($LASTEXITCODE -ne 0) {
+        throw "Release build failed with exit code $LASTEXITCODE."
+    }
 
     $parsers = @(
         "SepParserAdvSimdLoad4xNrwCmpOrBulkMoveMaskTzcnt",
@@ -19,12 +28,18 @@ Try {
     )
 
     foreach ($parser in $parsers) {
-        $env:SEPFORCEPARSER=$parser
+        $env:SEPFORCEPARSER = $parser
         Write-Output "Testing $parser Debug"
-        dotnet test .\src\Sep.Test\Sep.Test.csproj --nologo -c Debug --no-build --no-restore -p:TestTfmsInParallel=true -- /Parallel
+        dotnet test --project .\src\Sep.Test\Sep.Test.csproj -c Debug --no-build --no-restore
+        if ($LASTEXITCODE -ne 0) {
+            throw "Debug parser test failed for $parser with exit code $LASTEXITCODE."
+        }
         Write-Output "Testing $parser Release"
-        dotnet test .\src\Sep.Test\Sep.Test.csproj --nologo -c Release --no-build --no-restore -p:TestTfmsInParallel=true -- /Parallel
+        dotnet test --project .\src\Sep.Test\Sep.Test.csproj -c Release --no-build --no-restore
+        if ($LASTEXITCODE -ne 0) {
+            throw "Release parser test failed for $parser with exit code $LASTEXITCODE."
+        }
     }
-} Finally {
-    Remove-Item env:SEPFORCEPARSER
+} finally {
+    Remove-Item env:SEPFORCEPARSER -ErrorAction SilentlyContinue
 }

test-x64-x86.ps1

@@ -1,9 +1,26 @@
 #!/usr/bin/env pwsh
-Write-Output "Testing Debug X86"
-dotnet test --nologo -c Debug -- RunConfiguration.TargetPlatform=x86 /Parallel
-Write-Output "Testing Release X86"
-dotnet test --nologo -c Release -- RunConfiguration.TargetPlatform=x86 /Parallel
-Write-Output "Testing Debug X64"
-dotnet test --nologo -c Debug -- RunConfiguration.TargetPlatform=x64 /Parallel
-Write-Output "Testing Release X64"
-dotnet test --nologo -c Release --collect:"XPlat Code Coverage" -- RunConfiguration.TargetPlatform=x64 /Parallel
+$ErrorActionPreference = "Stop"
+$PSNativeCommandUseErrorActionPreference = $true
+
+$testProjects = @(
+    ".\src\Sep.Test\Sep.Test.csproj"
+    ".\src\Sep.XyzTest\Sep.XyzTest.csproj"
+)
+$configurations = @(
+    "Debug"
+    "Release"
+)
+$architectures = @(
+    "x86"
+    "x64"
+)
+
+foreach ($architecture in $architectures) {
+    foreach ($configuration in $configurations) {
+        Write-Output "Testing $configuration $($architecture.ToUpperInvariant())"
+
+        foreach ($project in $testProjects) {
+            dotnet test --project $project -c $configuration --arch $architecture
+        }
+    }
+}