Pester icon indicating copy to clipboard operation
Pester copied to clipboard

Add help for dynamic operator parameters

Open fflaten opened this issue 3 years ago • 4 comments
trafficstars

PR Summary

Add missing parameter help for dynamic parameters to solve this @ pester.dev

image

Help for dynamic parameters are not shown in console, but are published in the online docs. This PR adds helpmessage for:

  • Name parameter in Get-ShouldOperator
  • Operator-parameters in Should. They will now use the operator's synopsis if it's defined
  • Adds a hard-coded string Depends on operator being used. See Get-ShouldOperator -Name <Operator> for help. for other dynamic parameters in Should

Related #2128 Issue won't be fixed until command reference is regenerated in docs-repo after PR is merged.

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 avatar May 02 '22 17:05 fflaten

/azp run

fflaten avatar May 03 '22 10:05 fflaten

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar May 03 '22 10:05 azure-pipelines[bot]

Not sure how I broke the universe with the latest commit. TBC.. 😁

fflaten avatar May 03 '22 13:05 fflaten

Not sure how I broke the universe with the latest commit. TBC.. 😁

Ofc. 32+ parameter sets limit again. Pester.Tests.ps1 should probably be executed isolated as it's over limit (= corrupted) even before this PR.

Maybe rewrite this Describe-block as P-tests?

fflaten avatar May 03 '22 18:05 fflaten