Async methods excluded from coverage after updating to 2.8.1

[Malivil]

Using coverage format "OpenCover" and the coverlet.msbuild package
In version 2.8.0 an async method in my Web API class shows no coverage (because I haven't written tests yet):

In version 2.8.1, the same method is just excluded from coverage entirely:

The coverage.opencover.xml shows a complete missing section for the method:

I do not have a minimal reproduction example but if needed I can probably make one.
May be related to 528956b

[MarcoRossignoli]

Do you have ExcludeFromCodeCoverage attribute somewhere?
Do you see missing coverage only for this method in source or for all method of same class?

I do not have a minimal reproduction example but if needed I can probably make one.

Would be great!It's hard without repro.

cc: @matteoerigozzi

[Malivil]

This class does not have ExcludeFromCodeCoverage
Only the two methods marked async are not showing coverage.

I will try to make a simple reproduction project. I'll comment here with the repo when it's complete.

[MarcoRossignoli]

Ok thanks a lot!

[Malivil]

Here's a repro with the minimum example: https://github.com/Malivil/CoverletAsyncRepro
If you run the .bat file within the .Tests project it will generate a report in the TestResults folder

Changing the version of the coverlet.msbuild in the .Tests csproj to 2.8.0 and then re-running the .bat will show the function as uncovered as it should.

[MarcoRossignoli]

Great, stay on 2.8.0 for now I'll come back asap.

[bgoymer-unilink]

Hi there,

I've also noticed this issue recently, it appears to be something to do with the /p:ExcludeByAttribute=CompilerGeneratedAttribute argument, as removing this will cause the coverage to be successfully reported. Thanks for the tip of going back to 2.8.0 as that also solves the issue for now.

[MarcoRossignoli]

@Malivil can you confirm that removing /p:ExcludeByAttribute=CompilerGeneratedAttribute solve(disappear, but it's clearly a bug) issue?

[bgoymer-unilink]

@MarcoRossignoli I feel that I need to clarify; athough removing the argument appears to solve this issue, it will cause all our auto-implemented properties in our DTOs to record no coverage without being explictly tested, which we did not have to do prior to 2.8.1.

[MarcoRossignoli]

@bengoymer yes I mean this is a bug but if we know that is related to that parameter we can fix more fast.

[bgoymer-unilink]

@MarcoRossignoli perfect, thank you.

[MarcoRossignoli]

@bengoymer can you try to remove that param and filter out DTO using source file filtering(if DTO are in separate files)?
https://github.com/tonerdo/coverlet/blob/master/Documentation/MSBuildIntegration.md#source-files