Use dotnet test reporter to see coverage after build

.github/workflows/build.yml

@@ -31,4 +31,32 @@ jobs:
       - name: Build
         run: dotnet build --no-restore
       - name: Test
-        run: dotnet test --no-build --verbosity normal
+        id: test
+        run: dotnet test --no-build --logger "trx;LogFileName=TestResults.trx" --collect "XPlat Code Coverage"
+        continue-on-error: true
+      - name: Find .trx
+        id: trx
+        run: |
+          $path = (ls -Recurse -Filter *.trx | select -First 1).FullName
+          "path=$path" >> $env:GITHUB_OUTPUT
+      - name: Find coverage*.xml
+        id: coverage
+        run: |
+          $path = (ls -Recurse -Filter coverage*.xml | select -First 1).FullName
+          "path=$path" >> $env:GITHUB_OUTPUT
+      - name: Test report
+        if: ${{ !cancelled() }}
+        uses: bibipkins/[email protected]
+        env:
+          TRX_PATH: ${{ steps.trx.outputs.path }}
+          COVERAGE_PATH: ${{ steps.coverage.outputs.path }}
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          comment-title: Test results
+          results-path: ${{ env.TRX_PATH }}
+          coverage-type: cobertura
+          coverage-path: ${{ env.COVERAGE_PATH }}
+          coverage-threshold: 80
+      - name: Fail if any step has failed
+        if: ${{ failure() }}
+        run: exit 1

DisplayDetective.Common.targets

@@ -5,7 +5,7 @@
   </Target>
 
   <Target Name="CleanBinObj" AfterTargets="Clean">
-    <Message Importance="high" Text="Deleting bin/ and obj/ directories" />
-    <RemoveDir Directories="obj;bin" />
+    <Message Importance="high" Text="Deleting bin/, obj/ and TestResults/ directories" />
+    <RemoveDir Directories="obj;bin;TestResults" />
   </Target>
 </Project>

DisplayDetective.Library.Tests/Common/DisplayDetectiveServiceTests.cs

@@ -13,15 +13,15 @@ public class DisplayDetectiveServiceTests
 {
     private static IEnumerable<TheoryDataRow<string>> GetConfigFiles(string subDirectory)
     {
-        var directory = Path.Combine("testsettings", subDirectory);
+        var directory = Path.Combine("TestSettings", subDirectory);
         return Directory.GetFiles(directory, "*.json").Select(f => new TheoryDataRow<string>(f));
     }
 
     public static IEnumerable<TheoryDataRow<string>> GetGoodConfigFiles() => GetConfigFiles("good");
     public static IEnumerable<TheoryDataRow<string>> GetBadConfigFiles() => GetConfigFiles("bad");
 
     private static IConfiguration GetGoodConfig(string fileName) => new ConfigurationBuilder()
-        .AddJsonFile(Path.Combine("testsettings", "good", fileName + ".json"))
+        .AddJsonFile(Path.Combine("TestSettings", "good", fileName + ".json"))
         .Build();
 
     private static IConfiguration GoodConfiguration => GetGoodConfig("both-commands-with-args");
@@ -45,7 +45,7 @@ public void Ctor_InvalidConfig_Fails()
         Assert.Throws<InvalidDataException>(() => new DisplayDetectiveService(
             Mock.Of<ILogger<DisplayDetectiveService>>(),
             new ConfigurationBuilder()
-                .AddJsonFile(Path.Combine("testsettings", "invalid.json"))
+                .AddJsonFile(Path.Combine("TestSettings", "invalid.json"))
                 .Build(),
             Mock.Of<IDisplayMonitorService>(),
             Mock.Of<ICommandRunnerService>()));

DisplayDetective.Library.Tests/DisplayDetective.Library.Tests.csproj

@@ -10,17 +10,14 @@
     <Nullable>enable</Nullable>
     <WarningsAsErrors>nullable</WarningsAsErrors>
     <IsPackable>false</IsPackable>
-    <IsTestProject>true</IsTestProject>
-    <TestingPlatformDotnetTestSupport>true</TestingPlatformDotnetTestSupport>
-    <UseMicrosoftTestingPlatformRunner>true</UseMicrosoftTestingPlatformRunner>
     <RestorePackagesWithLockFile>true</RestorePackagesWithLockFile>
     <PrunePackageReference>true</PrunePackageReference>
     <DisableImplicitNuGetFallbackFolder>true</DisableImplicitNuGetFallbackFolder>
   </PropertyGroup>
 
   <ItemGroup>
     <Content Include="xunit.runner.json" CopyToOutputDirectory="PreserveNewest" />
-    <Content Include="testsettings\**\*.json" CopyToOutputDirectory="PreserveNewest" />
+    <Content Include="TestSettings\**\*.json" CopyToOutputDirectory="PreserveNewest" />
   </ItemGroup>
 
   <ItemGroup>
@@ -34,14 +31,8 @@
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.13.0" />
     <PackageReference Include="Moq" Version="4.20.72" />
     <PackageReference Include="xunit.v3" Version="1.1.0" />
-    <PackageReference Include="xunit.runner.visualstudio" Version="3.0.2">
-      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
-      <PrivateAssets>all</PrivateAssets>
-    </PackageReference>
-    <PackageReference Include="coverlet.collector" Version="6.0.4">
-      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
-      <PrivateAssets>all</PrivateAssets>
-    </PackageReference>
+    <PackageReference Include="xunit.runner.visualstudio" Version="3.0.2" />
+    <PackageReference Include="coverlet.collector" Version="6.0.4" />
   </ItemGroup>
 
   <Import Project="..\DisplayDetective.Common.targets" />

README.md

@@ -52,6 +52,7 @@ dotnet test
 cd DisplayDetective.CommandLineApp
 dotnet run -- list
 dotnet run -- monitor
+dotnet msbuild /t:clean
 ```
 
 ## 🤝 Contributing