From c7481b2e646a880433789a0287ad5b8e94fd2eff Mon Sep 17 00:00:00 2001
From: Sipke Schoorstra <sipkeschoorstra@outlook.com>
Date: Sat, 30 Mar 2024 21:59:23 +0100
Subject: [PATCH] Refactor synchronization provider selection in Program.cs

Rearranged the order in which synchronization providers are processed and removed some unnecessary empty lines. Specifically, moved the "File" case above the "Noop" case to optimize the synchronization provider selection process, as "File" is expected to be a more common use case.
---
 src/bundles/Elsa.Server.Web/Program.cs | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/src/bundles/Elsa.Server.Web/Program.cs b/src/bundles/Elsa.Server.Web/Program.cs
index b5c8bb89f..65de00d48 100644
--- a/src/bundles/Elsa.Server.Web/Program.cs
+++ b/src/bundles/Elsa.Server.Web/Program.cs
@@ -137,7 +137,7 @@ services
                             ef.UsePostgreSql(cockroachDbConnectionString!);
                         else
                             ef.UseSqlite(sqliteConnectionString);
-                        
+
                         ef.RunMigrations = runEFCoreMigrations;
                     });
 
@@ -146,8 +146,8 @@ services
 
                 if (useMemoryStores)
                     management.UseWorkflowInstances(feature => feature.WorkflowInstanceStore = sp => sp.GetRequiredService<MemoryWorkflowInstanceStore>());
-                
-                if(useMassTransit)
+
+                if (useMassTransit)
                     management.UseMassTransitDispatcher();
             })
             .UseWorkflowRuntime(runtime =>
@@ -167,7 +167,7 @@ services
                             ef.UsePostgreSql(cockroachDbConnectionString!);
                         else
                             ef.UseSqlite(sqliteConnectionString);
-                        
+
                         ef.RunMigrations = runEFCoreMigrations;
                     });
 
@@ -213,10 +213,11 @@ services
                                 var database = connectionMultiplexer.GetDatabase();
                                 return new RedisDistributedSynchronizationProvider(database);
                             }
-                        case "Noop":
-                            return new NoopDistributedSynchronizationProvider();
-                        default:
+                        case "File":
                             return new FileDistributedSynchronizationProvider(new DirectoryInfo(Path.Combine(Directory.GetCurrentDirectory(), "App_Data", "locks")));
+                        case "Noop":
+                        default:
+                            return new NoopDistributedSynchronizationProvider();
                     }
                 };
             })
@@ -282,7 +283,7 @@ services
                             ef.UsePostgreSql(cockroachDbConnectionString!);
                         else
                             ef.UseSqlite(sqliteConnectionString);
-                        
+
                         ef.RunMigrations = runEFCoreMigrations;
                     });
                 }