From 7194ae5c0f01a31a860cb4495e56c0fd1d77174a Mon Sep 17 00:00:00 2001 From: Roger Barreto <19890735+rogerbarreto@users.noreply.github.com> Date: Tue, 2 Sep 2025 14:58:27 +0100 Subject: [PATCH] .NET: Skipping non supported integration scenarios for OpenAI Responses (#586) * Skipping non supported integration scenarios for OpenAI Responses * Update dotnet/src/Microsoft.Extensions.AI.Agents.OpenAI/NewOpenAIResponsesChatClient.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --- ...nAIResponseChatClientAgentRunStreamingTests.cs | 15 +++++++++++++++ .../OpenAIResponseChatClientAgentRunTests.cs | 15 +++++++++++++++ .../OpenAIResponseRunStreamingTests.cs | 14 ++++++++++++++ .../OpenAIResponseRunTests.cs | 14 ++++++++++++++ 4 files changed, 58 insertions(+) diff --git a/dotnet/tests/OpenAIResponse.IntegrationTests/OpenAIResponseChatClientAgentRunStreamingTests.cs b/dotnet/tests/OpenAIResponse.IntegrationTests/OpenAIResponseChatClientAgentRunStreamingTests.cs index 171eecfd47..885f47a27a 100644 --- a/dotnet/tests/OpenAIResponse.IntegrationTests/OpenAIResponseChatClientAgentRunStreamingTests.cs +++ b/dotnet/tests/OpenAIResponse.IntegrationTests/OpenAIResponseChatClientAgentRunStreamingTests.cs @@ -1,13 +1,28 @@ // Copyright (c) Microsoft. All rights reserved. +using System.Threading.Tasks; using AgentConformance.IntegrationTests; namespace OpenAIResponse.IntegrationTests; public class OpenAIResponseStoreTrueChatClientAgentRunStreamingTests() : ChatClientAgentRunStreamingTests(() => new(store: true)) { + private const string SkipReason = "OpenAIResponse does not support empty messages"; + + [Fact(Skip = SkipReason)] + public override Task RunWithInstructionsAndNoMessageReturnsExpectedResultAsync() + { + return Task.CompletedTask; + } } public class OpenAIResponseStoreFalseChatClientAgentRunStreamingTests() : ChatClientAgentRunStreamingTests(() => new(store: false)) { + private const string SkipReason = "OpenAIResponse does not support empty messages"; + + [Fact(Skip = SkipReason)] + public override Task RunWithInstructionsAndNoMessageReturnsExpectedResultAsync() + { + return Task.CompletedTask; + } } diff --git a/dotnet/tests/OpenAIResponse.IntegrationTests/OpenAIResponseChatClientAgentRunTests.cs b/dotnet/tests/OpenAIResponse.IntegrationTests/OpenAIResponseChatClientAgentRunTests.cs index d146ac7dbe..1eabc64612 100644 --- a/dotnet/tests/OpenAIResponse.IntegrationTests/OpenAIResponseChatClientAgentRunTests.cs +++ b/dotnet/tests/OpenAIResponse.IntegrationTests/OpenAIResponseChatClientAgentRunTests.cs @@ -1,13 +1,28 @@ // Copyright (c) Microsoft. All rights reserved. +using System.Threading.Tasks; using AgentConformance.IntegrationTests; namespace OpenAIResponse.IntegrationTests; public class OpenAIResponseStoreTrueChatClientAgentRunTests() : ChatClientAgentRunTests(() => new(store: true)) { + private const string SkipReason = "OpenAIResponse does not support empty messages"; + + [Fact(Skip = SkipReason)] + public override Task RunWithInstructionsAndNoMessageReturnsExpectedResultAsync() + { + return Task.CompletedTask; + } } public class OpenAIResponseStoreFalseChatClientAgentRunTests() : ChatClientAgentRunTests(() => new(store: false)) { + private const string SkipReason = "OpenAIResponse does not support empty messages"; + + [Fact(Skip = SkipReason)] + public override Task RunWithInstructionsAndNoMessageReturnsExpectedResultAsync() + { + return Task.CompletedTask; + } } diff --git a/dotnet/tests/OpenAIResponse.IntegrationTests/OpenAIResponseRunStreamingTests.cs b/dotnet/tests/OpenAIResponse.IntegrationTests/OpenAIResponseRunStreamingTests.cs index 00f8a04a68..7a384a14ee 100644 --- a/dotnet/tests/OpenAIResponse.IntegrationTests/OpenAIResponseRunStreamingTests.cs +++ b/dotnet/tests/OpenAIResponse.IntegrationTests/OpenAIResponseRunStreamingTests.cs @@ -1,13 +1,27 @@ // Copyright (c) Microsoft. All rights reserved. +using System.Threading.Tasks; using AgentConformance.IntegrationTests; namespace OpenAIResponse.IntegrationTests; public class OpenAIResponseStoreTrueRunStreamingTests() : RunStreamingTests(() => new(store: true)) { + private const string SkipReason = "OpenAIResponse does not support empty messages"; + [Fact(Skip = SkipReason)] + public override Task RunWithNoMessageDoesNotFailAsync() + { + return Task.CompletedTask; + } } public class OpenAIResponseStoreFalseRunStreamingTests() : RunStreamingTests(() => new(store: false)) { + private const string SkipReason = "OpenAIResponse does not support empty messages"; + + [Fact(Skip = SkipReason)] + public override Task RunWithNoMessageDoesNotFailAsync() + { + return Task.CompletedTask; + } } diff --git a/dotnet/tests/OpenAIResponse.IntegrationTests/OpenAIResponseRunTests.cs b/dotnet/tests/OpenAIResponse.IntegrationTests/OpenAIResponseRunTests.cs index 1a8fbb3286..1d4f3a4034 100644 --- a/dotnet/tests/OpenAIResponse.IntegrationTests/OpenAIResponseRunTests.cs +++ b/dotnet/tests/OpenAIResponse.IntegrationTests/OpenAIResponseRunTests.cs @@ -1,13 +1,27 @@ // Copyright (c) Microsoft. All rights reserved. +using System.Threading.Tasks; using AgentConformance.IntegrationTests; namespace OpenAIResponse.IntegrationTests; public class OpenAIResponseStoreTrueRunTests() : RunTests(() => new(store: true)) { + private const string SkipReason = "OpenAIResponse does not support empty messages"; + [Fact(Skip = SkipReason)] + public override Task RunWithNoMessageDoesNotFailAsync() + { + return Task.CompletedTask; + } } public class OpenAIResponseStoreFalseRunTests() : RunTests(() => new(store: false)) { + private const string SkipReason = "OpenAIResponse does not support empty messages"; + + [Fact(Skip = SkipReason)] + public override Task RunWithNoMessageDoesNotFailAsync() + { + return Task.CompletedTask; + } }