Pack props into buildTransitive so MTP registration flows to transitive consumers

[jetersen]

Problem

In Microsoft.Testing.Platform mode, the extension registers itself through the TestingPlatformBuilderHook item declared in GitHubActionsTestLogger.props. That props file is packed only into build/, and NuGet imports build/ assets only for direct package references — buildTransitive/ is the folder that flows to transitive consumers.

So when the package is consumed transitively — e.g. an organization's shared testing package declares GitHubActionsTestLogger as a dependency so every test project gets it — the compile/runtime assets arrive fine, but the builder hook item never reaches the test project. The source-generated SelfRegisteredExtensions omits the extension and the reporter silently never runs.

Fix

Pack the same props file into both folders:

<Content Include="GitHubActionsTestLogger.props" Pack="true" PackagePath="build;buildTransitive" />

VSTest usage is unaffected (the logger assembly already flows transitively to the build output where VSTest discovers it; the props only matter for MTP registration).

Validation

Scenario: Consumer (xunit.v3.mtp-v2 test project) → references TestKit (packaged class library) → depends on GitHubActionsTestLogger. No direct reference from Consumer.

Package	GitHubActionsTestLogger.MtpIntegration.AddExtensions in generated SelfRegisteredExtensions.cs
3.0.4 (build/ only)	❌ absent — extension never registers
this branch (build/ + buildTransitive/)	✅ present

Also verified the produced nupkg contains both build/GitHubActionsTestLogger.props and buildTransitive/GitHubActionsTestLogger.props.

[Copilot]

Pull request overview

This PR fixes Microsoft.Testing.Platform (MTP) self-registration when GitHubActionsTestLogger is consumed transitively by ensuring the package’s .props file is available under buildTransitive/ (not just build/), allowing the TestingPlatformBuilderHook item to flow to downstream test projects.

Changes:

• Pack GitHubActionsTestLogger.props into both build/ and buildTransitive/ so MTP registration works for transitive consumers.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Tyrrrz]

Please run dotnet build to fix formatting

[jetersen]

Done 😅

[Tyrrrz]

Thanks!

[jetersen]

👏🎉 Thank you!