address test comments

Author: dbschmigelski

tests/strands/tools/test_decorator.py

@@ -1983,40 +1983,38 @@ def my_tool(name: str, tag: str | None = None) -> str:
 @pytest.mark.asyncio
 async def test_tool_result_event_carries_exception_runtime_error(alist):
     """Test that ToolResultEvent carries exception when tool raises RuntimeError."""
+    exception = RuntimeError("test runtime error")
 
     @strands.tool
     def error_tool():
         """Tool that raises a RuntimeError."""
-        raise RuntimeError("test runtime error")
+        raise exception
 
     tool_use = {"toolUseId": "test-id", "input": {}}
     events = await alist(error_tool.stream(tool_use, {}))
 
     result_event = events[-1]
     assert isinstance(result_event, ToolResultEvent)
-    assert hasattr(result_event, "exception")
-    assert isinstance(result_event.exception, RuntimeError)
-    assert str(result_event.exception) == "test runtime error"
+    assert result_event.exception is exception
     assert result_event.tool_result["status"] == "error"
 
 
 @pytest.mark.asyncio
 async def test_tool_result_event_carries_exception_value_error(alist):
     """Test that ToolResultEvent carries exception when tool raises ValueError."""
+    exception = ValueError("validation failed")
 
     @strands.tool
     def validation_error_tool():
         """Tool that raises a ValueError."""
-        raise ValueError("validation failed")
+        raise exception
 
     tool_use = {"toolUseId": "test-id", "input": {}}
     events = await alist(validation_error_tool.stream(tool_use, {}))
 
     result_event = events[-1]
     assert isinstance(result_event, ToolResultEvent)
-    assert hasattr(result_event, "exception")
-    assert isinstance(result_event.exception, ValueError)
-    assert str(result_event.exception) == "validation failed"
+    assert result_event.exception is exception
     assert result_event.tool_result["status"] == "error"
 
 
@@ -2041,17 +2039,17 @@ def success_tool():
 @pytest.mark.asyncio
 async def test_tool_result_event_carries_exception_assertion_error(alist):
     """Test that ToolResultEvent carries AssertionError for unexpected failures."""
+    exception = AssertionError("unexpected assertion failure")
 
     @strands.tool
     def assertion_error_tool():
         """Tool that raises an AssertionError."""
-        raise AssertionError("unexpected assertion failure")
+        raise exception
 
     tool_use = {"toolUseId": "test-id", "input": {}}
     events = await alist(assertion_error_tool.stream(tool_use, {}))
 
     result_event = events[-1]
     assert isinstance(result_event, ToolResultEvent)
-    assert isinstance(result_event.exception, AssertionError)
-    assert "unexpected assertion failure" in str(result_event.exception)
+    assert result_event.exception is exception
     assert result_event.tool_result["status"] == "error"