Skip to content

Coverage loss chained lambdas #1064

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.

Sign up for GitHub

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

Jump to bottom
Closed
SebastienPortier opened this issue Jan 20, 2021 · 2 comments
Closed

Coverage loss chained lambdas #1064

SebastienPortier opened this issue Jan 20, 2021 · 2 comments
Labels
duplicate This issue or pull request already exists

Comments

@SebastienPortier
Copy link

SebastienPortier commented Jan 20, 2021 •
edited
Loading

Hi !

I have updated "coverlet.collector" Version="3.0.1" with the fix you provided on severe coverage loss in 3.0.0.

Even if the situation is clearly better, I am still missing some coverage with lambda "chained".

I saw on this commit you worked on the lambda coverage, but the tests doesn't take in account scenarios like the screenshot below

Capture d’écran 2021-01-20 212502
Capture d’écran 2021-01-20 212717

Modifying the code to be on a single line will correct the coverage OK 👍

Capture d’écran 2021-01-21 101846

Thanks you in advance for support ;-)

Regards,
@MarcoRossignoli MarcoRossignoli added the duplicate This issue or pull request already exists label Jan 23, 2021
@MarcoRossignoli
Copy link
Collaborator

Yep it's a dup of #1056
is fixed on master this weekend I'll release the fix officially.
You can try with nightly for now https://github.com/coverlet-coverage/coverlet/blob/master/Documentation/ConsumeNightlyBuild.md

@SebastienPortier
Copy link
Author

Nice ! Thanks for answer regards

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
duplicate This issue or pull request already exists
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@SebastienPortier @MarcoRossignoli