From 6deaae40640f6de475e2e441d6b5a75154216a89 Mon Sep 17 00:00:00 2001 From: Claude Date: Mon, 27 Jul 2026 15:48:17 +0300 Subject: [PATCH] test(app-store): raise spawn lifecycle test timeout 3s -> 30s MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit TestSpawn_FastExitTriggersExitCode and TestSpawn_NonZeroExitPropagates wrap sup.spawn in a 3s context. Under parallel load the fake binary does not finish inside that budget, the context cancels, and spawn returns -1 instead of the real exit code — both tests fail at exactly 3.00s. They pass in ~0.3-0.5s when run in isolation. The assertion is on the exit code, not on latency, so the short deadline buys nothing and only encodes machine load into the result. Raise it to 30s. Test-only change. Co-Authored-By: Claude Opus 5 --- plugin/appstore/zz2_supervisor_lifecycle_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/plugin/appstore/zz2_supervisor_lifecycle_test.go b/plugin/appstore/zz2_supervisor_lifecycle_test.go index 929fc29..9163d46 100644 --- a/plugin/appstore/zz2_supervisor_lifecycle_test.go +++ b/plugin/appstore/zz2_supervisor_lifecycle_test.go @@ -111,7 +111,7 @@ func TestSpawn_FastExitTriggersExitCode(t *testing.T) { } a.Manifest.Binary.SHA256 = sum // pass the spawn-time TOCTOU re-verify sup := newSupervisor(Config{InstallRoot: dir}, Deps{}, newQuietLogger(t)) - ctx, cancel := context.WithTimeout(context.Background(), 3*time.Second) + ctx, cancel := context.WithTimeout(context.Background(), 30*time.Second) defer cancel() code := sup.spawn(ctx, a) if code != 0 { @@ -141,7 +141,7 @@ func TestSpawn_NonZeroExitPropagates(t *testing.T) { } a.Manifest.Binary.SHA256 = sum // pass the spawn-time TOCTOU re-verify sup := newSupervisor(Config{InstallRoot: dir}, Deps{}, newQuietLogger(t)) - ctx, cancel := context.WithTimeout(context.Background(), 3*time.Second) + ctx, cancel := context.WithTimeout(context.Background(), 30*time.Second) defer cancel() code := sup.spawn(ctx, a) if code != 42 {