feat!: MTP integration for spekt loggers

- Support command line params `--report-spekt-junit --report-spekt-junit-filename results.xml` or if you'd like to pass custom options `--report-spekt-junit  "StoreConsoleOutput=false"`
- Behavior differences in MTP reporters compared to VSTest loggers
  - Inconclusive tests are now reported as Skipped
  - Run level test messages are not available
  - Xunit: skip reason is not captured
  - Nunit: some properties are reported differently or skipped (seed, category, description)

Author: codito

.claude/settings.local.json

@@ -0,0 +1,11 @@
+{
+  "permissions": {
+    "allow": [
+      "Bash(dotnet:*)",
+      "Bash(find:*)",
+      "Bash(./build.sh:*)"
+    ],
+    "deny": [],
+    "ask": []
+  }
+}

.github/workflows/dotnet.yml

@@ -76,19 +76,25 @@ jobs:
           dotnet test -p:PackageVersion=${{ env.APP_BUILD_VERSION }} test/JUnit.Xml.TestLogger.UnitTests/JUnit.Xml.TestLogger.UnitTests.csproj -p:CollectCoverage=true -p:CoverletOutputFormat=opencover
       - name: Acceptance test
         run: |
-          dotnet test -p:PackageVersion=${{ env.APP_BUILD_VERSION }} test/TestLogger.AcceptanceTests/TestLogger.AcceptanceTests.csproj
-          dotnet test -p:PackageVersion=${{ env.APP_BUILD_VERSION }} test/Xunit.Xml.TestLogger.AcceptanceTests/Xunit.Xml.TestLogger.AcceptanceTests.csproj
-          dotnet test -p:PackageVersion=${{ env.APP_BUILD_VERSION }} test/NUnit.Xml.TestLogger.AcceptanceTests/NUnit.Xml.TestLogger.AcceptanceTests.csproj
-          dotnet test -p:PackageVersion=${{ env.APP_BUILD_VERSION }} test/JUnit.Xml.TestLogger.AcceptanceTests/JUnit.Xml.TestLogger.AcceptanceTests.csproj
-      - name: Upload Verification Files on Fail
+          echo "Running acceptance tests with verbose output..."
+          dotnet test -p:PackageVersion=${{ env.APP_BUILD_VERSION }} test/TestLogger.AcceptanceTests/TestLogger.AcceptanceTests.csproj --logger "console;verbosity=detailed" --blame
+          dotnet test -p:PackageVersion=${{ env.APP_BUILD_VERSION }} test/Xunit.Xml.TestLogger.AcceptanceTests/Xunit.Xml.TestLogger.AcceptanceTests.csproj --logger "console;verbosity=detailed" --blame
+          dotnet test -p:PackageVersion=${{ env.APP_BUILD_VERSION }} test/NUnit.Xml.TestLogger.AcceptanceTests/NUnit.Xml.TestLogger.AcceptanceTests.csproj --logger "console;verbosity=detailed" --blame
+          dotnet test -p:PackageVersion=${{ env.APP_BUILD_VERSION }} test/JUnit.Xml.TestLogger.AcceptanceTests/JUnit.Xml.TestLogger.AcceptanceTests.csproj --logger "console;verbosity=detailed" --blame
+      - name: Upload Debug Files on Fail
         if: ${{ failure() }}
         uses: actions/upload-artifact@v4
         with:
-          name: Acceptance Test Failure
+          name: Acceptance Test Failure Debug Files
           retention-days: 5
           path: |
             **/*.verified.txt
             **/*.received.txt
+            **/test-results-*.xml
+            **/test-results-*.json
+            **/*.log
+            test/assets/**/bin/Debug/netcoreapp3.1/
+            test/assets/**/bin/Debug/net9.0/
       - name: Codecov
         uses: codecov/codecov-action@v3.1.0
         with:

.gitignore

@@ -25,7 +25,7 @@ bld/
 [Ll]og/
 
 # Test results
-**/*test-results.xml
+**/*test-results*.xml
 **/test-results.json
 
 # Visual Studio 2015/2017 cache/options directory

CLAUDE.md

