fix: address review comments - move Write-Host, clarify dual-computation, fix fixture comment

Author: MariusStorhaug

src/tests/Module/PSModule/PSModule.Tests.ps1

@@ -40,12 +40,12 @@ if ($hasClassExporter) {
         $expectedEnumNames = @([regex]::Matches($Matches[1], '\[([^\]]+)\]') | ForEach-Object { $_.Groups[1].Value })
     }
 }
-Write-Host "Has class exporter: $hasClassExporter"
-Write-Host "Expected classes: $($expectedClassNames -join ', ')"
-Write-Host "Expected enums: $($expectedEnumNames -join ', ')"
 
-# Run-phase setup — recompute from $Path (available in both Discovery and Run phases).
-# Script-level variables above only exist during Discovery; BeforeAll runs only during Run.
+
+# Run-phase setup — recompute from $Path so that It/Context blocks can use these variables.
+# Pester v5 Discovery and Run are separate executions. The script-scope variables above drive
+# -Skip and -ForEach during Discovery. This BeforeAll recomputes the same values for the Run
+# phase, where It blocks actually execute. The duplication is intentional and required.
 BeforeAll {
     $moduleName = Split-Path -Path (Split-Path -Path $Path -Parent) -Leaf
     $moduleManifestPath = Join-Path -Path $Path -ChildPath "$moduleName.psd1"
@@ -64,9 +64,9 @@ BeforeAll {
             $expectedEnumNames = @([regex]::Matches($Matches[1], '\[([^\]]+)\]') | ForEach-Object { $_.Groups[1].Value })
         }
     }
-    Write-Host "Run phase - Module: $moduleName"
-    Write-Host "Run phase - Manifest: $moduleManifestPath"
-    Write-Host "Run phase - Has class exporter: $hasClassExporter"
+    Write-Host "Has class exporter: $hasClassExporter"
+    Write-Host "Expected classes: $($expectedClassNames -join ', ')"
+    Write-Host "Expected enums: $($expectedEnumNames -join ', ')"
 }
 
 Describe 'PSModule - Module tests' {

tests/outputTestRepo/outputs/module/PSModuleTest/PSModuleTest.psm1

@@ -383,7 +383,7 @@ $TypeAcceleratorsClass = [psobject].Assembly.GetType(
     'System.Management.Automation.TypeAccelerators'
 )
 # Ensure none of the types would clobber an existing type accelerator.
-# If a type accelerator with the same name exists, throw an exception.
+# If a type accelerator with the same name already exists, skip adding it.
 $ExistingTypeAccelerators = $TypeAcceleratorsClass::Get
 # Define the types to export with type accelerators.
 $ExportableEnums = @(