Implement CodeCoverage.ExcludeTests

pester / Pester

Pester is the ubiquitous test and mock framework for PowerShell.

https://pester.dev/

Other

3.1k stars 469 forks source link

Implement CodeCoverage.ExcludeTests #2523

Closed fflaten closed 3 months ago

fflaten commented 3 months ago

PR Summary

Temporary fix to implement CodeCoverage.ExcludeTests until CodeCoverage configuration parsing either implements old advanced interface or removes the old code and updates tests.

Fix #2514

PR Checklist

[x] PR has meaningful title
[x] Summary describes changes
[x] PR is ready to be merged
- If not, use the arrow next to Create Pull Request to mark it as a draft. PR can be marked Ready for review when it's ready.
[x] Tests are added/update (if required)
[ ] Documentation is updated/added (if required)

fflaten commented 3 months ago

Backport to 5.6.x patch or will this be considered breaking/feature after never working? Users who never noticed might get a sudden change in coverage report.

nohwnd commented 3 months ago

/backport to /rel/5.6.x

github-actions[bot] commented 3 months ago

Started backporting to /rel/5.6.x: https://github.com/pester/Pester/actions/runs/9758671448

github-actions[bot] commented 3 months ago

@nohwnd an error occurred while backporting to /rel/5.6.x, please check the run log for details!

Error: The specified backport target branch /rel/5.6.x wasn't found in the repo.

nohwnd commented 3 months ago

/backport to rel/5.6.x

github-actions[bot] commented 3 months ago

Started backporting to rel/5.6.x: https://github.com/pester/Pester/actions/runs/9758688151

nohwnd commented 3 months ago

Backporting, this is a bug fix, and ExcludeTests = $true is the default, so people who opted in either did not notice or did not care.