@@ -0,0 +1,98 @@
+# CLAUDE.md
+
+This file provides guidance to Claude Code (claude.ai/code) when working with code in this repository.
+
+## Project Overview
+
+This repository contains .NET test loggers for Visual Studio Test Platform (VSTest) and Microsoft.Testing.Platform (MTP) that generate JUnit, NUnit, and Xunit XML reports. The project enables integration with CI/CD systems like Circle CI and Gitlab.
+
+## Development Commands
+
+### Building and Testing
+
+- **Full build and test**: `./build.sh` (Linux/macOS) or `.\build.ps1` (Windows)
+- **Package only**: `dotnet pack`
+- **Run specific unit test**: `dotnet test test/TestLogger.UnitTests/TestLogger.UnitTests.csproj`
+- **Run specific acceptance test**: `dotnet test test/TestLogger.AcceptanceTests/TestLogger.AcceptanceTests.csproj`
+
+### Debugging Acceptance Tests
+
+If acceptance tests are failing, run the test asset with detailed output. Example below:
+
+```sh
+DOTNET_SYSTEM_GLOBALIZATION_INVARIANT=1 # prefix below commands with this on Linux
+
+# Reproduce VSTest logger issue
+dotnet test --logger:"json;LogFilePath=test-results-vstest.json" test/assets/Json.TestLogger.MSTest.NetCore.Tests/Json.TestLogger.MSTest.NetCore.Tests.csproj
+
+# Reproduce MTP logger issue
+dotnet test test/assets/Json.TestLogger.MSTest.NetCore.Tests/Json.TestLogger.MSTest.NetCore.Tests.csproj -p:IsMTP=true -- --report-spekt-json --report-spekt-json test-results-mtp.json
+```
+
+## Architecture
+
+### Core Components
+
+- **TestLogger**: Core library with interfaces and workflows in `src/TestLogger/Core/` supporting both VSTest and MTP
+- **Format-specific loggers**: JUnit, NUnit, and Xunit implementations in their respective directories
+- **Package projects**: Distributable NuGet packages in `src/*.Package/`
+
+### Key Interfaces and Workflows
+
+- `ITestRunBuilder`: Builds test run information
+- `ITestResultSerializer`: Serializes test results to different formats
+- `TestRunMessageWorkflow`: Handles test run messages
+- `TestRunCompleteWorkflow`: Handles test run completion
+
+### Testing Strategy
+
+- **Unit tests**: Core functionality testing in `test/*UnitTests/`
+- **Acceptance tests**: End-to-end testing with test projects in `test/*AcceptanceTests/`
+- **Package tests**: NuGet package validation in `test/TestLogger.PackageTests/`
+
+## Configuration
+
+### Build Properties
+
+- Version is managed via `version.txt` and CI workflow
+- Uses .NET 9.0 for development, supports netstandard2.0+ for packages
+- Code analysis with StyleCop enforced via `Directory.Build.props`
+
+### Code Style
+
+- C# latest language version
+- XML documentation required for public APIs
+- StyleCop analyzers enabled with custom ruleset
+- Warnings treated as errors
+
+## Project Structure
+
+```
+src/
+├── TestLogger/               # Core library
+├── JUnit.Xml.TestLogger/     # JUnit format implementation
+├── NUnit.Xml.TestLogger/     # NUnit format implementation
+├── Xunit.Xml.TestLogger/     # Xunit format implementation
+└── *.Package/               # NuGet package projects
+
+test/
+├── *UnitTests/              # Unit tests for each logger
+├── *AcceptanceTests/        # End-to-end tests
+├── TestLogger.UnitTests/     # Core library tests
+└── assets/                  # Test project assets
+```
+
+## CI/CD
+
+- Uses GitHub Actions with multi-OS testing (Ubuntu, Windows)
+- Automated versioning via release-please action
+- Coverage reporting to Codecov
+- Pre-release packages published to MyGet
+
+## Development Guidelines
+
+- Respect DRY, SOLID and similar clean code practices.
+- Code must be usable, correct and performant.
+- Add documentation for all public methods.
+- Add unit tests for every change. Follow the existing test conventions in the project.
+- Do not add unnecessary comments.

Directory.Packages.props

@@ -5,13 +5,13 @@
     </PropertyGroup>
     <ItemGroup>
         <PackageVersion Include="Microsoft.TestPlatform.ObjectModel" Version="15.5.0" />
-        <PackageVersion Include="Microsoft.Testing.Platform" Version="1.8.3" />
+        <PackageVersion Include="Microsoft.Testing.Platform" Version="1.8.4" />
         <PackageVersion Include="Newtonsoft.Json" Version="13.0.3" />
 
         <PackageVersion Include="Stylecop.Analyzers" Version="1.2.0-beta.556" />
 
-        <PackageVersion Include="MSTest.TestFramework" Version="3.8.3" />
-        <PackageVersion Include="MSTest.TestAdapter" Version="3.8.3" />
+        <PackageVersion Include="MSTest.TestFramework" Version="3.10.1" />
+        <PackageVersion Include="MSTest.TestAdapter" Version="3.10.1" />
         <PackageVersion Include="xunit" Version="2.9.3" />
         <PackageVersion Include="xunit.runner.visualstudio" Version="3.0.2" />
 

nuget.config

@@ -0,0 +1,8 @@
+<?xml version="1.0" encoding="utf-8"?>
+<configuration>
+  <packageSources>
+    <!--To inherit the global NuGet package sources remove the <clear/> line below -->
+    <clear />
+    <add key="nuget" value="https://api.nuget.org/v3/index.json" />
+  </packageSources>
+</configuration>

src/JUnit.Xml.TestLogger/JUnitReporterCommandLineOptionsProvider.cs

@@ -10,13 +10,11 @@ namespace Spekt.TestReporter.JUnit
     internal sealed class JUnitTestReporter : TestReporter
     {
         public JUnitTestReporter(JUnitTestReporterExtension extension, IServiceProvider serviceProvider)
-            : base(serviceProvider, extension)
+            : base(serviceProvider, extension, "junit")
         {
         }
 
-        protected override string FileNameOption => JUnitReporterCommandLineOptionsProvider.ReportJUnitFileNameOption;
-
-        protected override string ReportOption => JUnitReporterCommandLineOptionsProvider.ReportJUnitOption;
+        protected override string DefaultFileName => "TestResults.xml";
 
         protected override ITestResultSerializer CreateTestResultSerializer()
             => new JunitXmlSerializer();

src/JUnit.Xml.TestLogger/JUnitTestReporter.cs

@@ -16,7 +16,7 @@ public static void AddJUnitReportProvider(this ITestApplicationBuilder testAppli
             testApplicationBuilder.TestHost.AddDataConsumer(compositeExtension);
             testApplicationBuilder.TestHost.AddTestSessionLifetimeHandle(compositeExtension);
 
-            testApplicationBuilder.CommandLine.AddProvider(() => new JUnitReporterCommandLineOptionsProvider(extension));
+            testApplicationBuilder.CommandLine.AddProvider(() => new TestReporterCommandLineProvider(extension, "junit"));
         }
     }
 }