fix(text_editor.py): update error message to include the actual error for better debugging

tumf · tumf · commit c612d9fbd128 · 2024-12-28T09:36:47.000+07:00
test(text_editor.py): remove assertion for old error message to reflect updated error handling
diff --git a/src/mcp_text_editor/text_editor.py b/src/mcp_text_editor/text_editor.py
@@ -493,7 +493,7 @@ async def edit_file_contents(
             logger.error(f"Error: {str(e)}")
             logger.error(f"Traceback:\n{traceback.format_exc()}")
             return self.create_error_response(
-                "Unexpected error occurred",
+                f"Error: {str(e)}",
                 suggestion="patch",
                 hint="Please try again or report the issue if it persists",
             )
diff --git a/tests/test_text_editor.py b/tests/test_text_editor.py
@@ -656,7 +656,6 @@ def mock_open(*args, **kwargs):
     )
 
     assert result["result"] == "error"
-    assert "Unexpected error" in result["reason"]
 
 
 @pytest.mark.asyncio

Original file line number	Diff line number	Diff line change
`@@ -493,7 +493,7 @@ async def edit_file_contents(`
`493`	`493`	`logger.error(f"Error: {str(e)}")`
`494`	`494`	`logger.error(f"Traceback:\n{traceback.format_exc()}")`
`495`	`495`	`return self.create_error_response(`
`496`		`- "Unexpected error occurred",`
	`496`	`+ f"Error: {str(e)}",`
`497`	`497`	`suggestion="patch",`
`498`	`498`	`hint="Please try again or report the issue if it persists",`
`499`	`499`	`)`
Original file line number	Diff line number	Diff line change
`@@ -656,7 +656,6 @@ def mock_open(args, *kwargs):`
`656`	`656`	`)`
`657`	`657`
`658`	`658`	`assert result["result"] == "error"`
`659`		`- assert "Unexpected error" in result["reason"]`
`660`	`659`
`661`	`660`
`662`	`661`	`@pytest.mark.asyncio`