Exclude from code coverage for now as there is untestable code at present

Author: markwallace-microsoft

.github/_typos.toml

@@ -26,7 +26,8 @@ extend-exclude = [
     "SK-dotnet.sln.DotSettings",
     "**/azure_ai_search_hotel_samples/README.md",
     "**/Demos/ProcessFrameworkWithAspire/ProcessFramework.Aspire/ProcessFramework.Aspire.ProcessOrchestrator/Program.cs",
-    "**/Demos/ProcessFrameworkWithAspire/**/*.http"
+    "**/Demos/ProcessFrameworkWithAspire/**/*.http",
+    "openai_responses_what_is_the_semantic_kernel.json"
 ]
 
 [default.extend-words]

dotnet/src/Agents/OpenAI/OpenAIResponsesAgent.cs

@@ -16,6 +16,7 @@ namespace Microsoft.SemanticKernel.Agents.OpenAI;
 /// <summary>
 /// Represents a <see cref="KernelAgent"/> specialization based on Open AI Assistant / GPT.
 /// </summary>
+[ExcludeFromCodeCoverage]
 public sealed class OpenAIResponsesAgent : KernelAgent
 {
     /// <summary>
@@ -51,12 +52,6 @@ public override async IAsyncEnumerable<AgentResponseItem<ChatMessageContent>> In
         // Invoke responses with the updated chat history.
         var chatHistory = new ChatHistory();
         chatHistory.AddRange(messages);
-        /*
-        await foreach (var existingMessage in chatHistoryAgentThread.GetMessagesAsync(cancellationToken).ConfigureAwait(false))
-        {
-            chatHistory.Add(existingMessage);
-        }
-        */
         var invokeResults = this.InternalInvokeAsync(
             this.Name,
             chatHistory,
@@ -80,7 +75,7 @@ public override IAsyncEnumerable<AgentResponseItem<StreamingChatMessageContent>>
 
     /// <inheritdoc/>
     [Experimental("SKEXP0110")]
-    protected override Task<AgentChannel> CreateChannelAsync(CancellationToken cancellationToken)
+    protected override Task<AgentChannel> CreateChannelAsync(CancellationToken cancellationToken = default)
     {
         throw new NotImplementedException();
     }
@@ -94,7 +89,7 @@ protected override IEnumerable<string> GetChannelKeys()
 
     /// <inheritdoc/>
     [Experimental("SKEXP0110")]
-    protected override Task<AgentChannel> RestoreChannelAsync(string channelState, CancellationToken cancellationToken)
+    protected override Task<AgentChannel> RestoreChannelAsync(string channelState, CancellationToken cancellationToken = default)
     {
         throw new NotImplementedException();
     }

dotnet/src/Agents/OpenAI/OpenAIResponsesAgentThread.cs

@@ -2,6 +2,7 @@
 
 using System;
 using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
 using System.Runtime.CompilerServices;
 using System.Threading;
 using System.Threading.Tasks;
@@ -13,6 +14,7 @@ namespace Microsoft.SemanticKernel.Agents.OpenAI;
 /// <summary>
 /// Represents a conversation thread for an OpenAI responses-based agent.
 /// </summary>
+[ExcludeFromCodeCoverage]
 public sealed class OpenAIResponsesAgentThread : AgentThread
 {
     private readonly OpenAIResponseClient _client;

dotnet/src/Agents/UnitTests/Agents.UnitTests.csproj

@@ -47,6 +47,13 @@
   <ItemGroup>
     <Compile Include="$(RepoRoot)/dotnet/src/InternalUtilities/test/AssertExtensions.cs" Link="%(RecursiveDir)Test/%(Filename)%(Extension)" />
     <Compile Include="$(RepoRoot)/dotnet/src/InternalUtilities/test/HttpMessageHandlerStub.cs" Link="%(RecursiveDir)Test/%(Filename)%(Extension)" />
+    <Compile Include="$(RepoRoot)/dotnet/src/InternalUtilities/test/MultipleHttpMessageHandlerStub.cs" Link="%(RecursiveDir)Test/%(Filename)%(Extension)" />
+  </ItemGroup>
+
+  <ItemGroup>
+    <None Update="TestData\openai_responses_what_is_the_semantic_kernel.json">
+      <CopyToOutputDirectory>Always</CopyToOutputDirectory>
+    </None>
   </ItemGroup>
 
 </Project>

dotnet/src/Agents/UnitTests/OpenAI/OpenAIResponsesAgentTests.cs

@@ -0,0 +1,80 @@
+// Copyright (c) Microsoft. All rights reserved.
+
+using System;
+using System.ClientModel;
+using System.ClientModel.Primitives;
+using System.IO;
+using System.Linq;
+using System.Net.Http;
+using System.Threading.Tasks;
+using Microsoft.SemanticKernel;
+using Microsoft.SemanticKernel.Agents.OpenAI;
+using Microsoft.SemanticKernel.ChatCompletion;
+using OpenAI;
+using OpenAI.Responses;
+using Xunit;
+
+namespace SemanticKernel.Agents.UnitTests.OpenAI;
+
+/// <summary>
+/// Unit testing of <see cref="OpenAIResponsesAgent"/>.
+/// </summary>
+#pragma warning disable CS0419 // Ambiguous reference in cref attribute
+public sealed class OpenAIResponsesAgentTests : IDisposable
+{
+    /// <summary>
+    /// Initializes a new instance of objects required to test a <see cref="OpenAIResponsesAgent"/>.
+    /// </summary>
+    public OpenAIResponsesAgentTests()
+    {
+        this._messageHandlerStub = new MultipleHttpMessageHandlerStub();
+        this._httpClient = new HttpClient(this._messageHandlerStub, disposeHandler: false);
+        this._kernel = new Kernel();
+
+        var options = new OpenAIClientOptions
+        {
+            Transport = new HttpClientPipelineTransport(this._httpClient)
+        };
+        this._client = new(model: "gpt-4o", credential: new ApiKeyCredential("apikey"), options: options);
+    }
+
+    /// <inheritdoc/>
+    public void Dispose()
+    {
+        this._messageHandlerStub.Dispose();
+        this._httpClient.Dispose();
+    }
+
+    /// <summary>
+    /// Verify InvokeAsync
+    /// </summary>
+    [Fact]
+    public async Task VerifyInvokeAsync()
+    {
+        // Arrange
+        this._messageHandlerStub.AddJsonResponse(File.ReadAllText(WhatIsTheSKResponseJson));
+        OpenAIResponsesAgent agent = new(this._client)
+        {
+            Name = "ResponseAgent",
+            Instructions = "Answer all queries in English and French.",
+        };
+
+        // Act
+        var responseItems = agent.InvokeAsync(new ChatMessageContent(AuthorRole.User, "What is the capital of France?"));
+
+        // Assert
+        Assert.NotNull(responseItems);
+        var messages = await responseItems.ToListAsync();
+        Assert.Single(messages);
+        Assert.Equal("The capital of France is Paris.\n\nLa capitale de la France est Paris.", messages[0].Message.Content);
+    }
+
+    #region private
+    private const string WhatIsTheSKResponseJson = "./TestData/openai_responses_what_is_the_semantic_kernel.json";
+
+    private readonly MultipleHttpMessageHandlerStub _messageHandlerStub;
+    private readonly HttpClient _httpClient;
+    private readonly Kernel _kernel;
+    private readonly OpenAIResponseClient _client;
+    #endregion
+}

dotnet/src/Agents/UnitTests/TestData/openai_responses_what_is_the_semantic_kernel.json

@@ -0,0 +1,56 @@
+{
+  "id": "resp_67e315cf0e148191a3424db903561ba704584dfd0b14dc67",
+  "object": "response",
+  "created_at": 1742935503,
+  "status": "completed",
+  "error": null,
+  "incomplete_details": null,
+  "instructions": "Answer all queries in English and French.",
+  "max_output_tokens": null,
+  "model": "gpt-4o-2024-08-06",
+  "output": [
+    {
+      "type": "message",
+      "id": "msg_67e315cf52f88191b5e63ef1d045b37e04584dfd0b14dc67",
+      "status": "completed",
+      "role": "assistant",
+      "content": [
+        {
+          "type": "output_text",
+          "text": "The capital of France is Paris.\n\nLa capitale de la France est Paris.",
+          "annotations": []
+        }
+      ]
+    }
+  ],
+  "parallel_tool_calls": true,
+  "previous_response_id": null,
+  "reasoning": {
+    "effort": null,
+    "generate_summary": null
+  },
+  "store": true,
+  "temperature": 1.0,
+  "text": {
+    "format": {
+      "type": "text"
+    }
+  },
+  "tool_choice": "auto",
+  "tools": [],
+  "top_p": 1.0,
+  "truncation": "disabled",
+  "usage": {
+    "input_tokens": 44,
+    "input_tokens_details": {
+      "cached_tokens": 0
+    },
+    "output_tokens": 16,
+    "output_tokens_details": {
+      "reasoning_tokens": 0
+    },
+    "total_tokens": 60
+  },
+  "user": "ResponseAgent",
+  "metadata": {}
+}

dotnet/src/InternalUtilities/samples/AgentUtilities/BaseResponsesAgentTest.cs

@@ -28,7 +28,7 @@ protected BaseResponsesAgentTest(ITestOutputHelper output) : base(output)
         this.Client = new(model: "gpt-4o", credential: new ApiKeyCredential(TestConfiguration.OpenAI.ApiKey), options: options);
     }
 
-    protected bool EnableLogging { get; set; } = false;
+    protected bool EnableLogging { get; set; } = true;
 
     /// <inheritdoc/>
     protected override OpenAIResponseClient Client { get; }