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

WIP: Add Coverlet Coverage to builds #4722

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Conversation

SeanKilleen
Copy link
Member

@SeanKilleen SeanKilleen commented Jun 3, 2024

A continuation of #4286, which got stale. Using lessons from my recent(ish) blog post.

Work Involved

(I think -- still feeling my way around the project in this way)

  • Add coverlet.collector to the codebase
  • Add TRX logging output to all places using dotnet test
  • Combine coverage and surface test reports for each test run
  • Add Coverlet add-in to Cake script
  • Add Coverlet dotnet global tool
  • Utilize coverlet CLI tool or cake task in cake-based build items

@SeanKilleen
Copy link
Member Author

@stevenaw FYI, I restarted things here. Rough weekend, didn't get nearly as much time as I thought I would. But at least it's no longer stale, so, small win? 😅

Next up, I'll look through the cake file to understand where we're outputting our test assemblies and will figure out how to run coverlet on them (since we're not using dotnet test which makes sense, I'll probably look into how to use the Cake CLI add-in to do what we need to do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant