8000 Added RequireSudoOnUnix tags to PowerShellGet tests and unmark as pending by RDIL · Pull Request #8954 · PowerShell/PowerShell · GitHub
[go: up one dir, main page]
Skip to content

Added RequireSudoOnUnix tags to PowerShellGet tests and unmark as pending #8954

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
adityapatwardhan merged 14 commits into from
Mar 5, 2019
Merged

Added RequireSudoOnUnix tags to PowerShellGet tests and unmark as pending #8954

Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
a2c9455
Update comments in PowerShellGet tests for Travis CI
RDIL Feb 22, 2019
cb9fc8c
Update PowerShellGet.Tests.ps1
RDIL Feb 25, 2019
3cadc25
[feature]
RDIL Feb 25, 2019
190f9cc
Revert
RDIL Feb 25, 2019
9b4aa57
[feature] RequireSudoOnUnix
RDIL Feb 25, 2019
71cd7e1
Update PowerShellGet.Tests.ps1
RDIL Feb 28, 2019
15c6042
Fix
RDIL Feb 28, 2019
389f53b
Fixed tests!
RDIL Feb 28, 2019
78a9693
Revert
RDIL Feb 28, 2019
482d685
Missed one
RDIL Feb 28, 2019
fe89291
[feature] empty commit to trigger feature tests
RDIL Feb 28, 2019
81a64ec
Merge remote-tracking branch 'upstream/master' into patch-1
RDIL Mar 4, 2019
f3d1889
Add space where it should be
RDIL Mar 4, 2019
254fa96
[feature] get cla bot to work (empty commit)
RDIL Mar 5, 2019
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
12 changes: 5 additions & 7 deletions test/powershell/Modules/PowerShellGet/PowerShellGet.Tests.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ if(IsInbox)
{
$script:ProgramFilesPSPath = Microsoft.PowerShell.Management\Join-Path -Path $env:ProgramFiles -ChildPath "WindowsPowerShell"
}
elseif(IsCoreCLR){
elseif(IsCoreCLR) {
if(IsWindows) {
$script:ProgramFilesPSPath = Microsoft.PowerShell.Management\Join-Path -Path $env:ProgramFiles -ChildPath 'PowerShell'
}
Expand Down Expand Up @@ -145,7 +145,7 @@ Describe "PowerShellGet - Module tests" -tags "Feature" {
}
}

Describe "PowerShellGet - Module tests (Admin)" -tags @('Feature', 'RequireAdminOnWindows') {
Describe "PowerShellGet - Module tests (Admin)" -Tags @('Feature', 'RequireAdminOnWindows', 'RequireSudoOnUnix') {

BeforeAll {
if ($script:Initialized -eq $false) {
Expand All @@ -158,8 +158,7 @@ Describe "PowerShellGet - Module tests (Admin)" -tags @('Feature', 'RequireAdmin
Remove-InstalledModules
}

## Marked as 'Pending' on Linux for now because the test requires root privilege but we cannot do it now in our Travis CI Linux build
It "Should install a module correctly to the required location with AllUsers scope" -Pending:$IsLinux {
It "Should install a module correctly to the required location with AllUsers scope" {
Install-Module -Name $ContosoServer -Repository $RepositoryName -Scope AllUsers
$installedModuleInfo = Get-InstalledModule -Name $ContosoServer

Expand Down Expand Up @@ -215,7 +214,7 @@ Describe "PowerShellGet - Script tests" -tags "Feature" {
}
}

Describe "PowerShellGet - Script tests (Admin)" -tags @('Feature', 'RequireAdminOnWindows') {
Describe "PowerShellGet - Script tests (Admin)" -Tags @('Feature', 'RequireAdminOnWindows', 'RequireSudoOnUnix') {

BeforeAll {
if ($script:Initialized -eq $false) {
Expand All @@ -228,8 +227,7 @@ Describe "PowerShellGet - Script tests (Admin)" -tags @('Feature', 'RequireAdmin
Remove-InstalledScripts
}

## Marked as 'Pending' on Linux for now because the test requires root privilege but we cannot do it now in our Travis CI Linux build
It "Should install a script correctly to the required location with AllUsers scope" -Pending:$IsLinux {
It "Should install a script correctly to the required location with AllUsers scope" {
Install-Script -Name $FabrikamServerScript -Repository $RepositoryName -NoPathUpdate -Scope AllUsers
$installedScriptInfo = Get-InstalledScript -Name $FabrikamServerScript

Expand Down
0