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

Add help for dynamic operator parameters #2170

Merged
merged 8 commits into from Oct 3, 2022
Merged

Conversation

fflaten
Copy link
Collaborator

@fflaten fflaten commented May 2, 2022

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

  • PR has meaningful title
  • Summary describes changes
  • 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.
  • Tests are added/update (if required)
  • Documentation is updated/added (if required)

@fflaten
Copy link
Collaborator Author

fflaten commented May 3, 2022

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@fflaten
Copy link
Collaborator Author

fflaten commented May 3, 2022

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

@fflaten
Copy link
Collaborator Author

fflaten commented May 3, 2022

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?

@nohwnd nohwnd merged commit de77c6d into pester:main Oct 3, 2022
@fflaten fflaten deleted the parameterhelp branch October 3, 2022 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants