From f29a04c1bd915a2a551e689a09f9f05eade15b34 Mon Sep 17 00:00:00 2001 From: Reece Dunham Date: Thu, 17 Jan 2019 15:33:35 -0500 Subject: [PATCH] Remove appveyor environment checks (#8669) --- .../Enter-PSHostProcess.Tests.ps1 | 12 ++---------- .../Microsoft.PowerShell.Core/Set-PSDebug.Tests.ps1 | 4 ++-- 2 files changed, 4 insertions(+), 12 deletions(-) diff --git a/test/powershell/Modules/Microsoft.PowerShell.Core/Enter-PSHostProcess.Tests.ps1 b/test/powershell/Modules/Microsoft.PowerShell.Core/Enter-PSHostProcess.Tests.ps1 index 721ea25c0..1b2668391 100644 --- a/test/powershell/Modules/Microsoft.PowerShell.Core/Enter-PSHostProcess.Tests.ps1 +++ b/test/powershell/Modules/Microsoft.PowerShell.Core/Enter-PSHostProcess.Tests.ps1 @@ -16,12 +16,6 @@ Describe "Enter-PSHostProcess tests" -Tag Feature { $powershell = [System.Diagnostics.Process]::Start($si) } - if ($env:AppVeyor) { - $IsAppveyor = $true - } - else { - $IsAppveyor = $false - } } AfterAll { @@ -32,13 +26,11 @@ Describe "Enter-PSHostProcess tests" -Tag Feature { } } - # Skip on Appveyor due to PSReadline issue. - It "Can enter and exit another PSHost" -Skip:$IsAppVeyor { + It "Can enter and exit another PSHost" { "enter-pshostprocess -id $($pwsh.Id)`n`$pid`nexit-pshostprocess" | pwsh -c - | Should -Be $pwsh.Id } - # Skip on Appveyor due to PSReadline issue. - It "Can enter and exit another Windows PowerShell PSHost" -Skip:(!$IsWindows -or $IsAppVeyor) { + It "Can enter and exit another Windows PowerShell PSHost" -Skip:(!$IsWindows) { "enter-pshostprocess -id $($powershell.Id)`n`$pid`nexit-pshostprocess" | pwsh -c - | Should -Be $powershell.Id } diff --git a/test/powershell/Modules/Microsoft.PowerShell.Core/Set-PSDebug.Tests.ps1 b/test/powershell/Modules/Microsoft.PowerShell.Core/Set-PSDebug.Tests.ps1 index ca962e7f3..e6c5743f4 100644 --- a/test/powershell/Modules/Microsoft.PowerShell.Core/Set-PSDebug.Tests.ps1 +++ b/test/powershell/Modules/Microsoft.PowerShell.Core/Set-PSDebug.Tests.ps1 @@ -6,13 +6,13 @@ Describe "Set-PSDebug" -Tags "CI" { Set-PSDebug -Off } - It "Should be able to go through the tracing options" -Skip:($env:APPVEYOR -eq "TRUE") { + It "Should be able to go through the tracing options" { { Set-PSDebug -Trace 0 } | Should -Not -Throw { Set-PSDebug -Trace 1 } | Should -Not -Throw { Set-PSDebug -Trace 2 } | Should -Not -Throw } - It "Should be able to set strict" -Skip:($env:APPVEYOR -eq "TRUE") { + It "Should be able to set strict" { { Set-PSDebug -Strict } | Should -Not -Throw } }