Revert "ci: skip browser test on ci.bold.dev because the wrong chrome is installed"

Philip Zeyliger created 2 months ago

This reverts commit a7e489fa2234c2e99972f26b066ebe4e93ec1d73.

Change summary

claudetool/browse/browse_test.go | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)

Detailed changes

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