Coverage Issue - New Using + Async/Await + ConfigureAwait

[abbotware]

found during #649, this combination of code seems to have incorrect reports

        public async Task<ActionResult<IEnumerable<SystemUserItem>>> SystemUsers(CancellationToken ct)
        {
            using var data = this.factory.CreateReadOnly<SystemUserItem>();

            var list = await data.AllAsync(ct)
                .ConfigureAwait(false);

            return this.Ok(list);
        }

here is the final report output:

here is the diff of the coverage xml:

left side = unit test - code is not executed
right side = integration test - code is there doesnt appear to be any branches - all code execute under a single call

[MarcoRossignoli]

Thank's for reporting this

[moikot]

I've got the same issue. Using version 2.8.0 of coverlet.msbuild. But I don't have ConfigureAwait.

[MarcoRossignoli]

@moikot can provide a "repro"?
I mean a small project affected by the issue?

[moikot]

Please find it here https://github.com/moikot/async-await-coverage

[MarcoRossignoli]

Great thank's I'll take a look asap.

[MarcusOtter]

Having the same problem with my repo. Coverage dropped from 100% to 93% without any changes in source code.

See https://coveralls.io/builds/28220011 (repo: https://github.com/LeMorrow/APOD.Net/tree/develop)
Not sure exactly what information you need from me, let me know how I can help :)

[MarcoRossignoli]

@LeMorrow can you isolate a small repro the @moikot one(where I'm investigating) seem without ConfigureAwait.

[MarcoRossignoli]

Guys I found the issue...I'PR a fix asap.
Are two different issue, one related to netstandard project and the other related on new async machine state code.

[MarcoRossignoli]

Hi devs please can you try the master version using our nightly build and tell me if works as expected? https://github.com/tonerdo/coverlet/blob/master/Documentation/ConsumeNightlyBuild.md tomorrow fix PR will land inside packages!

Thank's a lot for helping coverlet!