coverlet.msbuild.tasks unit tests

[Bertk]

• add CoverageResultTaskTests.cs for coverlet.msbuild.tasks unit test
• allows local debugging of msbuild tasks
• removes system.console output in ReportWriter (potential conflicts - msbuild task logging)

[Bertk]

The CI build fails with /p:CollectCoverage=true and my local test execution fails with a different message

Everything is fine without /p:CollectCoverage=true.

... and suddenly the CI does not show test errors anymore with enabled code coverage.

[MarcoRossignoli]

I see formatting and updates, maybe it's better avoid to format here and add only tests to make the review easier, we could lose some small detail

[Bertk]

I see formatting and updates, maybe it's better avoid to format here and add only tests to make the review easier, we could lose some small detail

This is will be fixed with the other PR #1529 . The source code files are not in sync with the current definitions of .editorconfig. By the way, the format on save VS feature cannot be used until the source files are harmonized.

https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/style-rules/csharp-formatting-options

[MarcoRossignoli]

LGTM wait for @daveMueller green light

[Bertk]

@daveMueller By the way, could you please check whether mandatory checks are defined? Otherwise a PR with a broken build pipeline could be merged.

This configuration is available in Settings->branches->branch protection

[daveMueller]

@daveMueller By the way, could you please check whether mandatory checks are defined? Otherwise a PR with a broken build pipeline could be merged.

This configuration is available in Settings->branches->branch protection

Sorry I also don't have permission to look into that. Maybe @MarcoRossignoli can? But I'm pretty sure that PRs with a failed build job can't be merged.

[daveMueller]

Thanks a lot @Bertk 🙏