Skip to content

Add fix for failing unix tests #426

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jul 29, 2021
Merged

Add fix for failing unix tests #426

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
3edb51d
Add fix for failing MacOS tests
alerickson Jul 28, 2021
df89586
Add describe block for non-CI run tests
alerickson Jul 28, 2021
c85315f
Merge branch 'master' of https://github.com/PowerShell/PowerShellGet …
alerickson Jul 29, 2021
082bb6d
Add new line to end of file
alerickson Jul 29, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
45 changes: 32 additions & 13 deletions test/InstallPSResource.Tests.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -154,16 +154,7 @@ Describe 'Test Install-PSResource for Module' {
Install-PSResource -Name "TestModule" -Repository $TestGalleryName -Scope CurrentUser
$pkg = Get-Module "TestModule" -ListAvailable
$pkg.Name | Should -Be "TestModule"
$pkg.Path.Contains("/home/") | Should -Be $true
}

# Unix only
# Expected path should be similar to: '/usr/local/share/powershell/Modules'
It "Install resource under AllUsers scope - Unix only" -Skip:(Get-IsWindows) {
Install-PSResource -Name "TestModule" -Repository $TestGalleryName -Scope AllUsers
$pkg = Get-Module "TestModule" -ListAvailable
$pkg.Name | Should -Be "TestModule"
$pkg.Path.Contains("/usr/") | Should -Be $true
$pkg.Path.Contains("$env:HOME/.local") | Should -Be $true
}

# Unix only
Expand All @@ -172,7 +163,7 @@ Describe 'Test Install-PSResource for Module' {
Install-PSResource -Name "TestModule" -Repository $TestGalleryName
$pkg = Get-Module "TestModule" -ListAvailable
$pkg.Name | Should -Be "TestModule"
$pkg.Path.Contains("/home/") | Should -Be $true
$pkg.Path.Contains("$env:HOME/.local") | Should -Be $true
}

It "Should not install resource that is already installed" {
Expand Down Expand Up @@ -235,8 +226,36 @@ Describe 'Test Install-PSResource for Module' {
$pkg | Should -Not -BeNullOrEmpty
$pkg.Name | Should -Be $publishModuleName
}
}

<# Temporarily commented until -Tag is implemented for this Describe block
Describe 'Test Install-PSResource for interactive and root user scenarios' {

BeforeAll{
$TestGalleryName = Get-PoshTestGalleryName
$PSGalleryName = Get-PSGalleryName
$NuGetGalleryName = Get-NuGetGalleryName
Get-NewPSResourceRepositoryFile
Register-LocalRepos
}

AfterEach {
Uninstall-PSResource "TestModule", "testModuleWithlicense" -Force -ErrorAction SilentlyContinue
}

AfterAll {
Get-RevertPSResourceRepositoryFile
}

# Unix only manual test
# Expected path should be similar to: '/usr/local/share/powershell/Modules'
It "Install resource under AllUsers scope - Unix only" -Skip:(Get-IsWindows) {
Install-PSResource -Name "TestModule" -Repository $TestGalleryName -Scope AllUsers
$pkg = Get-Module "TestModule" -ListAvailable
$pkg.Name | Should -Be "TestModule"
$pkg.Path.Contains("/usr/") | Should -Be $true
}

<#
# This needs to be manually tested due to prompt
It "Install resource that requires accept license without -AcceptLicense flag" {
Install-PSResource -Name "testModuleWithlicense" -Repository $TestGalleryName
Expand All @@ -256,5 +275,5 @@ Describe 'Test Install-PSResource for Module' {

Set-PSResourceRepository PoshTestGallery -Trusted
}
#>
}
#>