refactor: simplify DI registration

cnblogs · Feb 13, 2024 · 8e73712 · 8e73712
1 parent 7956cd9
commit 8e73712
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 16 deletions.
diff --git a/src/SemanticKernel.DashScope/DashScopeServiceCollectionExtensions.cs b/src/SemanticKernel.DashScope/DashScopeServiceCollectionExtensions.cs
@@ -1,5 +1,7 @@
 using Cnblogs.SemanticKernel.Connectors.DashScope;
+using Microsoft.Extensions.Configuration;
 using Microsoft.Extensions.DependencyInjection;
+using Microsoft.Extensions.DependencyInjection.Extensions;
 using Microsoft.SemanticKernel.ChatCompletion;
 
 namespace Microsoft.SemanticKernel;
@@ -28,4 +30,23 @@ public static IKernelBuilder AddDashScopeChatCompletion(
         builder.Services.AddKeyedSingleton<IChatCompletionService>(serviceId, factory);
         return builder;
     }
+
+    public static IKernelBuilder AddDashScopeChatCompletion<T>(
+        this IKernelBuilder builder,
+        string? serviceId = null,
+        Action<HttpClient>? configureClient = null,
+        string configSectionPath = "dashscope") where T : class
+    {
+        if (!builder.Services.Any(s => s.ServiceType == typeof(IConfiguration)))
+        {
+            IConfiguration config = new ConfigurationBuilder()
+                .SetBasePath(Directory.GetCurrentDirectory())
+                .AddEnvironmentVariables()
+                .AddJsonFile("appsettings.json", true)
+                .AddUserSecrets<T>()
+                .Build();
+            builder.Services.TryAddSingleton(config);
+        }
+        return builder.AddDashScopeChatCompletion(serviceId, configureClient, configSectionPath);
+    }
 }
diff --git a/test/SemanticKernel.DashScope.IntegrationTest/DashScopeChatCompletionTests.cs b/test/SemanticKernel.DashScope.IntegrationTest/DashScopeChatCompletionTests.cs
@@ -1,7 +1,5 @@
 using System.Diagnostics;
 using System.Text;
-using Microsoft.Extensions.Configuration;
-using Microsoft.Extensions.DependencyInjection;
 using Microsoft.SemanticKernel;
 
 namespace SemanticKernel.DashScope.IntegrationTest;
@@ -13,8 +11,7 @@ public async Task ChatCompletion_InvokePromptAsync_WorksCorrectly()
     {
         // Arrange
         var builder = Kernel.CreateBuilder();
-        builder.Services.AddSingleton(GetConfiguration());
-        builder.AddDashScopeChatCompletion();
+        builder.AddDashScopeChatCompletion<DashScopeChatCompletionTests>();
         var kernel = builder.Build();
 
         var prompt = @"<message role=""user"">博客园是什么网站</message>";
@@ -40,8 +37,7 @@ public async Task ChatCompletion_InvokePromptStreamingAsync_WorksCorrectly()
     {
         // Arrange
         var builder = Kernel.CreateBuilder();
-        builder.Services.AddSingleton(GetConfiguration());
-        builder.AddDashScopeChatCompletion();
+        builder.AddDashScopeChatCompletion<DashScopeChatCompletionTests>();
         var kernel = builder.Build();
 
         // Act
@@ -52,18 +48,9 @@ public async Task ChatCompletion_InvokePromptStreamingAsync_WorksCorrectly()
         var sb = new StringBuilder();
         await foreach (var message in result)
         {
-            Trace.WriteLine(message);
+            Trace.Write(message);
             sb.Append(message);
         }
         Assert.Contains("博客园", sb.ToString());
     }
-
-    private static IConfiguration GetConfiguration()
-    {
-        return new ConfigurationBuilder()
-            .SetBasePath(Directory.GetCurrentDirectory())
-            .AddJsonFile("appsettings.json")
-            .AddUserSecrets<DashScopeChatCompletionTests>()
-            .Build();
-    }
 }