From d5a24277f32baf09cda0dac15a91c9c48ea39fcf Mon Sep 17 00:00:00 2001 From: Philip Zeyliger Date: Mon, 2 Feb 2026 16:24:54 -0800 Subject: [PATCH] Revert "ci: skip browser test on ci.bold.dev because the wrong chrome is installed" This reverts commit a7e489fa2234c2e99972f26b066ebe4e93ec1d73. --- claudetool/browse/browse_test.go | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/claudetool/browse/browse_test.go b/claudetool/browse/browse_test.go index 97bacceeef72a114a3e302cd8b224485265a9ffe..854e0cf5d07df52032c08452a08aab9a0879e4da 100644 --- a/claudetool/browse/browse_test.go +++ b/claudetool/browse/browse_test.go @@ -151,9 +151,9 @@ func TestNavigateTool(t *testing.T) { t.Skip("skipping navigate tool test in short mode") } - // Skip in CI where chromium snap doesn't work in cgroups - if os.Getenv("CI") != "" { - t.Skip("skipping browser test in CI") + // Skip on ci.bold.dev where chromium snap doesn't work in cgroups + if hostname, _ := os.Hostname(); strings.Contains(hostname, "bold.dev") { + t.Skip("skipping browser test on ci.bold.dev") } // Create browser tools instance @@ -175,9 +175,6 @@ func TestNavigateTool(t *testing.T) { // Call the tool toolOut := navTool.Run(ctx, []byte(inputJSON)) if toolOut.Error != nil { - if strings.Contains(toolOut.Error.Error(), "failed to start browser") { - t.Skip("Browser automation not available in this environment") - } t.Fatalf("Error running navigate tool: %v", toolOut.Error) } result := toolOut.LLMContent