diff --git a/action.yml b/action.yml index 74936e66..27011343 100644 --- a/action.yml +++ b/action.yml @@ -291,9 +291,9 @@ runs: WorkingDirectory: ${{ inputs.WorkingDirectory }} with: Debug: ${{ inputs.Debug }} - Prerelease: ${{ inputs.Prerelease }} Verbose: ${{ inputs.Verbose }} Version: ${{ inputs.Version }} + Prerelease: ${{ inputs.Prerelease }} WorkingDirectory: ${{ inputs.WorkingDirectory }} Path: ${{ steps.paths.outputs.TestPath }} StepSummary_Mode: ${{ inputs.StepSummary_Mode }} diff --git a/scripts/tests/Module/Module.Configuration.ps1 b/scripts/tests/Module/Module.Configuration.ps1 index e76f767f..de7fdea7 100644 --- a/scripts/tests/Module/Module.Configuration.ps1 +++ b/scripts/tests/Module/Module.Configuration.ps1 @@ -1,8 +1,11 @@ @{ - TestResult = @{ + TestResult = @{ Enabled = $true } - Output = @{ + CodeCoverage = @{ + Enabled = $true + } + Output = @{ Verbosity = 'Detailed' } } diff --git a/scripts/tests/Module/PSModule/PSModule.Container.ps1 b/scripts/tests/Module/PSModule/PSModule.Container.ps1 index 31d42d3c..05395f66 100644 --- a/scripts/tests/Module/PSModule/PSModule.Container.ps1 +++ b/scripts/tests/Module/PSModule/PSModule.Container.ps1 @@ -1,8 +1,6 @@ @{ Path = Get-ChildItem -Path $PSScriptRoot -Filter *.Tests.ps1 | Select-Object -ExpandProperty FullName Data = @{ - Path = $env:PSMODULE_INVOKE_PESTER_INPUT_Run_Path - Debug = $false - Verbose = $false + Path = $env:PSMODULE_INVOKE_PESTER_INPUT_Run_Path } } diff --git a/scripts/tests/Module/PSModule/PSModule.Tests.ps1 b/scripts/tests/Module/PSModule/PSModule.Tests.ps1 index 1253ce7f..f355e72e 100644 --- a/scripts/tests/Module/PSModule/PSModule.Tests.ps1 +++ b/scripts/tests/Module/PSModule/PSModule.Tests.ps1 @@ -2,6 +2,10 @@ 'PSReviewUnusedParameter', 'Path', Justification = 'Path is used to specify the path to the module to test.' )] +[Diagnostics.CodeAnalysis.SuppressMessageAttribute( + 'PSAvoidUsingWriteHost', '', + Justification = 'Log outputs to GitHub Actions logs.' +)] [CmdLetBinding()] Param( [Parameter(Mandatory)] @@ -23,14 +27,18 @@ Describe 'PSModule - Module tests' { Context 'Module Manifest' { It 'Module Manifest exists' { - $result = Test-Path -Path $moduleManifestPath - $result | Should -Be $true - Write-Verbose $result + LogGroup 'Module manifest' { + $result = Test-Path -Path $moduleManifestPath + $result | Should -Be $true + Write-Host "$($result | Format-List | Out-String)" + } } It 'Module Manifest is valid' { - $result = Test-ModuleManifest -Path $moduleManifestPath - $result | Should -Not -Be $null - Write-Verbose $result + LogGroup 'Validating Module Manifest' { + $result = Test-ModuleManifest -Path $moduleManifestPath + $result | Should -Not -Be $null + Write-Host "$($result | Format-List | Out-String)" + } } # It 'has a valid license URL' {} # It 'has a valid project URL' {} diff --git a/scripts/tests/SourceCode/PSModule/PSModule.Container.ps1 b/scripts/tests/SourceCode/PSModule/PSModule.Container.ps1 index 1f425cff..dbfef337 100644 --- a/scripts/tests/SourceCode/PSModule/PSModule.Container.ps1 +++ b/scripts/tests/SourceCode/PSModule/PSModule.Container.ps1 @@ -3,7 +3,5 @@ Data = @{ Path = $env:PSMODULE_INVOKE_PESTER_INPUT_Run_Path TestsPath = $env:LocalTestPath - Debug = $false - Verbose = $false } } diff --git a/tests/outputTestRepo/outputs/module/PSModuleTest/PSModuleTest.psm1 b/tests/outputTestRepo/outputs/module/PSModuleTest/PSModuleTest.psm1 index 3553dd95..63117b65 100644 --- a/tests/outputTestRepo/outputs/module/PSModuleTest/PSModuleTest.psm1 +++ b/tests/outputTestRepo/outputs/module/PSModuleTest/PSModuleTest.psm1 @@ -3,42 +3,42 @@ param() $scriptName = $MyInvocation.MyCommand.Name -Write-Verbose "[$scriptName] Importing module" +Write-Debug "[$scriptName] Importing module" #region - Data import -Write-Verbose "[$scriptName] - [data] - Processing folder" +Write-Debug "[$scriptName] - [data] - Processing folder" $dataFolder = (Join-Path $PSScriptRoot 'data') -Write-Verbose "[$scriptName] - [data] - [$dataFolder]" +Write-Debug "[$scriptName] - [data] - [$dataFolder]" Get-ChildItem -Path "$dataFolder" -Recurse -Force -Include '*.psd1' -ErrorAction SilentlyContinue | ForEach-Object { - Write-Verbose "[$scriptName] - [data] - [$($_.Name)] - Importing" + Write-Debug "[$scriptName] - [data] - [$($_.Name)] - Importing" New-Variable -Name $_.BaseName -Value (Import-PowerShellDataFile -Path $_.FullName) -Force - Write-Verbose "[$scriptName] - [data] - [$($_.Name)] - Done" + Write-Debug "[$scriptName] - [data] - [$($_.Name)] - Done" } -Write-Verbose "[$scriptName] - [data] - Done" +Write-Debug "[$scriptName] - [data] - Done" #endregion - Data import #region - From /init -Write-Verbose "[$scriptName] - [/init] - Processing folder" +Write-Debug "[$scriptName] - [/init] - Processing folder" #region - From /init/initializer.ps1 -Write-Verbose "[$scriptName] - [/init/initializer.ps1] - Importing" +Write-Debug "[$scriptName] - [/init/initializer.ps1] - Importing" Write-Verbose '-------------------------------' Write-Verbose '--- THIS IS AN INITIALIZER ---' Write-Verbose '-------------------------------' -Write-Verbose "[$scriptName] - [/init/initializer.ps1] - Done" +Write-Debug "[$scriptName] - [/init/initializer.ps1] - Done" #endregion - From /init/initializer.ps1 -Write-Verbose "[$scriptName] - [/init] - Done" +Write-Debug "[$scriptName] - [/init] - Done" #endregion - From /init #region - From /classes -Write-Verbose "[$scriptName] - [/classes] - Processing folder" +Write-Debug "[$scriptName] - [/classes] - Processing folder" #region - From /classes/Book.ps1 -Write-Verbose "[$scriptName] - [/classes/Book.ps1] - Importing" +Write-Debug "[$scriptName] - [/classes/Book.ps1] - Importing" class Book { # Class properties @@ -86,10 +86,10 @@ class Book { } } -Write-Verbose "[$scriptName] - [/classes/Book.ps1] - Done" +Write-Debug "[$scriptName] - [/classes/Book.ps1] - Done" #endregion - From /classes/Book.ps1 #region - From /classes/BookList.ps1 -Write-Verbose "[$scriptName] - [/classes/BookList.ps1] - Importing" +Write-Debug "[$scriptName] - [/classes/BookList.ps1] - Importing" class BookList { # Static property to hold the list of books @@ -178,17 +178,17 @@ class BookList { } } -Write-Verbose "[$scriptName] - [/classes/BookList.ps1] - Done" +Write-Debug "[$scriptName] - [/classes/BookList.ps1] - Done" #endregion - From /classes/BookList.ps1 -Write-Verbose "[$scriptName] - [/classes] - Done" +Write-Debug "[$scriptName] - [/classes] - Done" #endregion - From /classes #region - From /private -Write-Verbose "[$scriptName] - [/private] - Processing folder" +Write-Debug "[$scriptName] - [/private] - Processing folder" #region - From /private/Get-InternalPSModule.ps1 -Write-Verbose "[$scriptName] - [/private/Get-InternalPSModule.ps1] - Importing" +Write-Debug "[$scriptName] - [/private/Get-InternalPSModule.ps1] - Importing" Function Get-InternalPSModule { <# @@ -209,10 +209,10 @@ Function Get-InternalPSModule { Write-Output "Hello, $Name!" } -Write-Verbose "[$scriptName] - [/private/Get-InternalPSModule.ps1] - Done" +Write-Debug "[$scriptName] - [/private/Get-InternalPSModule.ps1] - Done" #endregion - From /private/Get-InternalPSModule.ps1 #region - From /private/Set-InternalPSModule.ps1 -Write-Verbose "[$scriptName] - [/private/Set-InternalPSModule.ps1] - Importing" +Write-Debug "[$scriptName] - [/private/Set-InternalPSModule.ps1] - Importing" Function Set-InternalPSModule { <# @@ -237,17 +237,17 @@ Function Set-InternalPSModule { Write-Output "Hello, $Name!" } -Write-Verbose "[$scriptName] - [/private/Set-InternalPSModule.ps1] - Done" +Write-Debug "[$scriptName] - [/private/Set-InternalPSModule.ps1] - Done" #endregion - From /private/Set-InternalPSModule.ps1 -Write-Verbose "[$scriptName] - [/private] - Done" +Write-Debug "[$scriptName] - [/private] - Done" #endregion - From /private #region - From /public -Write-Verbose "[$scriptName] - [/public] - Processing folder" +Write-Debug "[$scriptName] - [/public] - Processing folder" #region - From /public/Get-PSModuleTest.ps1 -Write-Verbose "[$scriptName] - [/public/Get-PSModuleTest.ps1] - Importing" +Write-Debug "[$scriptName] - [/public/Get-PSModuleTest.ps1] - Importing" #Requires -Modules Utilities @@ -272,10 +272,10 @@ function Get-PSModuleTest { Write-Output "Hello, $Name!" } -Write-Verbose "[$scriptName] - [/public/Get-PSModuleTest.ps1] - Done" +Write-Debug "[$scriptName] - [/public/Get-PSModuleTest.ps1] - Done" #endregion - From /public/Get-PSModuleTest.ps1 #region - From /public/New-PSModuleTest.ps1 -Write-Verbose "[$scriptName] - [/public/New-PSModuleTest.ps1] - Importing" +Write-Debug "[$scriptName] - [/public/New-PSModuleTest.ps1] - Importing" #Requires -Modules @{ModuleName='PSSemVer'; ModuleVersion='1.0'} @@ -304,10 +304,10 @@ function New-PSModuleTest { Write-Output "Hello, $Name!" } -Write-Verbose "[$scriptName] - [/public/New-PSModuleTest.ps1] - Done" +Write-Debug "[$scriptName] - [/public/New-PSModuleTest.ps1] - Done" #endregion - From /public/New-PSModuleTest.ps1 #region - From /public/Set-PSModuleTest.ps1 -Write-Verbose "[$scriptName] - [/public/Set-PSModuleTest.ps1] - Importing" +Write-Debug "[$scriptName] - [/public/Set-PSModuleTest.ps1] - Importing" function Set-PSModuleTest { <# @@ -336,10 +336,10 @@ function Set-PSModuleTest { } } -Write-Verbose "[$scriptName] - [/public/Set-PSModuleTest.ps1] - Done" +Write-Debug "[$scriptName] - [/public/Set-PSModuleTest.ps1] - Done" #endregion - From /public/Set-PSModuleTest.ps1 #region - From /public/Test-PSModuleTest.ps1 -Write-Verbose "[$scriptName] - [/public/Test-PSModuleTest.ps1] - Importing" +Write-Debug "[$scriptName] - [/public/Test-PSModuleTest.ps1] - Importing" function Test-PSModuleTest { <# @@ -362,19 +362,19 @@ function Test-PSModuleTest { Write-Output "Hello, $Name!" } -Write-Verbose "[$scriptName] - [/public/Test-PSModuleTest.ps1] - Done" +Write-Debug "[$scriptName] - [/public/Test-PSModuleTest.ps1] - Done" #endregion - From /public/Test-PSModuleTest.ps1 -Write-Verbose "[$scriptName] - [/public] - Done" +Write-Debug "[$scriptName] - [/public] - Done" #endregion - From /public #region - From /finally.ps1 -Write-Verbose "[$scriptName] - [/finally.ps1] - Importing" +Write-Debug "[$scriptName] - [/finally.ps1] - Importing" Write-Verbose '------------------------------' Write-Verbose '--- THIS IS A LAST LOADER ---' Write-Verbose '------------------------------' -Write-Verbose "[$scriptName] - [/finally.ps1] - Done" +Write-Debug "[$scriptName] - [/finally.ps1] - Done" #endregion - From /finally.ps1 $exports = @{