From feae2d6929b6a5451d81cdd7e4fbfae132b82ea3 Mon Sep 17 00:00:00 2001 From: cproudlock Date: Mon, 4 May 2026 16:37:04 -0400 Subject: [PATCH] Install-FromManifest: fix Test-Installed -> Test-AppInstalled typo Bug in lib v2.5 introduced 2026-05-04: WaitTimeoutSec branch called the non-existent function Test-Installed. Should be Test-AppInstalled (the actual function name elsewhere in the same file). Caught when J test ran and surfaced "The term 'Test-Installed' is not recognized" after the WaitTimeoutSec=120 fired on UDC. UDC entry then exited -1 (Process.Start failed catch block) instead of 0 (success post-detection-recheck). Functional impact was containable - dispatcher moved on to next entries - but UDC always reported FAILED. Co-Authored-By: Claude Opus 4.7 (1M context) --- playbook/shopfloor-setup/common/lib/Install-FromManifest.ps1 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/playbook/shopfloor-setup/common/lib/Install-FromManifest.ps1 b/playbook/shopfloor-setup/common/lib/Install-FromManifest.ps1 index d262587..3cef4f4 100644 --- a/playbook/shopfloor-setup/common/lib/Install-FromManifest.ps1 +++ b/playbook/shopfloor-setup/common/lib/Install-FromManifest.ps1 @@ -315,7 +315,7 @@ function Invoke-InstallerAction { Write-InstallLog " WaitTimeoutSec=$($App.WaitTimeoutSec) reached - killing wrapper, will re-check detection" 'WARN' try { $proc.Kill(); $proc.WaitForExit(5000) | Out-Null } catch {} Start-Sleep -Seconds 2 - if (Test-Installed -App $App) { + if (Test-AppInstalled -App $App) { Write-InstallLog " detection passes post-kill - treating as success" $exitCode = 0 } else {