ci: add test results and coverage reporting to PR comments (#153)

## Summary
- Add `dotnet-test-reporter` to CI workflow to post test results and
code coverage as a PR comment
- Collect XPlat Code Coverage (Cobertura) and TRX test results from both
test projects
- Gate the report to the Ubuntu matrix leg to avoid duplicate comments

## Test plan
- [x] Verify PR comment appears with test results and coverage after CI
runs
- [x] Confirm coverage threshold of 70% is enforced

Author: fuzzzerd

.github/workflows/ci.yml

@@ -1,6 +1,7 @@
 name: CI Builds
 permissions:
   contents: read
+  pull-requests: write
 
 on:
   pull_request:
@@ -36,7 +37,26 @@ jobs:
         run: dotnet build --no-restore
 
       - name: Test
-        run: dotnet test --no-build 
+        run: dotnet test --no-build --verbosity normal --logger trx --collect:"XPlat Code Coverage"
+
+      - name: Merge coverage reports
+        if: ${{ always() && matrix.os == 'ubuntu-latest' }}
+        uses: danielpalme/ReportGenerator-GitHub-Action@5
+        with:
+          reports: tests/**/coverage.cobertura.xml
+          targetdir: coverage
+          reporttypes: Cobertura
+
+      - name: Generate test report
+        uses: bibipkins/dotnet-test-reporter@v1.6.1
+        if: ${{ always() && matrix.os == 'ubuntu-latest' }}
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          comment-title: "Test Results"
+          results-path: tests/**/*.trx
+          coverage-path: coverage/Cobertura.xml
+          coverage-type: cobertura
+          coverage-threshold: 70
 
       - name: Publish
         run: dotnet publish src/SharpFM/SharpFM.csproj --runtime "${{ matrix.target }}" -c Debug

src/SharpFM.Plugin.XmlViewer/XmlViewerPanel.axaml.cs

@@ -1,10 +1,12 @@
+using System.Diagnostics.CodeAnalysis;
 using Avalonia.Controls;
 using AvaloniaEdit;
 using AvaloniaEdit.TextMate;
 using TextMateSharp.Grammars;
 
 namespace SharpFM.Plugin.XmlViewer;
 
+[ExcludeFromCodeCoverage]
 public partial class XmlViewerPanel : UserControl
 {
     private TextMate.Installation? _textMateInstallation;

src/SharpFM/App.axaml.cs

@@ -1,15 +1,17 @@
+using System;
+using System.Diagnostics.CodeAnalysis;
 using Avalonia;
 using Avalonia.Controls.ApplicationLifetimes;
 using Avalonia.Markup.Xaml;
 using SharpFM.ViewModels;
 using Microsoft.Extensions.Logging;
 using NLog.Extensions.Logging;
 using Microsoft.Extensions.DependencyInjection;
-using System;
 using SharpFM.Services;
 
 namespace SharpFM;
 
+[ExcludeFromCodeCoverage]
 public partial class App : Application
 {
     ILogger logger = LoggerFactory.Create(builder => builder.AddNLog()).CreateLogger<MainWindow>();

src/SharpFM/AvaloniaNLogSink.cs

@@ -1,4 +1,5 @@
 using System;
+using System.Diagnostics.CodeAnalysis;
 using Avalonia;
 using Avalonia.Logging;
 using NLog;
@@ -8,6 +9,7 @@ namespace SharpFM;
 /// <summary>
 /// Avalonia Log Sink that writes to NLog Loggers.
 /// </summary>
+[ExcludeFromCodeCoverage]
 public class AvaloniaNLogSink : ILogSink
 {
     /// <summary>
@@ -43,6 +45,7 @@ private static LogLevel LogLevelToNLogLevel(LogEventLevel level)
     }
 }
 
+[ExcludeFromCodeCoverage]
 public static class NLogSinkExtensions
 {
     /// <summary>

src/SharpFM/MainWindow.axaml.cs

@@ -1,5 +1,6 @@
 using System;
 using System.ComponentModel;
+using System.Diagnostics.CodeAnalysis;
 using Avalonia;
 using Avalonia.Controls;
 using Avalonia.Input;
@@ -14,6 +15,7 @@
 
 namespace SharpFM;
 
+[ExcludeFromCodeCoverage]
 public partial class MainWindow : Window
 {
     private readonly RegistryOptions _registryOptions;

src/SharpFM/Models/ClipRepository.cs

@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
 using System.IO;
 using NLog;
 
@@ -8,6 +9,7 @@ namespace SharpFM.Models;
 /// <summary>
 /// Clip File Repository.
 /// </summary>
+[ExcludeFromCodeCoverage]
 public class ClipRepository
 {
     private static readonly Logger Log = LogManager.GetCurrentClassLogger();

src/SharpFM/PluginManager/PluginManagerWindow.axaml.cs

@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
 using System.Linq;
 using Avalonia.Controls;
 using Avalonia.Platform.Storage;
@@ -9,6 +10,7 @@
 
 namespace SharpFM.PluginManager;
 
+[ExcludeFromCodeCoverage]
 public partial class PluginManagerWindow : Window
 {
     private readonly PluginManagerViewModel _viewModel = new();

src/SharpFM/Program.cs

@@ -1,9 +1,11 @@
-using Avalonia;
+using System;
+using System.Diagnostics.CodeAnalysis;
+using Avalonia;
 using NLog;
-using System;
 
 namespace SharpFM;
 
+[ExcludeFromCodeCoverage]
 class Program
 {
     // Initialization code. Don't use any Avalonia, third-party APIs or any

src/SharpFM/Schema/Editor/CalculationEditorWindow.axaml.cs

@@ -1,3 +1,4 @@
+using System.Diagnostics.CodeAnalysis;
 using Avalonia.Controls;
 using Avalonia.Interactivity;
 using AvaloniaEdit;
@@ -8,6 +9,7 @@
 
 namespace SharpFM.Schema.Editor;
 
+[ExcludeFromCodeCoverage]
 public partial class CalculationEditorWindow : Window
 {
     private readonly FmField _field;

src/SharpFM/Schema/Editor/TableEditorControl.axaml.cs

@@ -1,10 +1,12 @@
 using System;
+using System.Diagnostics.CodeAnalysis;
 using Avalonia.Controls;
 using Avalonia.Input;
 using SharpFM.Schema.Model;
 
 namespace SharpFM.Schema.Editor;
 
+[ExcludeFromCodeCoverage]
 public partial class TableEditorControl : UserControl
 {
     public static FieldDataType[] DataTypes { get; } = Enum.GetValues<FieldDataType>();