fixing tests

microsoft · May 19, 2023 · 9e9a6a1 · 9e9a6a1
1 parent cca83e2
commit 9e9a6a1
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 4 deletions.
diff --git a/dotnet/src/IntegrationTests/Connectors/OpenAI/OpenAICompletionTests.cs b/dotnet/src/IntegrationTests/Connectors/OpenAI/OpenAICompletionTests.cs
@@ -3,11 +3,13 @@
 using System;
 using System.Collections.Generic;
 using System.Linq;
+using System.Net;
 using System.Threading.Tasks;
 using Microsoft.Extensions.Configuration;
 using Microsoft.SemanticKernel;
 using Microsoft.SemanticKernel.AI;
 using Microsoft.SemanticKernel.Orchestration;
+using Microsoft.SemanticKernel.Reliability;
 using Microsoft.SemanticKernel.SkillDefinition;
 using SemanticKernel.IntegrationTests.TestSettings;
 using Xunit;
@@ -86,8 +88,10 @@ public async Task OpenAIChatAsTextTestAsync(string prompt, string expectedAnswer
     public async Task CanUseOpenAiChatForTextCompletionAsync()
     {
         // Note: we use OpenAi Chat Completion and GPT 3.5 Turbo
-        IKernel target = Kernel.Builder.WithLogger(this._logger).Build();
-        this.ConfigureChatOpenAI(target);
+        KernelBuilder builder = Kernel.Builder.WithLogger(this._logger);
+        this.ConfigureChatOpenAI(builder);
+
+        IKernel target = builder.Build();
 
         var func = target.CreateSemanticFunction(
             "List the two planets after '{{$input}}', excluding moons, using bullet points.");
@@ -169,17 +173,21 @@ public async Task AzureOpenAIHttpRetryPolicyTestAsync(string prompt, string expe
         // Arrange
         var retryConfig = new HttpRetryConfig();
         retryConfig.RetryableStatusCodes.Add(HttpStatusCode.Unauthorized);
-        IKernel target = Kernel.Builder.WithLogger(this._testOutputHelper).Configure(c => c.SetDefaultHttpRetryConfig(retryConfig)).Build();
+        KernelBuilder builder = Kernel.Builder
+            .WithLogger(this._testOutputHelper)
+            .Configure(c => c.SetDefaultHttpRetryConfig(retryConfig));
 
         var azureOpenAIConfiguration = this._configuration.GetSection("AzureOpenAI").Get<AzureOpenAIConfiguration>();
         Assert.NotNull(azureOpenAIConfiguration);
 
         // Use an invalid API key to force a 401 Unauthorized response
-        target.Config.AddAzureTextCompletionService(
+        builder.WithAzureTextCompletionService(
             deploymentName: azureOpenAIConfiguration.DeploymentName,
             endpoint: azureOpenAIConfiguration.Endpoint,
             apiKey: "INVALID_KEY");
 
+        IKernel target = builder.Build();
+
         IDictionary<string, ISKFunction> skill = TestHelpers.GetSkills(target, "SummarizeSkill");
 
         // Act

diff --git a/dotnet/src/SemanticKernel/Planning/Plan.cs b/dotnet/src/SemanticKernel/Planning/Plan.cs
@@ -101,6 +101,7 @@ public Plan(string goal)
     public Plan(string goal, params ISKFunction[] steps) : this(goal)
     {
         this.AddSteps(steps);
+
     }
 
     /// <summary>