Update OffsetInspect.Tests.ps1

Author: warpedatom

tests/OffsetInspect.Tests.ps1

@@ -1,65 +1,76 @@
-# Pester tests for OffsetInspect module / manifest
+# Pester tests for OffsetInspect
 
-# --------------- Path discovery ---------------
+# Helper to find the module manifest reliably
+function Get-OffsetInspectManifestPath {
+    param(
+        [string]$TestsRoot
+    )
 
-# tests folder
-$repoRoot = Split-Path -Parent $PSScriptRoot
+    $repoRoot = Split-Path -Parent $TestsRoot
 
-# Find the first OffsetInspect.psd1 anywhere under the repo
-$manifestItem = Get-ChildItem -Path $repoRoot -Filter 'OffsetInspect.psd1' -Recurse -File |
-    Select-Object -First 1
+    $manifestItem = Get-ChildItem -Path $repoRoot -Filter 'OffsetInspect.psd1' -Recurse -File |
+        Select-Object -First 1
 
-if (-not $manifestItem) {
-    throw "Could not find 'OffsetInspect.psd1' anywhere under '$repoRoot'. " +
-          "Make sure your module manifest exists and is committed."
-}
-
-$script:ManifestPath = $manifestItem.FullName
+    if (-not $manifestItem) {
+        throw "Could not find 'OffsetInspect.psd1' anywhere under '$repoRoot'. " +
+              "Make sure your module manifest exists and is committed."
+    }
 
-Write-Host "Using manifest at: $script:ManifestPath"
+    return $manifestItem.FullName
+}
 
-# --------------- Manifest tests ---------------
+# ---------------- Manifest tests ----------------
 
 Describe 'OffsetInspect module manifest' {
 
+    BeforeAll {
+        # Resolve once per Describe, inside Pester’s scope
+        $ManifestPath = Get-OffsetInspectManifestPath -TestsRoot $PSScriptRoot
+        Write-Host "Using manifest (manifest describe): $ManifestPath"
+    }
+
     It 'exists' {
-        Test-Path $script:ManifestPath | Should -BeTrue
+        Test-Path $ManifestPath | Should -BeTrue
     }
 
     It 'has a valid manifest' {
-        { Test-ModuleManifest -Path $script:ManifestPath } |
+        { Test-ModuleManifest -Path $ManifestPath } |
             Should -Not -Throw
     }
 
     It 'has expected module version' {
-        $m = Test-ModuleManifest -Path $script:ManifestPath
-        # Keep this pattern in sync with your release scheme
+        $m = Test-ModuleManifest -Path $ManifestPath
+        # Adjust regex if you bump major/minor later
         $m.Version.ToString() | Should -Match '^1\.0\.\d+$'
     }
 }
 
-# --------------- Export tests ---------------
+# ---------------- Export tests ----------------
 
 Describe 'OffsetInspect module exports' {
 
     BeforeAll {
-        $script:ModuleInfo = Import-Module $script:ManifestPath -Force -PassThru
+        $ManifestPath = Get-OffsetInspectManifestPath -TestsRoot $PSScriptRoot
+        Write-Host "Using manifest (exports describe): $ManifestPath"
+
+        # Import the module; throws if anything is wrong with manifest/module
+        $ModuleInfo = Import-Module $ManifestPath -Force -PassThru
     }
 
     It 'imports without error' {
-        { Import-Module $script:ManifestPath -Force } |
+        { Import-Module $ManifestPath -Force } |
             Should -Not -Throw
     }
 
     It 'exports Invoke-OffsetInspect' {
-        $cmd = Get-Command Invoke-OffsetInspect -Module $script:ModuleInfo.Name -ErrorAction Stop
+        $cmd = Get-Command Invoke-OffsetInspect -Module $ModuleInfo.Name -ErrorAction Stop
         $cmd | Should -Not -BeNullOrEmpty
     }
 
-    It 'exposes FilePaths and OffsetInputs as string[] parameters' {
-        $cmd = Get-Command Invoke-OffsetInspect -Module $script:ModuleInfo.Name -ErrorAction Stop
+    It 'FilePaths and OffsetInputs are string[] parameters' {
+        $cmd = Get-Command Invoke-OffsetInspect -Module $ModuleInfo.Name -ErrorAction Stop
 
-        $cmd.Parameters['FilePaths'].ParameterType     | Should -Be ([string[]])
-        $cmd.Parameters['OffsetInputs'].ParameterType  | Should -Be ([string[]])
+        $cmd.Parameters['FilePaths'].ParameterType    | Should -Be ([string[]])
+        $cmd.Parameters['OffsetInputs'].ParameterType | Should -Be ([string[]])
     }
 }