Merge pull request #686 from dgageot/remove-debug-traces-tests

Remove debug traces in tests

Author: rumpl

.golangci.yml

@@ -37,6 +37,8 @@ linters:
           msg: "do not use os.MkdirTemp() in tests, use t.TempDir()"
         - pattern: os\.Setenv()
           msg: "do not use os.Setenv() in tests, use t.Setenv()"
+        - pattern: fmt\.Print.*()
+          msg: "do not use fmt.Print() or fmt.Println() in tests"
     depguard:
       rules:
         internal:

pkg/tools/builtin/script_shell_test.go

@@ -2,7 +2,6 @@ package builtin
 
 import (
 	"encoding/json"
-	"fmt"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
@@ -33,7 +32,6 @@ func TestNewScriptShellTool_ToolNoArg(t *testing.T) {
 	assert.Len(t, allTools, 1)
 
 	schema, err := json.Marshal(allTools[0].Parameters)
-	fmt.Println(string(schema))
 	require.NoError(t, err)
 	assert.JSONEq(t, `{
 	"type": "object",

pkg/tools/codemode/codemode_test.go

@@ -3,7 +3,6 @@ package codemode
 import (
 	"context"
 	"encoding/json"
-	"fmt"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
@@ -41,7 +40,6 @@ func TestCodeModeTool_Tools(t *testing.T) {
 }`, string(inputSchema))
 
 	outputSchema, err := json.Marshal(fetchTool.OutputSchema)
-	fmt.Println(string(outputSchema))
 	require.NoError(t, err)
 	assert.JSONEq(t, `{
 	"type": "object",