Adding skip for integration tests

microsoft · Nov 12, 2024 · 8f4cefc · 8f4cefc
1 parent be34607
commit 8f4cefc
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 10 deletions.
diff --git a/.../Connectors/Connectors.AzureAIInference/Services/AzureAIInferenceChatCompletionService.cs b/.../Connectors/Connectors.AzureAIInference/Services/AzureAIInferenceChatCompletionService.cs
@@ -17,7 +17,7 @@ namespace Microsoft.SemanticKernel.Connectors.AzureAIInference;
 /// <summary>
 /// Chat completion service for Azure AI Inference.
 /// </summary>
-[Obsolete("Dedicated AzureAIInferenceChatCompletionService is deprecated. Use OllamaApiClient.AsChatCompletionService() instead.")]
+[Obsolete("Dedicated AzureAIInferenceChatCompletionService is deprecated. Use Azure.AI.Inference.ChatCompletionsClient.AsChatClient().AsChatCompletionService() instead.")]
 public sealed class AzureAIInferenceChatCompletionService : IChatCompletionService
 {
     private readonly ChatClientCore _core;

diff --git a/...nTests/Connectors/AzureAIInference/AzureAIInferenceChatCompletion_FunctionCallingTests.cs b/...nTests/Connectors/AzureAIInference/AzureAIInferenceChatCompletion_FunctionCallingTests.cs
@@ -44,7 +44,7 @@ public async Task CanAutoInvokeKernelFunctionsWithComplexTypeParametersAsync()
         Assert.Contains("42.8", result.GetValue<string>(), StringComparison.InvariantCulture); // The WeatherPlugin always returns 42.8 for Dublin, Ireland.
     }
 
-    [Fact]//(Skip = "For manual verification only")]
+    [Fact(Skip = "For manual verification only")]
     public async Task CanAutoInvokeKernelFunctionsWithPrimitiveTypeParametersAsync()
     {
         // Arrange
@@ -60,7 +60,7 @@ public async Task CanAutoInvokeKernelFunctionsWithPrimitiveTypeParametersAsync()
         Assert.Contains("10", result.GetValue<string>(), StringComparison.InvariantCulture);
     }
 
-    [Fact]//(Skip = "For manual verification only")]
+    [Fact(Skip = "For manual verification only")]
     public async Task CanAutoInvokeKernelFunctionsWithEnumTypeParametersAsync()
     {
         // Arrange
@@ -76,7 +76,7 @@ public async Task CanAutoInvokeKernelFunctionsWithEnumTypeParametersAsync()
         Assert.Contains("rain", result.GetValue<string>(), StringComparison.OrdinalIgnoreCase);
     }
 
-    [Fact]//(Skip = "For manual verification only")]
+    [Fact(Skip = "For manual verification only")]
     public async Task CanAutoInvokeKernelFunctionFromPromptAsync()
     {
         // Arrange
@@ -102,7 +102,7 @@ public async Task CanAutoInvokeKernelFunctionFromPromptAsync()
         Assert.Contains("Transportation", result.GetValue<string>(), StringComparison.InvariantCultureIgnoreCase);
     }
 
-    [Fact]//(Skip = "For manual verification only")]
+    [Fact(Skip = "For manual verification only")]
     public async Task ConnectorAgnosticFunctionCallingModelClassesCanBeUsedForManualFunctionCallingAsync()
     {
         // Arrange
@@ -142,7 +142,7 @@ public async Task ConnectorAgnosticFunctionCallingModelClassesCanBeUsedForManual
         Assert.Contains("rain", messageContent.Content, StringComparison.InvariantCultureIgnoreCase);
     }
 
-    [Fact]//(Skip = "For manual verification only")]
+    [Fact(Skip = "For manual verification only")]
     public async Task ConnectorAgnosticFunctionCallingModelClassesCanPassFunctionExceptionToConnectorAsync()
     {
         // Arrange
@@ -186,7 +186,7 @@ public async Task ConnectorAgnosticFunctionCallingModelClassesCanPassFunctionExc
         TestHelpers.AssertChatErrorExcuseMessage(messageContent.Content);
     }
 
-    [Fact]//(Skip = "For manual verification only")]
+    [Fact(Skip = "For manual verification only")]
     public async Task ConnectorAgnosticFunctionCallingModelClassesSupportSimulatedFunctionCallsAsync()
     {
         // Arrange
@@ -215,7 +215,7 @@ public async Task ConnectorAgnosticFunctionCallingModelClassesSupportSimulatedFu
         Assert.Contains("tornado", messageContent.Content, StringComparison.InvariantCultureIgnoreCase);
     }
 
-    [Fact]//(Skip = "For manual verification only")]
+    [Fact(Skip = "For manual verification only")]
     public async Task ConnectorAgnosticFunctionCallingModelClassesCanBeUsedForAutoFunctionCallingAsync()
     {
         // Arrange
@@ -264,7 +264,7 @@ public async Task ConnectorAgnosticFunctionCallingModelClassesCanBeUsedForAutoFu
         Assert.NotNull(getWeatherForCityFunctionCallResult.Result);
     }
 
-    [Fact]//(Skip = "For manual verification only")]
+    [Fact(Skip = "For manual verification only")]
     public async Task SubsetOfFunctionsCanBeUsedForFunctionCallingAsync()
     {
         // Arrange
@@ -288,7 +288,7 @@ public async Task SubsetOfFunctionsCanBeUsedForFunctionCallingAsync()
         Assert.Contains("Friday", result.Content, StringComparison.InvariantCulture);
     }
 
-    [Fact]//(Skip = "For manual verification only")]
+    [Fact(Skip = "For manual verification only")]
     public async Task RequiredFunctionShouldBeCalledAsync()
     {
         // Arrange