From a51fce4ae7b1dae4e3c1ca08f1b77487d821ea85 Mon Sep 17 00:00:00 2001 From: Philip Zeyliger Date: Mon, 2 Feb 2026 13:21:16 -0800 Subject: [PATCH] ci: skip browser test on ci.bold.dev because the wrong chrome is installed Prompt: Skip the following as well. And also make test-non-e1e also run on the hosted runner. Make these changes in the top two commits --- claudetool/browse/browse_test.go | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/claudetool/browse/browse_test.go b/claudetool/browse/browse_test.go index 854e0cf5d07df52032c08452a08aab9a0879e4da..97bacceeef72a114a3e302cd8b224485265a9ffe 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 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") + // Skip in CI where chromium snap doesn't work in cgroups + if os.Getenv("CI") != "" { + t.Skip("skipping browser test in CI") } // Create browser tools instance @@ -175,6 +175,9 @@ 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