fix: use filepath to construct mcp test write path

code-asher · code-asher · commit a3171ad9c0a1 · 2025-09-12T15:47:10.000-08:00
Hopefully fixes the test flake:

  file path must be absolute: "/test/some/path"

In Windows, starting with `/` is not considered absolute.  No idea how
this only flakes rather than always failing though.
diff --git a/codersdk/toolsdk/toolsdk_test.go b/codersdk/toolsdk/toolsdk_test.go
@@ -569,14 +569,17 @@ func TestTools(t *testing.T) {
 		tb, err := toolsdk.NewDeps(client)
 		require.NoError(t, err)
 
+		tmpdir := os.TempDir()
+		filePath := filepath.Join(tmpdir, "write")
+
 		_, err = testTool(t, toolsdk.WorkspaceWriteFile, tb, toolsdk.WorkspaceWriteFileArgs{
 			Workspace: workspace.Name,
-			Path:      "/test/some/path",
+			Path:      filePath,
 			Content:   []byte("content"),
 		})
 		require.NoError(t, err)
 
-		b, err := afero.ReadFile(fs, "/test/some/path")
+		b, err := afero.ReadFile(fs, filePath)
 		require.NoError(t, err)
 		require.Equal(t, []byte("content"), b)
 	})
