feat: Enhance test coverage for various modules by adding new test cases and improving existing ones

Author: stiwicourage

tests/private/build/BuildManifest.Tests.ps1

@@ -42,25 +42,27 @@ Describe 'Build-Manifest' {
         $manifest.Description | Should -Be 'A module'
     }
 
-    It 'adds resources/ paths when CopyResourcesToModuleRoot is false' {
+    It 'records resource manifest entries for CopyResourcesToModuleRoot=<CopyResourcesToModuleRoot>' -ForEach @(
+        @{
+            CopyResourcesToModuleRoot = $false
+            ExpectedFormat = 'resources/MyFormat.Format.ps1xml'
+            ExpectedType = 'resources/MyTypes.Types.ps1xml'
+        }
+        @{
+            CopyResourcesToModuleRoot = $true
+            ExpectedFormat = 'MyFormat.Format.ps1xml'
+            ExpectedType = 'MyTypes.Types.ps1xml'
+        }
+    ) {
+        $script:ctx.CopyResourcesToModuleRoot = $CopyResourcesToModuleRoot
         Set-Content -Path (Join-Path $script:ctx.ResourcesDir 'MyFormat.Format.ps1xml') -Value '<x/>'
         Set-Content -Path (Join-Path $script:ctx.ResourcesDir 'MyTypes.Types.ps1xml') -Value '<x/>'
         Mock Get-NovaBuildProjectInfo { $script:ctx }
         Mock Assert-ManifestSchema {}
         Build-Manifest -ProjectInfo ([pscustomobject]@{})
         $manifest = Import-PowerShellDataFile -Path $script:ctx.ManifestFilePSD1
-        ($manifest.FormatsToProcess -join ',') | Should -Match 'resources'
-        ($manifest.TypesToProcess -join ',') | Should -Match 'resources'
-    }
-
-    It 'adds bare filenames when CopyResourcesToModuleRoot is true' {
-        $script:ctx.CopyResourcesToModuleRoot = $true
-        Set-Content -Path (Join-Path $script:ctx.ResourcesDir 'MyFormat.Format.ps1xml') -Value '<x/>'
-        Mock Get-NovaBuildProjectInfo { $script:ctx }
-        Mock Assert-ManifestSchema {}
-        Build-Manifest -ProjectInfo ([pscustomobject]@{})
-        $manifest = Import-PowerShellDataFile -Path $script:ctx.ManifestFilePSD1
-        $manifest.FormatsToProcess | Should -Be 'MyFormat.Format.ps1xml'
+        $manifest.FormatsToProcess | Should -Be $ExpectedFormat
+        $manifest.TypesToProcess | Should -Be $ExpectedType
     }
 
     It 'sets Prerelease when version has a prerelease label' {

tests/private/cli/FormatNovaCliCommandResult.Tests.ps1

@@ -54,6 +54,11 @@ Describe 'Format-NovaCliCommandResult' {
         Format-NovaCliCommandResult -Command 'build' -Result 'raw' | Should -Be 'raw'
     }
 
+    It 'formats version update results for the bump command' {
+        $result = [pscustomobject]@{Applied=$true;PreviousVersion='1.0.0';NewVersion='1.1.0';Label='minor';CommitCount=4}
+        Format-NovaCliCommandResult -Command 'bump' -Result $result | Should -Be (Format-NovaCliVersionUpdateResult -Result $result)
+    }
+
     It 'passes through a structured result that does not match update or bump shapes' {
         $result = [pscustomobject]@{Custom = 'value'}
         Format-NovaCliCommandResult -Command 'package' -Result $result | Should -Be $result

tests/private/cli/ReadAwesomeStandardPrompt.Tests.ps1

@@ -11,4 +11,23 @@ Describe 'Read-AwesomeStandardPrompt' {
         $hostUi | Add-Member -MemberType ScriptMethod -Name Prompt -Value {param($caption, $message, $fields) return [pscustomobject]@{Values = 'x'}}
         Read-AwesomeStandardPrompt -Ask @{} -HostUi $hostUi | Should -Be 'final'
     }
+
+    It 'writes the retry message before prompting again when validation fails once' {
+        $script:promptCallCount = 0
+        $hostUi = [pscustomobject]@{}
+        $hostUi | Add-Member -MemberType ScriptMethod -Name Prompt -Value {
+            param($caption, $message, $fields)
+            $script:promptCallCount += 1
+            return [pscustomobject]@{Attempt = $script:promptCallCount}
+        }
+
+        Mock Test-AwesomePromptRequiresRetry {
+            return $Response.Attempt -eq 1
+        }
+        Mock Write-AwesomePromptRetryMessage {}
+
+        Read-AwesomeStandardPrompt -Ask @{} -HostUi $hostUi | Should -Be 'final'
+        $script:promptCallCount | Should -Be 2
+        Should -Invoke Write-AwesomePromptRetryMessage -Times 1
+    }
 }

tests/private/release/InitiateGitRepo.Tests.ps1

@@ -35,6 +35,12 @@ Describe 'New-InitiateGitRepo' {
         Mock Invoke-NovaGitCommand {throw 'kaboom'}
         {New-InitiateGitRepo -DirectoryPath $script:dir} | Should -Throw '*kaboom*'
     }
+
+    It 'writes a success verbose message when git init succeeds' {
+        Mock Invoke-NovaGitCommand {return [pscustomobject]@{ExitCode=0; Output=@()}}
+        $verbose = New-InitiateGitRepo -DirectoryPath $script:dir -Confirm:$false -Verbose 4>&1
+        @($verbose | Where-Object Message -EQ 'Git repository initialized successfully').Count | Should -Be 1
+    }
 }
 
 Describe 'Get-NovaGitInitializationFailureMessage' {

tests/private/release/InvokeNovaVersionUpdateWorkflow.Tests.ps1

@@ -34,6 +34,12 @@ Describe 'Get-NovaVersionUpdateAdvisoryMessage' {
     }
 }
 
+Describe 'Get-NovaVersionUpdateProjectFile' {
+    It 'returns null when the workflow context has no ProjectInfo' {
+        Get-NovaVersionUpdateProjectFile -WorkflowContext ([pscustomobject]@{Other = 1}) | Should -BeNullOrEmpty
+    }
+}
+
 Describe 'Get-NovaVersionUpdateResult' {
     It 'assembles a structured result' {
         $ctx = [pscustomobject]@{

tests/private/release/PublishNovaBuiltModule.Tests.ps1

@@ -38,4 +38,13 @@ Describe 'Publish-NovaBuiltModule' {
         Publish-NovaBuiltModule -ProjectInfo $script:project
         Should -Invoke Publish-NovaBuiltModuleToDirectory -Times 1 -ParameterFilter {$ModuleDirectoryPath -eq '/local/resolved'}
     }
+
+    It 'uses Get-NovaProjectInfo when ProjectInfo is not supplied' {
+        Mock Get-NovaProjectInfo { return $script:project }
+        Mock Publish-NovaBuiltModuleToDirectory {}
+        Mock Resolve-NovaLocalPublishPath {return '/local/resolved'}
+        Publish-NovaBuiltModule
+        Should -Invoke Get-NovaProjectInfo -Times 1
+        Should -Invoke Publish-NovaBuiltModuleToDirectory -Times 1 -ParameterFilter {$ProjectInfo -eq $script:project}
+    }
 }

tests/private/shared/GetResourceFilePath.Tests.ps1

@@ -30,4 +30,13 @@ Describe 'Get-ResourceFilePath' {
 
         {Get-ResourceFilePath -FileName 'Missing.json'} | Should -Throw
     }
+
+    It 'writes a verbose fallback message when project resource discovery throws' {
+        Mock Get-NovaProjectInfo { throw 'no project context' }
+        Mock Test-Path { return $true }
+
+        $output = Get-ResourceFilePath -FileName 'Schema.json' -Verbose 4>&1
+        ($output | Out-String) | Should -Match 'Project resource discovery unavailable'
+        ($output | Out-String) | Should -Match 'Schema.json'
+    }
 }

tests/private/shared/ImportNovaBuiltModuleForCi.Tests.ps1

@@ -36,4 +36,20 @@ Describe 'Import-NovaBuiltModuleForCi' {
 
         {Import-NovaBuiltModuleForCi -ProjectRoot '/p'} | Should -Throw
     }
+
+    It 'refreshes the imported module when the built manifest exists' {
+        $projectInfo = [pscustomobject]@{ProjectName = 'Demo'; OutputModuleDir = '/dist/Demo'}
+        Mock Get-NovaBuiltModuleManifestPathForCi { return '/dist/Demo/Demo.psd1' }
+        Mock Test-Path { return $true }
+        Mock Get-Module { return @([pscustomobject]@{Name = 'Demo'}) }
+        Mock Remove-Module {}
+        Mock Import-Module { return 'imported-module' }
+
+        $result = Import-NovaBuiltModuleForCi -ProjectInfo $projectInfo
+
+        $result | Should -Be 'imported-module'
+        Should -Invoke Get-Module -Times 1 -ParameterFilter { $Name -eq 'Demo' -and $All }
+        Should -Invoke Remove-Module -Times 1
+        Should -Invoke Import-Module -Times 1 -ParameterFilter { $Name -eq '/dist/Demo/Demo.psd1' -and $Force -and $Global -and $PassThru }
+    }
 }

tests/public/InitializeNovaModule.Tests.ps1

@@ -23,4 +23,9 @@ Describe 'Initialize-NovaModule' {
         Initialize-NovaModule -Path '/tmp/x' -WhatIf
         $script:workflowCalled | Should -BeFalse
     }
+
+    It 'defaults Path from the current location when Path is omitted' {
+        Initialize-NovaModule
+        $script:ctxArgs.Path | Should -Be (Get-Location).Path
+    }
 }

tests/public/UpdateNovaModuleVersion.TestSupport.ps1

@@ -2,7 +2,7 @@ function Invoke-NovaVersionUpdateCiActivation {param($ProjectRoot, $Parameters,
     return $script:ciActivation
 }
 function Get-NovaVersionUpdateWorkflowContext {param($ProjectRoot, [switch]$PreviewRelease, [switch]$ContinuousIntegrationRequested, [switch]$OverrideWarningRequested)
-    $script:ctxArgs = @{Preview=[bool]$PreviewRelease; CI=[bool]$ContinuousIntegrationRequested; Override=[bool]$OverrideWarningRequested}
+    $script:ctxArgs = @{ProjectRoot=$ProjectRoot; Preview=[bool]$PreviewRelease; CI=[bool]$ContinuousIntegrationRequested; Override=[bool]$OverrideWarningRequested}
     return [pscustomobject]@{Target=$ProjectRoot; Action='Bump'}
 }
 function Invoke-NovaVersionUpdateWorkflow {param($WorkflowContext, [switch]$ShouldRun, [switch]$WhatIfEnabled)