Refactor send_chat_history_messages calls for improved readability

Author: Johan Broberg

tests/tooling/extensions/openai/test_e2e.py

@@ -76,9 +76,7 @@ async def test_send_chat_history_e2e_success(self, service, mock_turn_context):
     # E2E-02
     @pytest.mark.asyncio
     @pytest.mark.unit
-    async def test_send_chat_history_e2e_server_error(
-        self, service, mock_turn_context
-    ):
+    async def test_send_chat_history_e2e_server_error(self, service, mock_turn_context):
         """Test full end-to-end flow with HTTP 500 error."""
         messages = [
             MockUserMessage(content="Hello"),
@@ -199,9 +197,7 @@ def capture_args(*args, **kwargs):
 
             mock_send.side_effect = capture_args
 
-            await service.send_chat_history_messages(
-                mock_turn_context, sample_openai_messages
-            )
+            await service.send_chat_history_messages(mock_turn_context, sample_openai_messages)
 
             # Verify all messages are ChatHistoryMessage instances
             assert captured_messages is not None
@@ -343,9 +339,7 @@ def capture_post(*args, **kwargs):
             mock_session_instance.post.side_effect = capture_post
             mock_session_class.return_value.__aenter__.return_value = mock_session_instance
 
-            await service.send_chat_history_messages(
-                mock_turn_context, sample_openai_messages
-            )
+            await service.send_chat_history_messages(mock_turn_context, sample_openai_messages)
 
             # Verify headers
             assert captured_headers is not None

tests/tooling/extensions/openai/test_send_chat_history.py

@@ -216,9 +216,7 @@ async def test_send_chat_history_messages_default_orchestrator_name(
         ) as mock_send:
             mock_send.return_value = OperationResult.success()
 
-            await service.send_chat_history_messages(
-                mock_turn_context, sample_openai_messages
-            )
+            await service.send_chat_history_messages(mock_turn_context, sample_openai_messages)
 
             # Verify default orchestrator name
             call_args = mock_send.call_args
@@ -238,9 +236,7 @@ async def test_send_chat_history_messages_delegates_to_config_service(
         ) as mock_send:
             mock_send.return_value = OperationResult.success()
 
-            await service.send_chat_history_messages(
-                mock_turn_context, sample_openai_messages
-            )
+            await service.send_chat_history_messages(mock_turn_context, sample_openai_messages)
 
             # Verify delegation
             mock_send.assert_called_once()
@@ -394,20 +390,14 @@ async def test_send_chat_history_messages_client_error(
     # EH-04
     @pytest.mark.asyncio
     @pytest.mark.unit
-    async def test_send_chat_history_messages_conversion_error(
-        self, service, mock_turn_context
-    ):
+    async def test_send_chat_history_messages_conversion_error(self, service, mock_turn_context):
         """Test send_chat_history_messages handles conversion errors gracefully."""
         sample_messages = [MockUserMessage(content="Hello")]
 
-        with patch.object(
-            service, "_convert_openai_messages_to_chat_history"
-        ) as mock_convert:
+        with patch.object(service, "_convert_openai_messages_to_chat_history") as mock_convert:
             mock_convert.side_effect = Exception("Conversion failed")
 
-            result = await service.send_chat_history_messages(
-                mock_turn_context, sample_messages
-            )
+            result = await service.send_chat_history_messages(mock_turn_context, sample_messages)
 
             assert result.succeeded is False
             assert len(result.errors) == 1