Be sure to change psdefaultparametervalue in the global scope (PowerShell#4977)

JamesWTruher · adityapatwardhan · commit f69b760f2bc5 · 2017-10-02T16:58:47.000-07:00
* Be sure that you change psdefaultparametervalue in the global scope

* [Feature]
diff --git a/test/powershell/Modules/Microsoft.PowerShell.Security/CertificateProvider.Tests.ps1 b/test/powershell/Modules/Microsoft.PowerShell.Security/CertificateProvider.Tests.ps1
@@ -28,15 +28,15 @@ Describe "Certificate Provider tests" -Tags "CI" {
         if(!$IsWindows)
         {
             # Skip for non-Windows platforms
-            $defaultParamValues = $PSdefaultParameterValues.Clone()
-            $PSdefaultParameterValues = @{ "it:skip" = $true }
+            $defaultParamValues = $global:PSDefaultParameterValues.Clone()
+            $global:PSDefaultParameterValues = @{ "it:skip" = $true }
         }        
     }
 
     AfterAll {
         if(!$IsWindows)
         {
-            $global:PSdefaultParameterValues = $defaultParamValues
+            $global:PSDefaultParameterValues = $defaultParamValues
         }
     }
 
@@ -85,8 +85,8 @@ Describe "Certificate Provider tests" -Tags "Feature" {
         else
         {
             # Skip for non-Windows platforms
-            $defaultParamValues = $PSdefaultParameterValues.Clone()
-            $PSdefaultParameterValues = @{ "it:skip" = $true }
+            $defaultParamValues = $global:PSDefaultParameterValues.Clone()
+            $PSDefaultParameterValues = @{ "it:skip" = $true }
         }        
     }
     
@@ -98,7 +98,7 @@ Describe "Certificate Provider tests" -Tags "Feature" {
         }
         else
         {
-            $global:PSdefaultParameterValues = $defaultParamValues
+            $global:PSDefaultParameterValues = $defaultParamValues
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -28,15 +28,15 @@ Describe "Certificate Provider tests" -Tags "CI" {`
`28`	`28`	`if(!$IsWindows)`
`29`	`29`	`{`
`30`	`30`	`# Skip for non-Windows platforms`
`31`		`- $defaultParamValues = $PSdefaultParameterValues.Clone()`
`32`		`- $PSdefaultParameterValues = @{ "it:skip" = $true }`
	`31`	`+ $defaultParamValues = $global:PSDefaultParameterValues.Clone()`
	`32`	`+ $global:PSDefaultParameterValues = @{ "it:skip" = $true }`
`33`	`33`	`}`
`34`	`34`	`}`
`35`	`35`
`36`	`36`	`AfterAll {`
`37`	`37`	`if(!$IsWindows)`
`38`	`38`	`{`
`39`		`- $global:PSdefaultParameterValues = $defaultParamValues`
	`39`	`+ $global:PSDefaultParameterValues = $defaultParamValues`
`40`	`40`	`}`
`41`	`41`	`}`
`42`	`42`
`@@ -85,8 +85,8 @@ Describe "Certificate Provider tests" -Tags "Feature" {`
`85`	`85`	`else`
`86`	`86`	`{`
`87`	`87`	`# Skip for non-Windows platforms`
`88`		`- $defaultParamValues = $PSdefaultParameterValues.Clone()`
`89`		`- $PSdefaultParameterValues = @{ "it:skip" = $true }`
	`88`	`+ $defaultParamValues = $global:PSDefaultParameterValues.Clone()`
	`89`	`+ $PSDefaultParameterValues = @{ "it:skip" = $true }`
`90`	`90`	`}`
`91`	`91`	`}`
`92`	`92`
`@@ -98,7 +98,7 @@ Describe "Certificate Provider tests" -Tags "Feature" {`
`98`	`98`	`}`
`99`	`99`	`else`
`100`	`100`	`{`
`101`		`- $global:PSdefaultParameterValues = $defaultParamValues`
	`101`	`+ $global:PSDefaultParameterValues = $defaultParamValues`
`102`	`102`	`}`
`103`	`103`	`}`
`104`	`104`