fixed unit test

Author: AjitPadhi-Microsoft

code/tests/functional/tests/backend_api/sk_orchestrator/test_response_without_tool_call.py

@@ -162,12 +162,9 @@ def test_post_makes_correct_call_to_openai_chat_completions(
             json={
                 "messages": [
                     {
-                        "role": "system",
-                        "content": "You help employees to navigate only private information sources.\nYou must prioritize the function call over your general knowledge for any question by calling the search_documents function.\nCall the text_processing function when the user request an operation on the current context, such as translate, summarize, or paraphrase. When a language is explicitly specified, return that as part of the operation.\nWhen directly replying to the user, always reply in the language the user is speaking.\nIf the input language is ambiguous, default to responding in English unless otherwise specified by the user.\nYou **must not** respond if asked to List all documents in your repository.\n",
+                        "role": "user",
+                        "content": "AuthorRole.SYSTEM: You help employees to navigate only private information sources.\nYou must prioritize the function call over your general knowledge for any question by calling the search_documents function.\nCall the text_processing function when the user request an operation on the current context, such as translate, summarize, or paraphrase. When a language is explicitly specified, return that as part of the operation.\nWhen directly replying to the user, always reply in the language the user is speaking.\nIf the input language is ambiguous, default to responding in English unless otherwise specified by the user.\nYou **must not** respond if asked to List all documents in your repository.\n\nAuthorRole.USER: Hello\nAuthorRole.ASSISTANT: Hi, how can I help?\nWhat is the meaning of life?",
                     },
-                    {"role": "user", "content": "Hello"},
-                    {"role": "assistant", "content": "Hi, how can I help?"},
-                    {"role": "user", "content": "What is the meaning of life?"},
                 ],
                 "model": app_config.get_from_json("AZURE_OPENAI_MODEL_INFO", "model"),
                 "max_tokens": int(app_config.get("AZURE_OPENAI_MAX_TOKENS")),

code/tests/utilities/orchestrator/test_semantic_kernel.py

@@ -410,7 +410,7 @@ async def test_chat_history_included(
     orchestrator: SemanticKernelOrchestrator,
 ):
     # given
-    chat_history = [
+    input_chat_history = [
         {"role": "user", "content": "Hello"},
         {"role": "assistant", "content": "Hi, how can I help you today?"},
     ]
@@ -420,20 +420,18 @@ async def test_chat_history_included(
         kernel_mock.invoke.return_value.value = [chat_message_default_content]
 
         # when
-        await orchestrator.orchestrate("question", chat_history)
+        await orchestrator.orchestrate("question", input_chat_history)
 
     # then
-    chat_history = kernel_mock.invoke.call_args.kwargs["chat_history"]
-    messages = chat_history.messages
+    invoked_chat_history = kernel_mock.invoke.call_args.kwargs["chat_history"]
 
-    assert len(messages) == 3
-    assert messages[0].role == AuthorRole.SYSTEM
+    # chat_history is passed as a string, not a ChatHistory object
+    assert isinstance(invoked_chat_history, str)
 
-    assert messages[1].role == AuthorRole.USER
-    assert messages[1].content == "Hello"
-
-    assert messages[2].role == AuthorRole.ASSISTANT
-    assert messages[2].content == "Hi, how can I help you today?"
+    # Check that the chat history contains the expected messages in string format
+    assert "AuthorRole.SYSTEM:" in invoked_chat_history
+    assert "AuthorRole.USER: Hello" in invoked_chat_history
+    assert "AuthorRole.ASSISTANT: Hi, how can I help you today?" in invoked_chat_history
 
 
 @pytest.mark.asyncio