fix(test): defer yq detection out of conversion BeforeAll

Linux CI can expose multiple yq Application matches that break
Initialize-ConversionIntegration. Probe yq lazily with caching and
harden Get-CachedExternalCommand resolution instead.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: F

profile.d/bootstrap/CommandCache.ps1

@@ -539,9 +539,10 @@ function global:Get-CachedExternalCommand {
 
         # Prefer explicit test mocks over host binaries with the same name.
         if ($global:TestRegisteredMockCommands -and $global:TestRegisteredMockCommands.Contains($candidate)) {
-            $mockFunction = Get-Command -Name $candidate -CommandType Function -ErrorAction SilentlyContinue
-            if ($mockFunction -and $mockFunction.Name.Equals($candidate, [StringComparison]::OrdinalIgnoreCase)) {
-                return $mockFunction
+            foreach ($mockFunction in @(Get-Command -Name $candidate -CommandType Function -ErrorAction SilentlyContinue)) {
+                if ($mockFunction.Name.Equals($candidate, [StringComparison]::OrdinalIgnoreCase)) {
+                    return $mockFunction
+                }
             }
         }
 
@@ -552,23 +553,40 @@ function global:Get-CachedExternalCommand {
                 continue
             }
 
-            if ($isYqLookup -and -not (Test-IsMikefarahYqExecutable -Executable $executablePath)) {
-                continue
+            if ($isYqLookup) {
+                try {
+                    if (-not (Test-IsMikefarahYqExecutable -Executable $executablePath)) {
+                        continue
+                    }
+                }
+                catch {
+                    continue
+                }
             }
 
             return $application
         }
 
         # Profile aliases with the same name as a binary would otherwise recurse into wrapper functions.
-        $functionCmd = Get-Command -Name $candidate -CommandType Function -ErrorAction SilentlyContinue
-        if ($functionCmd -and $functionCmd.Name.Equals($candidate, [StringComparison]::OrdinalIgnoreCase)) {
-            return $functionCmd
+        foreach ($functionCmd in @(Get-Command -Name $candidate -CommandType Function -ErrorAction SilentlyContinue)) {
+            if ($functionCmd.Name.Equals($candidate, [StringComparison]::OrdinalIgnoreCase)) {
+                return $functionCmd
+            }
         }
 
         foreach ($externalScript in @(Get-Command -Name $candidate -CommandType ExternalScript -ErrorAction SilentlyContinue)) {
             if ($isYqLookup) {
                 $exe = Resolve-CommandExecutablePath -CommandInfo $externalScript
-                if ([string]::IsNullOrWhiteSpace($exe) -or -not (Test-IsMikefarahYqExecutable -Executable $exe)) {
+                if ([string]::IsNullOrWhiteSpace($exe)) {
+                    continue
+                }
+
+                try {
+                    if (-not (Test-IsMikefarahYqExecutable -Executable $exe)) {
+                        continue
+                    }
+                }
+                catch {
                     continue
                 }
             }
@@ -599,6 +617,9 @@ function global:Clear-TestCachedCommandCache {
     }
 
     $global:TestCachedCommandCache.Clear()
+    if (Get-Command Clear-TestMikefarahYqAvailabilityCache -ErrorAction SilentlyContinue) {
+        Clear-TestMikefarahYqAvailabilityCache
+    }
     return $true
 }
 

tests/TestSupport/TestModuleLoading.ps1

@@ -1692,14 +1692,8 @@ function Initialize-ConversionIntegration {
         }
     }
 
-    if (Get-Command Test-MikefarahYqAvailable -ErrorAction SilentlyContinue) {
-        try {
-            $script:MikefarahYqAvailable = Test-MikefarahYqAvailable
-        }
-        catch {
-            $script:MikefarahYqAvailable = $false
-        }
-    }
+    # yq availability is resolved lazily via Test-MikefarahYqAvailable when tests call
+    # Skip-IfMikefarahYqUnavailable; probing here broke unrelated conversion BeforeAll hooks on Linux CI.
 }
 
 

tests/TestSupport/ToolDetection.ps1

@@ -863,67 +863,102 @@ function Test-MikefarahYqAvailable {
     #>
     [CmdletBinding()]
     [OutputType([bool])]
-    param()
-
-    if (Get-Command Get-CachedExternalCommand -ErrorAction SilentlyContinue) {
-        $yqCmd = Get-CachedExternalCommand -Name 'yq'
-        if (-not $yqCmd) {
-            return $false
-        }
-
-        if (Get-Command Test-IsMikefarahYqExecutable -ErrorAction SilentlyContinue) {
-            $executable = if (Get-Command Resolve-CommandExecutablePath -ErrorAction SilentlyContinue) {
-                Resolve-CommandExecutablePath -CommandInfo $yqCmd
-            }
-            elseif (-not [string]::IsNullOrWhiteSpace($yqCmd.Name)) {
-                [string]$yqCmd.Name
-            }
-            else {
-                $null
-            }
-            return Test-IsMikefarahYqExecutable -Executable $executable
-        }
+    param(
+        [switch]$Refresh
+    )
 
-        return $true
+    if (-not $Refresh -and (Get-Variable -Name MikefarahYqAvailabilityCache -Scope Script -ErrorAction SilentlyContinue)) {
+        return [bool]$script:MikefarahYqAvailabilityCache
     }
 
-    foreach ($candidate in @('go-yq', 'yq')) {
-        foreach ($cmd in @(Get-Command $candidate -CommandType Application -ErrorAction SilentlyContinue)) {
-            $exe = if (Get-Command Resolve-CommandExecutablePath -ErrorAction SilentlyContinue) {
-                Resolve-CommandExecutablePath -CommandInfo $cmd
-            }
-            elseif (-not [string]::IsNullOrWhiteSpace($cmd.Name)) {
-                [string]$cmd.Name
-            }
-            else {
-                $null
-            }
-            if ([string]::IsNullOrWhiteSpace($exe)) {
-                continue
-            }
-
-            if (Get-Command Test-IsMikefarahYqExecutable -ErrorAction SilentlyContinue) {
-                if (Test-IsMikefarahYqExecutable -Executable $exe) {
-                    return $true
+    try {
+        $available = $false
+
+        if (Get-Command Get-CachedExternalCommand -ErrorAction SilentlyContinue) {
+            $yqCmd = Get-CachedExternalCommand -Name 'yq'
+            if ($yqCmd) {
+                if (Get-Command Test-IsMikefarahYqExecutable -ErrorAction SilentlyContinue) {
+                    $executable = if (Get-Command Resolve-CommandExecutablePath -ErrorAction SilentlyContinue) {
+                        Resolve-CommandExecutablePath -CommandInfo $yqCmd
+                    }
+                    elseif (-not [string]::IsNullOrWhiteSpace($yqCmd.Name)) {
+                        [string]$yqCmd.Name
+                    }
+                    else {
+                        $null
+                    }
+                    $available = Test-IsMikefarahYqExecutable -Executable $executable
                 }
-            }
-            else {
-                $versionOutput = (& $exe --version 2>&1 | Out-String).Trim()
-                if ($versionOutput -match 'mikefarah|github\.com/mikefarah') {
-                    return $true
+                else {
+                    $available = $true
                 }
+            }
+        }
+        else {
+            foreach ($candidate in @('go-yq', 'yq')) {
+                foreach ($cmd in @(Get-Command $candidate -CommandType Application -ErrorAction SilentlyContinue)) {
+                    $exe = if (Get-Command Resolve-CommandExecutablePath -ErrorAction SilentlyContinue) {
+                        Resolve-CommandExecutablePath -CommandInfo $cmd
+                    }
+                    elseif (-not [string]::IsNullOrWhiteSpace($cmd.Name)) {
+                        [string]$cmd.Name
+                    }
+                    else {
+                        $null
+                    }
+                    if ([string]::IsNullOrWhiteSpace($exe)) {
+                        continue
+                    }
 
-                if ($versionOutput -notmatch '^yq\s+\d') {
-                    $evalHelp = (& $exe eval --help 2>&1 | Out-String)
-                    if ($evalHelp -match 'evaluates' -and $evalHelp -notmatch 'jq_filter') {
-                        return $true
+                    if (Get-Command Test-IsMikefarahYqExecutable -ErrorAction SilentlyContinue) {
+                        if (Test-IsMikefarahYqExecutable -Executable $exe) {
+                            $available = $true
+                            break
+                        }
+                    }
+                    else {
+                        $versionOutput = (& $exe --version 2>&1 | Out-String).Trim()
+                        if ($versionOutput -match 'mikefarah|github\.com/mikefarah') {
+                            $available = $true
+                            break
+                        }
+
+                        if ($versionOutput -notmatch '^yq\s+\d') {
+                            $evalHelp = (& $exe eval --help 2>&1 | Out-String)
+                            if ($evalHelp -match 'evaluates' -and $evalHelp -notmatch 'jq_filter') {
+                                $available = $true
+                                break
+                            }
+                        }
                     }
                 }
+
+                if ($available) {
+                    break
+                }
             }
         }
+
+        $script:MikefarahYqAvailabilityCache = $available
+        return $available
     }
+    catch {
+        $script:MikefarahYqAvailabilityCache = $false
+        return $false
+    }
+}
 
-    return $false
+function Clear-TestMikefarahYqAvailabilityCache {
+    <#
+    .SYNOPSIS
+        Clears the cached result used by Test-MikefarahYqAvailable.
+    #>
+    [CmdletBinding()]
+    param()
+
+    if (Get-Variable -Name MikefarahYqAvailabilityCache -Scope Script -ErrorAction SilentlyContinue) {
+        $script:MikefarahYqAvailabilityCache = $null
+    }
 }
 
 function Get-TestMikefarahYqSkipMessage {