Fix TestDistributedLockProvider registration to properly decorate IDistributedLockProvider (#7166)

* Initial plan

* Fix TestDistributedLockProvider registration to use Decorate pattern and fix variable reference bug

Co-authored-by: sfmskywalker <938393+sfmskywalker@users.noreply.github.com>

* Add runtime check for TestDistributedLockProvider registration

Co-authored-by: sfmskywalker <938393+sfmskywalker@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: sfmskywalker <938393+sfmskywalker@users.noreply.github.com>

Author: Copilot

src/modules/Elsa.Workflows.Runtime.Distributed/Services/DistributedWorkflowClient.cs

@@ -146,7 +146,7 @@ private static ResiliencePipeline CreateRetryPipeline(
                 Delay = TimeSpan.FromMilliseconds(500),
                 BackoffType = DelayBackoffType.Exponential,
                 UseJitter = true,
-                ShouldHandle = new PredicateBuilder().Handle<Exception>(ex => transientExceptionDetectionService.IsTransient(ex)),
+                ShouldHandle = new PredicateBuilder().Handle<Exception>(transientExceptionDetector.IsTransient),
                 OnRetry = args =>
                 {
                     logger.LogWarning(args.Outcome.Exception, "Transient error acquiring lock for workflow instance {WorkflowInstanceId}. Attempt {AttemptNumber} of {MaxAttempts}.", workflowInstanceId, args.AttemptNumber + 1, maxRetryAttempts);

test/component/Elsa.Workflows.ComponentTests/Helpers/Fixtures/WorkflowServer.cs

@@ -129,7 +129,17 @@ protected override void ConfigureWebHost(IWebHostBuilder builder)
 
         builder.ConfigureTestServices(services =>
         {
-            services.AddSingleton<TestDistributedLockProvider>();
+            // Decorate IDistributedLockProvider with TestDistributedLockProvider so tests use it
+            services.Decorate<IDistributedLockProvider, TestDistributedLockProvider>();
+            
+            // Also register TestDistributedLockProvider as itself so tests can access it directly for configuration
+            services.AddSingleton(sp =>
+            {
+                var provider = sp.GetRequiredService<IDistributedLockProvider>();
+                if (provider is not TestDistributedLockProvider testProvider)
+                    throw new InvalidOperationException($"Expected IDistributedLockProvider to be decorated with TestDistributedLockProvider, but got {provider.GetType().Name}");
+                return testProvider;
+            });
 
             services
                 .AddSingleton<SignalManager>()