1f252f8bba
# PR Summary Wrap tests in pester blocks to prepare for pesterv5 ## PR Context <!-- Provide a little reasoning as to why this Pull Request helps and why you have opened it. --> ## PR Checklist - [x] [PR has a meaningful title](https://github.com/PowerShell/PowerShell/blob/master/.github/CONTRIBUTING.md#pull-request---submission) - Use the present tense and imperative mood when describing your changes - [x] [Summarized changes](https://github.com/PowerShell/PowerShell/blob/master/.github/CONTRIBUTING.md#pull-request---submission) - [x] [Make sure all `.h`, `.cpp`, `.cs`, `.ps1` and `.psm1` files have the correct copyright header](https://github.com/PowerShell/PowerShell/blob/master/.github/CONTRIBUTING.md#pull-request---submission) - [x] This PR is ready to merge and is not [Work in Progress](https://github.com/PowerShell/PowerShell/blob/master/.github/CONTRIBUTING.md#pull-request---work-in-progress). - If the PR is work in progress, please add the prefix `WIP:` or `[ WIP ]` to the beginning of the title (the `WIP` bot will keep its status check at `Pending` while the prefix is present) and remove the prefix when the PR is ready. - **[Breaking changes](https://github.com/PowerShell/PowerShell/blob/master/.github/CONTRIBUTING.md#making-breaking-changes)** - [x] None - **OR** - [ ] [Experimental feature(s) needed](https://github.com/MicrosoftDocs/PowerShell-Docs/blob/staging/reference/6/Microsoft.PowerShell.Core/About/about_Experimental_Features.md) - [ ] Experimental feature name(s): <!-- Experimental feature name(s) here --> - **User-facing changes** - [x] Not Applicable - **OR** - [ ] [Documentation needed](https://github.com/PowerShell/PowerShell/blob/master/.github/CONTRIBUTING.md#pull-request---submission) - [ ] Issue filed: <!-- Number/link of that issue here --> - **Testing - New and feature** - [x] N/A or can only be tested interactively - **OR** - [ ] [Make sure you've added a new test if existing tests do not effectively test the code changed](https://github.com/PowerShell/PowerShell/blob/master/.github/CONTRIBUTING.md#before-submitting) - **Tooling** - [x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted. - **OR** - [ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include: - Impact on [PowerShell Editor Services](https://github.com/PowerShell/PowerShellEditorServices) which is used in the [PowerShell extension](https://github.com/PowerShell/vscode-powershell) for VSCode (which runs in a different PS Host). - Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features. - Impact on [PSScriptAnalyzer](https://github.com/PowerShell/PSScriptAnalyzer) (which provides linting & formatting in the editor extensions). - Impact on [EditorSyntax](https://github.com/PowerShell/EditorSyntax) (which provides syntax highlighting with in VSCode, GitHub, and many other editors).
125 lines
4.3 KiB
PowerShell
125 lines
4.3 KiB
PowerShell
# Copyright (c) Microsoft Corporation.
|
|
# Licensed under the MIT License.
|
|
|
|
Describe -Name "Windows MSI" -Fixture {
|
|
BeforeAll {
|
|
function Test-Elevated {
|
|
[CmdletBinding()]
|
|
[OutputType([bool])]
|
|
Param()
|
|
|
|
# if the current Powershell session was called with administrator privileges,
|
|
# the Administrator Group's well-known SID will show up in the Groups for the current identity.
|
|
# Note that the SID won't show up unless the process is elevated.
|
|
return (([Security.Principal.WindowsIdentity]::GetCurrent()).Groups -contains "S-1-5-32-544")
|
|
}
|
|
|
|
function Invoke-Msiexec {
|
|
param(
|
|
[Parameter(ParameterSetName = 'Install', Mandatory)]
|
|
[Switch]$Install,
|
|
|
|
[Parameter(ParameterSetName = 'Uninstall', Mandatory)]
|
|
[Switch]$Uninstall,
|
|
|
|
[Parameter(Mandatory)]
|
|
[ValidateScript({Test-Path -Path $_})]
|
|
[String]$MsiPath,
|
|
|
|
[Parameter(ParameterSetName = 'Install')]
|
|
[HashTable] $Properties
|
|
|
|
)
|
|
$action = "$($PSCmdlet.ParameterSetName)ing"
|
|
if ($Install.IsPresent) {
|
|
$switch = '/I'
|
|
} else {
|
|
$switch = '/x'
|
|
}
|
|
|
|
$additionalOptions = @()
|
|
if ($Properties) {
|
|
foreach ($key in $Properties.Keys) {
|
|
$additionalOptions += "$key=$($Properties.$key)"
|
|
}
|
|
}
|
|
|
|
$argumentList = "$switch $MsiPath /quiet /l*vx $msiLog $additionalOptions"
|
|
$msiExecProcess = Start-Process msiexec.exe -Wait -ArgumentList $argumentList -NoNewWindow -PassThru
|
|
if ($msiExecProcess.ExitCode -ne 0) {
|
|
$exitCode = $msiExecProcess.ExitCode
|
|
throw "$action MSI failed and returned error code $exitCode."
|
|
}
|
|
}
|
|
|
|
$msiX64Path = $env:PsMsiX64Path
|
|
|
|
# Get any existing powershell in the path
|
|
$beforePath = @(([System.Environment]::GetEnvironmentVariable('PATH', 'MACHINE')) -split ';' |
|
|
Where-Object {$_ -like '*files\powershell*'})
|
|
|
|
$msiLog = Join-Path -Path $TestDrive -ChildPath 'msilog.txt'
|
|
|
|
foreach ($pathPart in $beforePath) {
|
|
Write-Warning "Found existing PowerShell path: $pathPart"
|
|
}
|
|
|
|
if (!(Test-Elevated)) {
|
|
Write-Warning "Tests must be elevated"
|
|
}
|
|
$uploadedLog = $false
|
|
}
|
|
BeforeEach {
|
|
$error.Clear()
|
|
}
|
|
AfterEach {
|
|
if ($error.Count -ne 0 -and !$uploadedLog) {
|
|
Copy-Item -Path $msiLog -Destination $env:temp -Force
|
|
Write-Verbose "MSI log is at $env:temp\msilog.txt" -Verbose
|
|
$uploadedLog = $true
|
|
}
|
|
}
|
|
|
|
Context "Add Path disabled" {
|
|
It "MSI should install without error" -Skip:(!(Test-Elevated)) {
|
|
{
|
|
Invoke-MsiExec -Install -MsiPath $msiX64Path -Properties @{ADD_PATH = 0}
|
|
} | Should -Not -Throw
|
|
}
|
|
|
|
It "MSI should have not be updated path" -Skip:(!(Test-Elevated)) {
|
|
$psPath = ([System.Environment]::GetEnvironmentVariable('PATH', 'MACHINE')) -split ';' |
|
|
Where-Object {$_ -like '*files\powershell*' -and $_ -notin $beforePath}
|
|
|
|
$psPath | Should -BeNullOrEmpty
|
|
}
|
|
|
|
It "MSI should uninstall without error" -Skip:(!(Test-Elevated)) {
|
|
{
|
|
Invoke-MsiExec -Uninstall -MsiPath $msiX64Path
|
|
} | Should -Not -Throw
|
|
}
|
|
}
|
|
|
|
Context "Add Path enabled" {
|
|
It "MSI should install without error" -Skip:(!(Test-Elevated)) {
|
|
{
|
|
Invoke-MsiExec -Install -MsiPath $msiX64Path -Properties @{ADD_PATH = 1}
|
|
} | Should -Not -Throw
|
|
}
|
|
|
|
It "MSI should have updated path" -Skip:(!(Test-Elevated)) {
|
|
$psPath = ([System.Environment]::GetEnvironmentVariable('PATH', 'MACHINE')) -split ';' |
|
|
Where-Object {$_ -like '*files\powershell*' -and $_ -notin $beforePath}
|
|
|
|
$psPath | Should -Not -BeNullOrEmpty
|
|
}
|
|
|
|
It "MSI should uninstall without error" -Skip:(!(Test-Elevated)) {
|
|
{
|
|
Invoke-MsiExec -Uninstall -MsiPath $msiX64Path
|
|
} | Should -Not -Throw
|
|
}
|
|
}
|
|
}
|