Skip to content

Fix CI test logger failure on .NET 10 / vstest 18#1

Merged
rockfordlhotka merged 1 commit intomainfrom
fix/ci-github-test-logger
Feb 26, 2026
Merged

Fix CI test logger failure on .NET 10 / vstest 18#1
rockfordlhotka merged 1 commit intomainfrom
fix/ci-github-test-logger

Conversation

@rockfordlhotka
Copy link
Member

Summary

  • GitHubActionsTestLogger 2.x uses the legacy VSTest extension discovery mechanism, which is broken in vstest 18.0.1 (.NET 10 SDK)
  • Switches to v3.0.1, which targets the new Microsoft Testing Platform (MTP) and auto-registers via TestingPlatformBuilderHook — no explicit --logger flag needed
  • Adds tests/Directory.Build.props to enable UseMicrosoftTestingPlatformRunner and include the logger package across all test projects

Test plan

  • Verified dotnet test --no-build --configuration Release passes all 112 tests locally
  • CI should pass on this PR without the "Could not find a test logger with FriendlyName 'github'" error

🤖 Generated with Claude Code

GitHubActionsTestLogger 2.x uses the legacy VSTest extension discovery
mechanism which no longer works with vstest 18.0.1 shipped in the .NET 10
SDK. Switch to v3.0.1, which targets Microsoft.Testing.Platform (MTP) and
auto-registers via TestingPlatformBuilderHook — no explicit --logger flag
required.

- Add tests/Directory.Build.props to enable UseMicrosoftTestingPlatformRunner
  and include GitHubActionsTestLogger in all test projects (chains to root
  Directory.Build.props)
- Bump GitHubActionsTestLogger to 3.0.1 in Directory.Packages.props
- Remove --logger "github;..." from the CI test step (auto-registered by MTP)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@rockfordlhotka rockfordlhotka merged commit 5cf6e33 into main Feb 26, 2026
1 check passed
@rockfordlhotka rockfordlhotka deleted the fix/ci-github-test-logger branch February 26, 2026 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant