From a8d932d11fbb5f287fc68ca7ca3086078db7bc9f Mon Sep 17 00:00:00 2001 From: SergeyMenshykh Date: Tue, 2 Jul 2024 10:50:53 +0100 Subject: [PATCH 1/2] feat(AzureAISDKV2): move AzureOpenAIChatCompletionService to the Services folder to follow the approach taken in the OpenAIV2 project. --- .../AzureOpenAIChatCompletionService.cs | 0 1 file changed, 0 insertions(+), 0 deletions(-) rename dotnet/src/Connectors/Connectors.AzureOpenAI/{ChatCompletion => Services}/AzureOpenAIChatCompletionService.cs (100%) diff --git a/dotnet/src/Connectors/Connectors.AzureOpenAI/ChatCompletion/AzureOpenAIChatCompletionService.cs b/dotnet/src/Connectors/Connectors.AzureOpenAI/Services/AzureOpenAIChatCompletionService.cs similarity index 100% rename from dotnet/src/Connectors/Connectors.AzureOpenAI/ChatCompletion/AzureOpenAIChatCompletionService.cs rename to dotnet/src/Connectors/Connectors.AzureOpenAI/Services/AzureOpenAIChatCompletionService.cs From 4c4d8a8089926af7ede89e8dda898ecdbf9c21d1 Mon Sep 17 00:00:00 2001 From: SergeyMenshykh Date: Tue, 2 Jul 2024 10:57:31 +0100 Subject: [PATCH 2/2] fix: mve AzureOpenAIChatCompletionServiceTests to the Services folder --- .../AzureOpenAIChatCompletionServiceTests.cs | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) rename dotnet/src/Connectors/Connectors.AzureOpenAI.UnitTests/{ChatCompletion => Services}/AzureOpenAIChatCompletionServiceTests.cs (99%) diff --git a/dotnet/src/Connectors/Connectors.AzureOpenAI.UnitTests/ChatCompletion/AzureOpenAIChatCompletionServiceTests.cs b/dotnet/src/Connectors/Connectors.AzureOpenAI.UnitTests/Services/AzureOpenAIChatCompletionServiceTests.cs similarity index 99% rename from dotnet/src/Connectors/Connectors.AzureOpenAI.UnitTests/ChatCompletion/AzureOpenAIChatCompletionServiceTests.cs rename to dotnet/src/Connectors/Connectors.AzureOpenAI.UnitTests/Services/AzureOpenAIChatCompletionServiceTests.cs index 3b3c90687b45..13e09bd39e71 100644 --- a/dotnet/src/Connectors/Connectors.AzureOpenAI.UnitTests/ChatCompletion/AzureOpenAIChatCompletionServiceTests.cs +++ b/dotnet/src/Connectors/Connectors.AzureOpenAI.UnitTests/Services/AzureOpenAIChatCompletionServiceTests.cs @@ -20,7 +20,7 @@ using Moq; using OpenAI.Chat; -namespace SemanticKernel.Connectors.AzureOpenAI.UnitTests.ChatCompletion; +namespace SemanticKernel.Connectors.AzureOpenAI.UnitTests.Services; /// /// Unit tests for @@ -690,7 +690,7 @@ public async Task GetChatMessageContentsWithChatMessageContentItemCollectionAndS public async Task FunctionCallsShouldBePropagatedToCallersViaChatMessageItemsOfTypeFunctionCallContentAsync() { // Arrange - this._messageHandlerStub.ResponsesToReturn.Add(new HttpResponseMessage(System.Net.HttpStatusCode.OK) + this._messageHandlerStub.ResponsesToReturn.Add(new HttpResponseMessage(HttpStatusCode.OK) { Content = new StringContent(AzureOpenAITestHelper.GetTestResponse("chat_completion_multiple_function_calls_test_response.json")) }); @@ -752,7 +752,7 @@ public async Task FunctionCallsShouldBePropagatedToCallersViaChatMessageItemsOfT public async Task FunctionCallsShouldBeReturnedToLLMAsync() { // Arrange - this._messageHandlerStub.ResponsesToReturn.Add(new HttpResponseMessage(System.Net.HttpStatusCode.OK) + this._messageHandlerStub.ResponsesToReturn.Add(new HttpResponseMessage(HttpStatusCode.OK) { Content = new StringContent(AzureOpenAITestHelper.GetTestResponse("chat_completion_test_response.json")) }); @@ -810,7 +810,7 @@ public async Task FunctionCallsShouldBeReturnedToLLMAsync() public async Task FunctionResultsCanBeProvidedToLLMAsOneResultPerChatMessageAsync() { // Arrange - this._messageHandlerStub.ResponsesToReturn.Add(new HttpResponseMessage(System.Net.HttpStatusCode.OK) + this._messageHandlerStub.ResponsesToReturn.Add(new HttpResponseMessage(HttpStatusCode.OK) { Content = new StringContent(AzureOpenAITestHelper.GetTestResponse("chat_completion_test_response.json")) }); @@ -858,7 +858,7 @@ public async Task FunctionResultsCanBeProvidedToLLMAsOneResultPerChatMessageAsyn public async Task FunctionResultsCanBeProvidedToLLMAsManyResultsInOneChatMessageAsync() { // Arrange - this._messageHandlerStub.ResponsesToReturn.Add(new HttpResponseMessage(System.Net.HttpStatusCode.OK) + this._messageHandlerStub.ResponsesToReturn.Add(new HttpResponseMessage(HttpStatusCode.OK) { Content = new StringContent(AzureOpenAITestHelper.GetTestResponse("chat_completion_test_response.json")) });