Skip to content

Tests for Get-Module #27790

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

Open
Guillermo López-Anglada (guillermooo) wants to merge 16 commits into
base: master
Choose a base branch
from
+91 −0
Open

Tests for Get-Module #27790

Changes from all commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
7327290
Add `Describe`
guillermooo Aug 9, 2026
f542dbe
Add `BeforeAll`
guillermooo Aug 9, 2026
b4f6c5c
Add test
guillermooo Aug 9, 2026
a8390fe
Add test
guillermooo Aug 9, 2026
2eaf439
Add test
guillermooo Aug 9, 2026
e946e4d
Add test
guillermooo Aug 9, 2026
8448b34
Add new `Context`
guillermooo Aug 9, 2026
231af68
Add `BeforeAll` and `AfterAll`
guillermooo Aug 9, 2026
6b75ee8
Add test
guillermooo Aug 9, 2026
2b52fb6
Add test
guillermooo Aug 9, 2026
6258e72
Review: Avoid Windows-only convention
guillermooo Aug 10, 2026
95e9342
Review: Use `Join-Path`
guillermooo Aug 10, 2026
8dbc54d
Review: Fix `Should -Throw` tests
guillermooo Aug 10, 2026
bb7d1db
Review: Typo
guillermooo Aug 10, 2026
0b6503e
Review: Use dedicated $env:PSModulePath for tests
guillermooo Aug 10, 2026
c6d6a31
Refactor
guillermooo Aug 10, 2026
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
91 changes: 91 additions & 0 deletions test/powershell/Modules/Microsoft.PowerShell.Core/Get-Module.Tests.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -280,6 +280,97 @@ Describe "Get-Module -ListAvailable" -Tags "CI" {
}
}

Describe 'Get-Module -ListAvailable -(FullyQualifiedName|Name) <path> when argument is absolute path' -Tags "CI" {
BeforeAll {
$oldPSModulePath = $env:PSModulePath
$psModulePath = New-Item -ItemType Directory (Join-Path $TestDrive modules)
$env:PSModulePath = $psModulePath
}

AfterAll {
$env:PSModulePath = $oldPSModulePath
}

It 'wrongly returns module information instead of $null or error for missing script module' {
$path = [System.IO.Path]::GetFullPath((Join-Path $pwd missing.psm1))
Test-Path $path | Should -BeFalse
Get-Module -ListAvailable -FullyQualifiedName $path | Should -BeOfType ([System.Management.Automation.PSModuleInfo])
Get-Module -ListAvailable -Name $path | Should -BeOfType ([System.Management.Automation.PSModuleInfo])
}

It 'wrongly returns module information instead of $null or error for missing script module under $env:PSModulePath' {
$path = [System.IO.Path]::GetFullPath((Join-Path $psModulePath missing.psm1))
Test-Path $path | Should -BeFalse
Get-Module -ListAvailable -FullyQualifiedName $path | Should -BeOfType ([System.Management.Automation.PSModuleInfo])
Get-Module -ListAvailable -Name $path | Should -BeOfType ([System.Management.Automation.PSModuleInfo])
}

It 'writes error for missing manifest module' {
$path = [System.IO.Path]::GetFullPath((Join-Path $pwd missing))
Test-Path $path | Should -BeFalse

$err = { Get-Module -ListAvailable -FullyQualifiedName $path -ErrorAction Stop } | Should -Throw -PassThru
$err.Exception.Message | Should -BeLike '*Update the Name parameter*'

$err = { Get-Module -ListAvailable -Name $path -ErrorAction Stop } | Should -Throw -PassThru
$err.Exception.Message | Should -BeLike '*Update the Name parameter*'
}

It 'writes error for missing manifest module under $env:PSModulePath' {
$path = [System.IO.Path]::GetFullPath((Join-Path $psModulePath missing))
Test-Path $path | Should -BeFalse

$err = { Get-Module -ListAvailable -FullyQualifiedName $path -ErrorAction Stop } | Should -Throw -PassThru
$err.Exception.Message | Should -BeLike '*Update the Name parameter*'

$err = { Get-Module -ListAvailable -Name $path -ErrorAction Stop } | Should -Throw -PassThru
$err.Exception.Message | Should -BeLike '*Update the Name parameter*'
}

Context 'Locating existing script module' {
BeforeAll {
$psModulePath = $env:PSModulePath

# Script modules
#
# Under $env:PSModulePath. TODO: Is this a supported scenario?
$inPSModulePathLooseFilePath = Join-Path $psModulePath 'loose.psm1'
New-Item -ItemType File -Force $inPSModulePathLooseFilePath > $null
#
# Under $pwd
$inPSModulePathLooseFilePathPwd = Join-Path $pwd 'loose.psm1'
New-Item -ItemType File -Force $inPSModulePathLooseFilePathPwd > $null
Comment thread
guillermooo marked this conversation as resolved.
}

AfterAll {
Remove-Item $inPSModulePathLooseFilePath
Remove-Item $inPSModulePathLooseFilePathPwd
}

# TODO: This looks like a bug.
It 'wrongly writes error instead of returning module information for existing script module under $env:PSModulePath using basename' {
Test-Path (Join-Path $psModulePath loose.psm1) | Should -BeTrue

$err = { Get-Module -ListAvailable -Name (Join-Path $psModulePath loose) -ErrorAction Stop } | Should -Throw -PassThru
$err.Exception.Message | Should -BeLike '*Update the Name parameter*'

$err = { Get-Module -ListAvailable -FullyQualifiedName (Join-Path $psModulePath loose) -ErrorAction Stop } | Should -Throw -PassThru
$err.Exception.Message | Should -BeLike '*Update the Name parameter*'
}

# TODO: This looks like a bug.
It 'wrongly writes error instead of returning module information for existing script module using basename' {
Test-Path (Join-Path $pwd loose.psm1) | Should -BeTrue

$err = { Get-Module -ListAvailable -Name (Join-Path $pwd loose) -ErrorAction Stop } | Should -Throw -PassThru
$err.Exception.Message | Should -BeLike '*Update the Name parameter*'

$err = { Get-Module -ListAvailable -FullyQualifiedName (Join-Path $pwd loose) -ErrorAction Stop } | Should -Throw -PassThru
$err.Exception.Message | Should -BeLike '*Update the Name parameter*'
}
}
}

Describe 'Get-Module -ListAvailable with path' -Tags "CI" {
BeforeAll {
$moduleName = 'Banana'
Expand Down