fix: Fail integ tests if tool execution fails

Author: clareliguori

e2e_tests/python/server_clients/server.py

@@ -133,8 +133,4 @@ async def execute_tool(
                     await asyncio.sleep(delay)
                 else:
                     logging.error("Max retries reached. Failing.")
-                    return {
-                        "toolUseId": tool_use_id,
-                        "content": [{"text": f"Error executing tool: {str(e)}"}],
-                        "status": "error",
-                    }
+                    raise Exception(f"Error executing tool: {str(e)}")

e2e_tests/typescript/src/server_clients/server.ts

@@ -127,11 +127,7 @@ export abstract class Server {
           await new Promise((resolve) => setTimeout(resolve, delay * 1000));
         } else {
           logger.error("Max retries reached. Failing.");
-          return {
-            toolUseId: toolUseId,
-            content: [{ text: `Error executing tool: ${String(e)}` }],
-            status: "error",
-          };
+          throw new Error(`Error executing tool: ${String(e)}`);
         }
       }
     }