Two test suites fail locally in devcontainers but pass in CI:
TestIsContainerEnvironment_EnvVars
Ambient env vars (e.g. REMOTE_CONTAINERS) in devcontainers cause false positives due to non-deterministic Go map iteration. ContainerRuntime() may find the wrong env var first.
Test_UpdateAction_Run_SwitchChannel_CheckForUpdateError / Test_UpdateAction_Run_NoChannelNoConfigFlags
Tests expect failure via noopCommandRunner, but CheckForUpdate uses an HTTP client. In CI, IsRunningOnCI() short-circuits before reaching CheckForUpdate, masking the bug. Locally, real HTTP calls succeed and Run() returns nil.
Fix: #7906
Two test suites fail locally in devcontainers but pass in CI:
TestIsContainerEnvironment_EnvVarsAmbient env vars (e.g.
REMOTE_CONTAINERS) in devcontainers cause false positives due to non-deterministic Go map iteration.ContainerRuntime()may find the wrong env var first.Test_UpdateAction_Run_SwitchChannel_CheckForUpdateError/Test_UpdateAction_Run_NoChannelNoConfigFlagsTests expect failure via
noopCommandRunner, butCheckForUpdateuses an HTTP client. In CI,IsRunningOnCI()short-circuits before reachingCheckForUpdate, masking the bug. Locally, real HTTP calls succeed andRun()returns nil.Fix: #7906