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.

Sign up for GitHub

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

Jump to bottom

Migrate Pester Test Syntax from v3 -> v4 #120

Closed
PlagueHO opened this issue Oct 12, 2017 · 0 comments · Fixed by #125
Closed

Migrate Pester Test Syntax from v3 -> v4 #120

PlagueHO opened this issue Oct 12, 2017 · 0 comments · Fixed by #125
Assignees
@PlagueHO
Labels
enhancement The issue is an enhancement request.

Comments

@PlagueHO
Copy link
Member

Details of the scenario you tried and the problem that is occurring:
As of Pester v4 the syntax of the Should cmdlet has changed from Should Be to Should -Be.

There is a helpful script that can be used to easily perform the conversion:
https://gist.github.com/indented-automation/aeb14825e39dd8849beee44f681fbab3

This script should be used to correct the Pester syntax.
@PlagueHO PlagueHO added enhancement The issue is an enhancement request. help wanted The issue is up for grabs for anyone in the community. labels Oct 12, 2017
@PlagueHO PlagueHO self-assigned this Oct 12, 2017
@PlagueHO PlagueHO mentioned this issue Nov 12, 2017
Merged
5 tasks
PlagueHO added a commit that referenced this issue Nov 17, 2017
@PlagueHO
Merge pull request #125 from PlagueHO/Issue-120
006cab5 
Converted Pester tests to V4 - Fixes #120
@joeyaiello joeyaiello removed the help wanted The issue is up for grabs for anyone in the community. label Nov 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@PlagueHO PlagueHO

Labels
enhancement The issue is an enhancement request.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Converted Pester tests to V4 - Fixes #120 PlagueHO/StorageDsc
2 participants
@joeyaiello @PlagueHO