Add missing test file - Githubissues

pester / Pester

Pester is the ubiquitous test and mock framework for PowerShell.

https://pester.dev/

Other

3.11k stars 473 forks source link

Add missing test file #2525

Closed fflaten closed 4 months ago

fflaten commented 4 months ago

PR Summary

I'm not sure how #2523 passed checks, but this file was not committed. Discovered in backport PR #2524 where tests failed.

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)

nohwnd commented 4 months ago

Did you have closer look on why the tests in the pipeline did not fail when the file was missing?

fflaten commented 4 months ago

Had a look now. The Coverage P-tests are skipped in v6 pipeline due to use of test.ps1 -CC parameter.

nohwnd commented 4 months ago

Yeah I need to fix that, or run those tests separately. I think I disabled them because they were turning off tracing and breaking the overall code coverage.

nohwnd commented 4 months ago

https://github.com/pester/Pester/issues/2526