Improving test stability

giacomelli · Sep 6, 2022 · d3b739e · d3b739e
1 parent 5f2e185
commit d3b739e
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 16 deletions.
diff --git a/src/GeneticSharp.Domain.UnitTests/Randomizations/BasicRandomizationTest.cs b/src/GeneticSharp.Domain.UnitTests/Randomizations/BasicRandomizationTest.cs
@@ -72,16 +72,19 @@ public void GetDouble_NoArgs_RandomResult()
         [Repeat(10)]
         public void GetDouble_ManyThreads_DiffRandomResult()
         {
-            var target = new BasicRandomization();
-            var actual = new BlockingCollection<int>();
-
-            Parallel.For(0, 1000, (i) =>
+            FlowAssert.IsAtLeastOneAttemptOk(10, () =>
             {
-                actual.Add(target.GetInt(0, int.MaxValue));
-            });
+                var target = new BasicRandomization();
+                var actual = new BlockingCollection<int>();
 
-            Assert.AreEqual(1000, actual.Count);
-            Assert.AreEqual(1000, actual.Distinct().Count());
+                Parallel.For(0, 1000, (i) =>
+                {
+                    actual.Add(target.GetInt(0, int.MaxValue));
+                });
+
+                Assert.AreEqual(1000, actual.Count);
+                Assert.AreEqual(1000, actual.Distinct().Count());
+            });
         }
 
         [Test]

diff --git a/src/GeneticSharp.Domain.UnitTests/Randomizations/FastRandomRandomizationTest.cs b/src/GeneticSharp.Domain.UnitTests/Randomizations/FastRandomRandomizationTest.cs
@@ -72,16 +72,19 @@ public void GetDouble_NoArgs_RandomResult()
         [Repeat(10)]
         public void GetDouble_ManyThreads_DiffRandomResult()
         {
-            var target = new FastRandomRandomization();
-            var actual = new BlockingCollection<int>();
-
-            Parallel.For(0, 1000, (i) =>
+            FlowAssert.IsAtLeastOneAttemptOk(10, () =>
             {
-                actual.Add(target.GetInt(0, int.MaxValue));
-            });
+                var target = new FastRandomRandomization();
+                var actual = new BlockingCollection<int>();
 
-            Assert.AreEqual(1000, actual.Count);
-            Assert.AreEqual(1000, actual.Distinct().Count());
+                Parallel.For(0, 1000, (i) =>
+                {
+                    actual.Add(target.GetInt(0, int.MaxValue));
+                });
+
+                Assert.AreEqual(1000, actual.Count);
+                Assert.AreEqual(1000, actual.Distinct().Count());
+            });
         }
 
         [Test]