From 704d4cd8348e2180cfdeff1e79a3913d2cb1bd5a Mon Sep 17 00:00:00 2001
From: Sipke Schoorstra <sipkeschoorstra@outlook.com>
Date: Fri, 29 Dec 2023 11:34:58 +0100
Subject: [PATCH] Remove ElsaClient and related interfaces

Files ElsaClient.cs, IElsaClient.cs and IElsaClientFactory.cs have been deleted from the Elsa.Api.Client project. The commit also includes an update in the DependencyInjectionExtensions.cs file where IElsaClient service registration is removed, reflecting these changes.
---
 .../Elsa.Api.Client/Contracts/IElsaClient.cs  | 37 ----------------
 .../Contracts/IElsaClientFactory.cs           | 14 ------
 .../DependencyInjectionExtensions.cs          |  3 --
 .../Elsa.Api.Client/Services/ElsaClient.cs    | 43 -------------------
 4 files changed, 97 deletions(-)
 delete mode 100644 src/clients/Elsa.Api.Client/Contracts/IElsaClient.cs
 delete mode 100644 src/clients/Elsa.Api.Client/Contracts/IElsaClientFactory.cs
 delete mode 100644 src/clients/Elsa.Api.Client/Services/ElsaClient.cs

diff --git a/src/clients/Elsa.Api.Client/Contracts/IElsaClient.cs b/src/clients/Elsa.Api.Client/Contracts/IElsaClient.cs
deleted file mode 100644
index 1530f9190..000000000
--- a/src/clients/Elsa.Api.Client/Contracts/IElsaClient.cs
+++ /dev/null
@@ -1,37 +0,0 @@
-using Elsa.Api.Client.Resources.ActivityDescriptorOptions.Contracts;
-using Elsa.Api.Client.Resources.ActivityDescriptors.Contracts;
-using Elsa.Api.Client.Resources.Scripting.Contracts;
-using Elsa.Api.Client.Resources.WorkflowDefinitions.Contracts;
-using Elsa.Api.Client.Resources.WorkflowInstances.Contracts;
-
-namespace Elsa.Api.Client.Contracts;
-
-/// <summary>
-/// Represents a client for the Elsa API. Each API is exposed as a property.
-/// </summary>
-public interface IElsaClient
-{
-    /// <summary>
-    /// Gets the workflow definitions API.
-    /// </summary>
-    IWorkflowDefinitionsApi WorkflowDefinitions { get; }
-
-    /// <summary>
-    /// Gets the activity descriptors API.
-    /// </summary>
-    IActivityDescriptorsApi ActivityDescriptors { get; }
-
-    /// <summary>
-    /// Get the activity descriptor Options API
-    /// </summary>
-    IActivityDescriptorOptionsApi ActivityDescriptorOptions { get; }
-    /// <summary>
-    /// Gets the workflow instances API.
-    /// </summary>
-    IWorkflowInstancesApi WorkflowInstances { get; }
-
-    /// <summary>
-    /// Gets the javascript API.
-    /// </summary>
-    IJavaScriptApi JavaScript { get; }
-}
\ No newline at end of file
diff --git a/src/clients/Elsa.Api.Client/Contracts/IElsaClientFactory.cs b/src/clients/Elsa.Api.Client/Contracts/IElsaClientFactory.cs
deleted file mode 100644
index cf109850e..000000000
--- a/src/clients/Elsa.Api.Client/Contracts/IElsaClientFactory.cs
+++ /dev/null
@@ -1,14 +0,0 @@
-using Elsa.Api.Client.Options;
-
-namespace Elsa.Api.Client.Contracts;
-
-/// <summary>
-/// A factory for creating <see cref="IElsaClient"/> instances.
-/// </summary>
-public interface IElsaClientFactory
-{
-    /// <summary>
-    /// Creates a new <see cref="IElsaClient"/> instance.
-    /// </summary>
-    IElsaClient CreateClient(Action<ElsaClientOptions> configureOptions, Action<ElsaClientBuilderOptions>? configureHttpClientBuilder = default);
-}
\ No newline at end of file
diff --git a/src/clients/Elsa.Api.Client/Extensions/DependencyInjectionExtensions.cs b/src/clients/Elsa.Api.Client/Extensions/DependencyInjectionExtensions.cs
index 86168c4ab..ff389d9fc 100644
--- a/src/clients/Elsa.Api.Client/Extensions/DependencyInjectionExtensions.cs
+++ b/src/clients/Elsa.Api.Client/Extensions/DependencyInjectionExtensions.cs
@@ -1,6 +1,5 @@
 using System.Text.Json;
 using System.Text.Json.Serialization;
-using Elsa.Api.Client.Contracts;
 using Elsa.Api.Client.Converters;
 using Elsa.Api.Client.HttpMessageHandlers;
 using Elsa.Api.Client.Options;
@@ -17,7 +16,6 @@ using Elsa.Api.Client.Resources.WorkflowActivationStrategies.Contracts;
 using Elsa.Api.Client.Resources.WorkflowDefinitions.Contracts;
 using Elsa.Api.Client.Resources.WorkflowExecutionContexts.Contracts;
 using Elsa.Api.Client.Resources.WorkflowInstances.Contracts;
-using Elsa.Api.Client.Services;
 using JetBrains.Annotations;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Options;
@@ -66,7 +64,6 @@ public static class DependencyInjectionExtensions
         configureBuilderOptions?.Invoke(builderOptions);
 
         services.Configure(configureOptions ?? (_ => { }));
-        services.AddScoped<IElsaClient, ElsaClient>();
         services.AddApi<IWorkflowDefinitionsApi>(builderOptions);
         services.AddApi<IWorkflowInstancesApi>(builderOptions);
         services.AddApi<IActivityDescriptorsApi>(builderOptions);
diff --git a/src/clients/Elsa.Api.Client/Services/ElsaClient.cs b/src/clients/Elsa.Api.Client/Services/ElsaClient.cs
deleted file mode 100644
index 3a0008512..000000000
--- a/src/clients/Elsa.Api.Client/Services/ElsaClient.cs
+++ /dev/null
@@ -1,43 +0,0 @@
-using Elsa.Api.Client.Contracts;
-using Elsa.Api.Client.Resources.ActivityDescriptorOptions.Contracts;
-using Elsa.Api.Client.Resources.ActivityDescriptors.Contracts;
-using Elsa.Api.Client.Resources.Scripting.Contracts;
-using Elsa.Api.Client.Resources.WorkflowDefinitions.Contracts;
-using Elsa.Api.Client.Resources.WorkflowInstances.Contracts;
-
-namespace Elsa.Api.Client.Services;
-
-/// <inheritdoc />
-public class ElsaClient : IElsaClient
-{
-    /// <summary>
-    /// Initializes a new instance of the <see cref="ElsaClient"/> class.
-    /// </summary>
-    public ElsaClient(IWorkflowDefinitionsApi workflowDefinitions
-        , IWorkflowInstancesApi workflowInstances
-        , IActivityDescriptorsApi activityDescriptors
-        , IActivityDescriptorOptionsApi activityDescriptorOptions
-        , IJavaScriptApi javaScript)
-    {
-        WorkflowDefinitions = workflowDefinitions;
-        WorkflowInstances = workflowInstances;
-        ActivityDescriptors = activityDescriptors;
-        ActivityDescriptorOptions = activityDescriptorOptions;
-        JavaScript = javaScript;
-    }
-
-    /// <inheritdoc />
-    public IWorkflowDefinitionsApi WorkflowDefinitions { get; }
-
-    /// <inheritdoc />
-    public IActivityDescriptorsApi ActivityDescriptors { get; }
-    
-    /// <inheritdoc />
-    public IActivityDescriptorOptionsApi ActivityDescriptorOptions { get; }
-
-    /// <inheritdoc />
-    public IWorkflowInstancesApi WorkflowInstances { get; }
-
-    /// <inheritdoc />
-    public IJavaScriptApi JavaScript { get; }
-}
\ No newline at end of file