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

Validation of the parameter $PesterResult value doesn't work as expected #11

Closed
it-praktyk opened this issue Aug 14, 2016 · 1 comment
Closed
Labels

Comments

@it-praktyk
Copy link
Collaborator

Even if in the parameter $PesterResult definitions is added (a code based on version 1.3.0 - please see the pull request #10 )

[Parameter(Mandatory = $true, Position = 0, ValueFromPipeline = $True, ValueFromPipelinebyPropertyName = $True, HelpMessage = 'Pester results Object', ParameterSetName = 'AllParamSet')]
        [Parameter(Mandatory = $true, Position = 0, ValueFromPipeline = $True, ValueFromPipelinebyPropertyName = $True, HelpMessage = 'Pester results Object', ParameterSetName = 'PassedOnlyParamSet')]
        [Parameter(Mandatory = $true, Position = 0, ValueFromPipeline = $True, ValueFromPipelinebyPropertyName = $True, HelpMessage = 'Pester results Object', ParameterSetName = 'FailedOnlyParamSet')]
        [Parameter(Mandatory = $true, Position = 0, ValueFromPipeline = $True, ValueFromPipelinebyPropertyName = $True, HelpMessage = 'Pester results Object', ParameterSetName = 'SummaryOnlyParamSet')]
        [Parameter(Mandatory = $false, Position = 0, ValueFromPipeline = $True, ValueFromPipelinebyPropertyName = $True, HelpMessage = 'Pester results Object', ParameterSetName = 'VersionOnlyParamSet')]
        [ValidateNotNullorEmpty()]
        [Array]$PesterResult,

the validation doesn't work as expected.

2016-08-14 22_20_49-windows powershell

Additional validation on provided tests results need to be added. I'll do it soon as a additional commit for the pull request #10

@it-praktyk
Copy link
Collaborator Author

I just added explicit throw for that issue - the pull request it-praktyk@d766e5b what is the part of #10

@equelin equelin added the bug label Aug 16, 2016
@equelin equelin closed this as completed Aug 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants