Downgrade Microsoft.Testing.Extensions.CodeCoverage version to 18.1.0

[thomhurst]

The latest version seems to be crashing test suites with broken pipe errors

[thomhurst]

Summary

Downgrades Microsoft.Testing.Extensions.CodeCoverage from 18.3.1 to 18.1.0 to fix broken pipe errors.

Critical Issues

None found ✅

This is a straightforward dependency downgrade to resolve runtime crashes. The change:

• Only modifies the package version in Directory.Packages.props
• Does not affect any TUnit critical rules (no source gen, no API changes, no reflection)
• Uses Microsoft.Testing.Platform ecosystem (correct, not VSTest)
• Includes minor EOF newline fix

Verdict

✅ APPROVE - No critical issues

The downgrade is a practical fix for a blocking runtime issue. Once the upstream bug is resolved, this can be reverted.

[Copilot]

Pull request overview

This PR downgrades the Microsoft.Testing.Extensions.CodeCoverage package from version 18.3.1 to 18.1.0 to address broken pipe errors that have been occurring in test suites with the latest version.

Key Changes:

• Downgraded Microsoft.Testing.Extensions.CodeCoverage from 18.3.1 to 18.1.0 in centralized package management

[Copilot]

The version numbering for Microsoft.Testing.Extensions.CodeCoverage (18.1.0) differs significantly from other Microsoft.Testing.Extensions packages (2.0.2). This suggests CodeCoverage may follow a different versioning scheme. Consider verifying that version 18.1.0 is compatible with Microsoft.Testing.Platform version 2.0.2, as package version mismatches could cause integration issues.

Suggested change

	<PackageVersion Include="Microsoft.Testing.Extensions.CodeCoverage" Version="18.1.0" />
	<PackageVersion Include="Microsoft.Testing.Extensions.CodeCoverage" Version="2.0.2" />

[thomhurst]

Summary

Downgrades Microsoft.Testing.Extensions.CodeCoverage from 18.3.1 to 18.1.0 to fix broken pipe crashes, and adds --coverage and --report-trx arguments to pipeline test modules.

Critical Issues

None found ✅

Suggestions

Consider documenting the version issue: Since this is a workaround for a broken upstream package version, consider adding a comment in Directory.Packages.props near line 40 to document why this specific version is pinned and what issue it resolves (broken pipe errors in 18.3.1). This will help future maintainers understand when it's safe to upgrade.

Verify test coverage reporting works: The addition of --coverage and --report-trx arguments to the pipeline test modules (TestNugetPackageModule.cs:71-73 and TestAspNetCoreNugetPackageModule.cs:50-52) appears to be related to the downgrade. Ensure that coverage reporting still functions correctly with version 18.1.0.

Verdict

✅ APPROVE - No critical issues

The downgrade is a pragmatic fix for a blocking issue. The changes are minimal and low-risk.