fix race condition

Author: lofcz

src/FastCloner.Tests/ConcurrentTests.cs

@@ -1,8 +1,9 @@
 using System.Collections.Concurrent;
 using FastCloner.Code;
-using System.Threading.Tasks;
 
 namespace FastCloner.Tests;
+
+[NotInParallel("FastClonerGlobalState")]
 public class ConcurrentTests(int maxRecursionDepth) : BaseTestFixture(maxRecursionDepth)
 {
     private class TestClass

src/FastCloner.Tests/TypeBehaviorTests.cs

@@ -825,21 +825,46 @@ public async Task GlobalConfigPublish_BumpsCacheVersion_ForSubsequentReaders()
         };
 
         long startingVersion = FastClonerCache.GetCacheVersion();
-
-        FastCloner.MaxRecursionDepth = 999;
+        
+        FastCloner.SetTypeBehavior<int>(CloneBehavior.Reference);
         _ = simple.DeepClone();
         _ = dictionary.DeepClone();
 
         long firstMutationVersion = FastClonerCache.GetCacheVersion();
         await Assert.That(firstMutationVersion).IsGreaterThan(startingVersion);
 
-        FastCloner.MaxRecursionDepth = 998;
+        FastCloner.ClearTypeBehavior<int>();
         _ = simple.DeepClone();
         _ = dictionary.DeepClone();
 
         await Assert.That(FastClonerCache.GetCacheVersion()).IsGreaterThan(firstMutationVersion);
     }
 
+    [Test]
+    public async Task MaxRecursionDepth_Update_Does_Not_Bump_Cache_Version()
+    {
+        FastCloner.ClearCache();
+        FastCloner.ClearAllTypeBehaviors();
+        FastCloner.SetDisableOptionalFeatures(false);
+
+        SimpleClass simple = new SimpleClass { IntValue = 7, StringValue = "Seven" };
+        _ = simple.DeepClone();
+
+        long versionBefore = FastClonerCache.GetCacheVersion();
+
+        FastCloner.MaxRecursionDepth = 999;
+        SimpleClass clone1 = simple.DeepClone();
+
+        FastCloner.MaxRecursionDepth = 998;
+        SimpleClass clone2 = simple.DeepClone();
+
+        await Assert.That(FastClonerCache.GetCacheVersion()).IsEqualTo(versionBefore);
+        await Assert.That(clone1.IntValue).IsEqualTo(7);
+        await Assert.That(clone2.IntValue).IsEqualTo(7);
+
+        FastCloner.MaxRecursionDepth = 1_000;
+    }
+
     [Test]
     public async Task RuntimeConfig_Create_DefaultValues_CanStaySingletonOrBecomeDistinctSnapshot()
     {

src/FastCloner/FastCloner.cs

@@ -45,7 +45,7 @@ public static int MaxRecursionDepth
                 if (maxRecursionDepth == value)
                     return;
 
-                PublishEngine(CreatePublishedEngine(value, disableOptionalFeatures, GetTypeBehaviors()));
+                PublishMaxRecursionDepth(value);
             }
         }
     }
@@ -197,6 +197,25 @@ private static void PublishEngine(FastClonerPublishedEngine engine)
         FastClonerCache.ClearCache();
         Volatile.Write(ref publishedEngine, engine);
     }
+    
+    private static void PublishMaxRecursionDepth(int value)
+    {
+        FastClonerPublishedEngine current = Volatile.Read(ref publishedEngine);
+        FastClonerRuntimeConfig oldConfig = current.RuntimeConfig;
+        FastClonerRuntimeConfig newConfig = FastClonerRuntimeConfig.Create(
+            value,
+            oldConfig.DisableOptionalFeatures,
+            oldConfig.CloneTypeBehaviors(),
+            oldConfig.CacheKey,
+            useStartupDefaultSingleton: true);
+
+        FastClonerPublishedEngine newEngine = ReferenceEquals(newConfig, FastClonerRuntimeConfig.Default)
+            ? FastClonerPublishedEngine.StartupDefault
+            : FastClonerPublishedEngine.CreateConfigured(newConfig);
+
+        maxRecursionDepth = value;
+        Volatile.Write(ref publishedEngine, newEngine);
+    }
 
     private static FastClonerPublishedEngine CreatePublishedEngine(
         int maxRecursionDepth,