Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] coverlet.msbuild Support for Microsoft.Testing.Platform (not to be confused with VSTest Platform) #1715

Open
Arthri opened this issue Dec 1, 2024 · 0 comments
Labels
feature-request New feature request untriaged To be investigated

Comments

@Arthri
Copy link

Arthri commented Dec 1, 2024

Describe the bug
coverlet.msbuild does not work with the new testing platform server mode

To Reproduce
Create a new unit tests project using TUnit. Add coverlet.msbuild as a dependency to the project. Run with the command dotnet test -p:CollectCoverage=true -p:CoverletOutputFormat=opencover

Expected behavior
A code coverage report is generated somewhere, anywhere.

Actual behavior
No code coverage report is generated.

Configuration (please complete the following information):
Please provide more information on your .NET configuration:
* Which coverlet package and version was used? coverlet.msbuild and 6.0.2
* Which version of .NET is the code running on? 9.0.100
* What OS and version, and what distro if applicable? Windows 10 22H2
* What is the architecture (x64, x86, ARM, ARM64)? x64
* Do you know whether it is specific to that configuration? Uncertain.

Additional context
One of the two instrumentation targets, InstrumentModulesAfterBuild, depends on(runs after) the BuildProject target, which no longer exists under the new Microsoft Testing Platform. Therefore, instrumentation is not ran regardless of what happens.

I've found two solutions to the problem, but neither of which are extensively tested.

  1. Recreate BuildProject, so that the instrumentation target has something to hook onto and is able to run.
    Add a Directory.Build.targets in the same directory as the project, then set its contents to the following,
    <Project>
      <Target Name="BuildProject" BeforeTargets="InvokeTestingPlatform" />
    </Project>
    It doesn't matter that BuildProject is empty and does nothing, because the project build is handled by the _BuildAndInvokeTestingPlatform target.
  2. Pass the --no-build option to dotnet test, for example, dotnet test -p:CollectCoverage=true -p:CoverletOutputFormat=opencover --no-build. This solution depends on the other instrumentation target, InstrumentModulesNoBuild, which depends on(runs before) the VSTest target. The aforementioned target still exists on the new testing platform.

coverlet.console works, from prior testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request New feature request untriaged To be investigated
Projects
None yet
Development

No branches or pull requests

2 participants