Adding test coverage for Registry provider#2307
Adding test coverage for Registry provider#2307mirichmo merged 1 commit intoPowerShell:masterfrom ChrisUbben:PortTestMultiMachineProvidersRegistry
Conversation
|
Hi @ChrisUbben, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
TTYL, MSBOT; |
There was a problem hiding this comment.
This should only run on Windows systems. Add
if(-not $IsWindows)
{
return
}
Before the Describe
There was a problem hiding this comment.
I talked to @JamesWTruher, he said that this is the more correct way of skipping all tests in a describe block:
#skip all tests on non-windows platform
$originalDefaultParameterValues = $PSDefaultParameterValues.Clone()
if ($IsWindows -eq $false) {
$PSDefaultParameterValues["it:skip"] = $true
}
There was a problem hiding this comment.
These should be redirected to $null as well
Could you both take a look at this for me? @Francisco-Gamino @JamesWTruher
Thanks!