fix(dev): Actually report code coverage (#231)

* chore: Update test projects to use MSTest.Sdk and remove unnecessary package references

* ci: Update test command in GitHub Actions to use MSTest.Sdk

* ci: Enhance test command in GitHub Actions to include Cobertura format and convert results to JUnit

* ci: Add trx2junit for GitHub Actions build

Author: hangy

.config/dotnet-tools.json

@@ -0,0 +1,13 @@
+{
+  "version": 1,
+  "isRoot": true,
+  "tools": {
+    "trx2junit": {
+      "version": "2.1.0",
+      "commands": [
+        "trx2junit"
+      ],
+      "rollForward": false
+    }
+  }
+}

.github/workflows/dotnet.yml

@@ -26,19 +26,26 @@ jobs:
         uses: actions/setup-dotnet@v4
         with:
           global-json-file: global.json
+      - name: Restore tools
+        run: dotnet tool restore
       - name: Restore dependencies
         run: dotnet restore --locked-mode
       - name: Build
         run: dotnet build --no-restore
       - name: Test
-        run: dotnet test --no-build --verbosity normal --collect:"XPlat Code Coverage"
+        run: dotnet test --no-build --verbosity normal -p:TestingPlatformCommandLineArguments="--report-trx --coverage --coverage-output-format cobertura --coverage-output coverage.cobertura.xml"
+      - name: Convert test results
+        if: ${{ !cancelled() }}
+        run: |
+          find . -name "*.trx" -exec dotnet tool run trx2junit --output TestResults/JUnit {} +
       - name: Upload coverage reports to Codecov
         uses: codecov/codecov-action@v5.1.2
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
-          slug: hangy/AntiXss/hangy/AntiXss
+          slug: hangy/AntiXss
       - name: Upload test results to Codecov
         if: ${{ !cancelled() }}
         uses: codecov/test-results-action@v1
         with:
+          files: TestResults/JUnit/*.xml
           token: ${{ secrets.CODECOV_TOKEN }}

Microsoft.Security.Application.Encoder.UnitTests/Microsoft.Security.Application.Encoder.UnitTests.csproj

@@ -1,19 +1,13 @@
-<Project Sdk="Microsoft.NET.Sdk">
+<Project Sdk="MSTest.Sdk">
   <PropertyGroup>
     <TargetFramework>net9.0</TargetFramework>
     <RootNamespace>Microsoft.Security.Application.Tests</RootNamespace>
     <AssemblyTitle>AntiXSS Unit Tests</AssemblyTitle>
     <AssemblyDescription>Unit Tests for the AntiXSS Library</AssemblyDescription>
-    <IsTestProject>true</IsTestProject>
+    <TestingExtensionsProfile>AllMicrosoft</TestingExtensionsProfile>
   </PropertyGroup>
 
   <ItemGroup>
     <ProjectReference Include="..\Microsoft.Security.Application.Encoder\Microsoft.Security.Application.Encoder.csproj" />
   </ItemGroup>
-
-  <ItemGroup>
-    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.13.0" />
-    <PackageReference Include="MSTest.TestAdapter" Version="3.8.2" />
-    <PackageReference Include="MSTest.TestFramework" Version="3.8.2" />
-  </ItemGroup>
 </Project>

Microsoft.Security.Application.HtmlSanitization.Tests/Microsoft.Security.Application.HtmlSanitization.Tests.csproj

@@ -1,18 +1,12 @@
-<Project Sdk="Microsoft.NET.Sdk">
+<Project Sdk="MSTest.Sdk">
   <PropertyGroup>
     <TargetFramework>net9.0</TargetFramework>
     <RootNamespace>Microsoft.Security.Application.HtmlSanitization.Tests</RootNamespace>
     <AssemblyName>Microsoft.Security.Application.HtmlSanitization.Tests</AssemblyName>
-    <IsTestProject>true</IsTestProject>
+    <TestingExtensionsProfile>AllMicrosoft</TestingExtensionsProfile>
   </PropertyGroup>
 
   <ItemGroup>
     <ProjectReference Include="..\Microsoft.Security.Application.HtmlSanitization\Microsoft.Security.Application.HtmlSanitization.csproj" />
   </ItemGroup>
-
-  <ItemGroup>
-    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.13.0" />
-    <PackageReference Include="MSTest.TestAdapter" Version="3.8.2" />
-    <PackageReference Include="MSTest.TestFramework" Version="3.8.2" />
-  </ItemGroup>
 </Project>

global.json

@@ -2,5 +2,8 @@
   "sdk": {
     "version": "9.0.200",
     "rollForward": "latestFeature"
+  },
+  "msbuild-sdks": {
+    "MSTest.Sdk": "3.8.2"
   }
 }