Refactor integration tests to check for null before restoring Agent XPs value

johlju · johlju · commit f090d96ed66a · 2025-09-22T18:06:26.000+02:00
diff --git a/tests/Integration/Commands/Set-SqlDscConfigurationOption.Integration.Tests.ps1 b/tests/Integration/Commands/Set-SqlDscConfigurationOption.Integration.Tests.ps1
@@ -46,7 +46,7 @@ Describe 'Set-SqlDscConfigurationOption' -Tag @('Integration_SQL2017', 'Integrat
 
     AfterAll {
         # Restore the original Agent XPs value
-        if ($script:originalAgentXPsValue)
+        if ($null -ne $script:originalAgentXPsValue)
         {
             Set-SqlDscConfigurationOption -ServerObject $script:serverObject -Name 'Agent XPs' -Value $script:originalAgentXPsValue.RunValue -Force -ErrorAction 'SilentlyContinue'
         }
diff --git a/tests/Integration/Commands/Test-SqlDscConfigurationOption.Integration.Tests.ps1 b/tests/Integration/Commands/Test-SqlDscConfigurationOption.Integration.Tests.ps1
@@ -46,7 +46,7 @@ Describe 'Test-SqlDscConfigurationOption' -Tag @('Integration_SQL2017', 'Integra
 
     AfterAll {
         # Restore the original Agent XPs value
-        if ($script:originalAgentXPsValue)
+        if ($null -ne $script:originalAgentXPsValue)
         {
             Set-SqlDscConfigurationOption -ServerObject $script:serverObject -Name 'Agent XPs' -Value $script:originalAgentXPsValue.RunValue -Force -ErrorAction 'SilentlyContinue'
         }

Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ Describe 'Set-SqlDscConfigurationOption' -Tag @('Integration_SQL2017', 'Integrat`
`46`	`46`
`47`	`47`	`AfterAll {`
`48`	`48`	`# Restore the original Agent XPs value`
`49`		`- if ($script:originalAgentXPsValue)`
	`49`	`+ if ($null -ne $script:originalAgentXPsValue)`
`50`	`50`	`{`
`51`	`51`	`Set-SqlDscConfigurationOption -ServerObject $script:serverObject -Name 'Agent XPs' -Value $script:originalAgentXPsValue.RunValue -Force -ErrorAction 'SilentlyContinue'`
`52`	`52`	`}`
Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ Describe 'Test-SqlDscConfigurationOption' -Tag @('Integration_SQL2017', 'Integra`
`46`	`46`
`47`	`47`	`AfterAll {`
`48`	`48`	`# Restore the original Agent XPs value`
`49`		`- if ($script:originalAgentXPsValue)`
	`49`	`+ if ($null -ne $script:originalAgentXPsValue)`
`50`	`50`	`{`
`51`	`51`	`Set-SqlDscConfigurationOption -ServerObject $script:serverObject -Name 'Agent XPs' -Value $script:originalAgentXPsValue.RunValue -Force -ErrorAction 'SilentlyContinue'`
`52`	`52`	`}`