output options provider

DavidVollmers · May 4, 2024 · 7d1995f · 7d1995f
1 parent f35eca3
commit 7d1995f
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 6 deletions.
diff --git a/src/Doki.CommandLine/JsonOutputOptionsProvider.cs b/src/Doki.CommandLine/JsonOutputOptionsProvider.cs
@@ -6,14 +6,25 @@ namespace Doki.CommandLine;
 
 internal class JsonOutputOptionsProvider : IOutputOptionsProvider
 {
-    public TOptions RequireOptions<TOutput, TOptions>(string outputType) where TOutput : class, IOutput
+    private static readonly JsonSerializerOptions JsonSerializerOptions = new()
+    {
+        PropertyNameCaseInsensitive = true
+    };
+
+    private readonly Dictionary<string, JsonElement?> _options = new();
+
+    public TOptions? GetOptions<TOutput, TOptions>(string outputType) where TOutput : class, IOutput
         where TOptions : class, IOutputOptions<TOutput>
     {
-        throw new NotImplementedException();
+        ArgumentNullException.ThrowIfNull(outputType);
+
+        return _options.TryGetValue(outputType, out var options)
+            ? options?.Deserialize<TOptions>(JsonSerializerOptions)
+            : null;
     }
 
     public void AddOptions(string outputType, JsonElement? options)
     {
-        throw new NotImplementedException();
+        _options[outputType] = options;
     }
 }
diff --git a/src/Doki.Output.Extensions/IOutputOptionsProvider.cs b/src/Doki.Output.Extensions/IOutputOptionsProvider.cs
@@ -2,6 +2,6 @@
 
 public interface IOutputOptionsProvider
 {
-    TOptions RequireOptions<TOutput, TOptions>(string outputType) where TOutput : class, IOutput
+    TOptions? GetOptions<TOutput, TOptions>(string outputType) where TOutput : class, IOutput
         where TOptions : class, IOutputOptions<TOutput>;
 }
diff --git a/src/Doki.Output.Extensions/OutputExtensions.cs b/src/Doki.Output.Extensions/OutputExtensions.cs
@@ -19,9 +19,9 @@ public static IServiceCollection AddOutputOptions<TOutput, TOptions>(this IServi
         services.AddSingleton<IOutputOptions<TOutput>>(provider =>
         {
             var optionsProvider = provider.GetService<IOutputOptionsProvider>();
-            if (optionsProvider != null) return optionsProvider.RequireOptions<TOutput, TOptions>(outputType);
+            var options = optionsProvider?.GetOptions<TOutput, TOptions>(outputType);
 
-            return ActivatorUtilities.CreateInstance<TOptions>(provider);
+            return options ?? ActivatorUtilities.CreateInstance<TOptions>(provider);
         });
 
         return services;