Retry time-based tests (#583)

bitfaster · web-flow · commit 32929107c53f · 2024-05-12T19:14:39.000-07:00
* +tests

* tlfu added

---------
diff --git a/BitFaster.Caching.UnitTests/Lfu/ConcurrentLfuCoreTests.cs b/BitFaster.Caching.UnitTests/Lfu/ConcurrentLfuCoreTests.cs
@@ -3,6 +3,7 @@
 using System.Collections.Generic;
 using System.Threading.Tasks;
 using BitFaster.Caching.Lfu;
+using BitFaster.Caching.UnitTests.Retry;
 using FluentAssertions;
 using Xunit;
 
@@ -85,7 +86,7 @@ public void WhenItemIsUpdatedItIsUpdated()
             value.Should().Be(2);
         }
 
-        [Fact]
+        [RetryFact]
         public void WhenItemDoesNotExistUpdatedAddsItem()
         {
             lfu.AddOrUpdate(1, 2);
diff --git a/BitFaster.Caching.UnitTests/Lfu/ConcurrentTLfuTests.cs b/BitFaster.Caching.UnitTests/Lfu/ConcurrentTLfuTests.cs
@@ -112,7 +112,7 @@ public void WhenKeyTypeMismatchTryGetTimeToExpireReturnsFalse()
 
         // policy can expire after write
 
-        [Fact]
+        [RetryFact]
         public void WhenItemIsNotExpiredItIsNotRemoved()
         {
             lfu.GetOrAdd(1, valueFactory.Create);
diff --git a/BitFaster.Caching.UnitTests/Lru/ConcurrentLruAfterAccessTests.cs b/BitFaster.Caching.UnitTests/Lru/ConcurrentLruAfterAccessTests.cs
@@ -46,7 +46,7 @@ public void TimeToLiveIsCtorArg()
             this.lru.Policy.ExpireAfterAccess.Value.TimeToLive.Should().Be(timeToLive);
         }
 
-        [Fact]
+        [RetryFact]
         public void WhenItemIsNotExpiredItIsNotRemoved()
         {
             lru.GetOrAdd(1, valueFactory.Create);
diff --git a/BitFaster.Caching.UnitTests/Lru/ConcurrentTLruTests.cs b/BitFaster.Caching.UnitTests/Lru/ConcurrentTLruTests.cs
@@ -48,7 +48,7 @@ public void TimeToLiveIsCtorArg()
             this.lru.Policy.ExpireAfterWrite.Value.TimeToLive.Should().Be(timeToLive);
         }
 
-        [Fact]
+        [RetryFact]
         public void WhenItemIsNotExpiredItIsNotRemoved()
         {
             lru.GetOrAdd(1, valueFactory.Create);

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`	`using System.Collections.Generic;`
`4`	`4`	`using System.Threading.Tasks;`
`5`	`5`	`using BitFaster.Caching.Lfu;`
	`6`	`+using BitFaster.Caching.UnitTests.Retry;`
`6`	`7`	`using FluentAssertions;`
`7`	`8`	`using Xunit;`
`8`	`9`
`@@ -85,7 +86,7 @@ public void WhenItemIsUpdatedItIsUpdated()`
`85`	`86`	`value.Should().Be(2);`
`86`	`87`	`}`
`87`	`88`
`88`		`- [Fact]`
	`89`	`+ [RetryFact]`
`89`	`90`	`public void WhenItemDoesNotExistUpdatedAddsItem()`
`90`	`91`	`{`
`91`	`92`	`lfu.AddOrUpdate(1, 2);`
Original file line number	Diff line number	Diff line change
`@@ -112,7 +112,7 @@ public void WhenKeyTypeMismatchTryGetTimeToExpireReturnsFalse()`
`112`	`112`
`113`	`113`	`// policy can expire after write`
`114`	`114`
`115`		`- [Fact]`
	`115`	`+ [RetryFact]`
`116`	`116`	`public void WhenItemIsNotExpiredItIsNotRemoved()`
`117`	`117`	`{`
`118`	`118`	`lfu.GetOrAdd(1, valueFactory.Create);`
Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ public void TimeToLiveIsCtorArg()`
`46`	`46`	`this.lru.Policy.ExpireAfterAccess.Value.TimeToLive.Should().Be(timeToLive);`
`47`	`47`	`}`
`48`	`48`
`49`		`- [Fact]`
	`49`	`+ [RetryFact]`
`50`	`50`	`public void WhenItemIsNotExpiredItIsNotRemoved()`
`51`	`51`	`{`
`52`	`52`	`lru.GetOrAdd(1, valueFactory.Create);`
Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ public void TimeToLiveIsCtorArg()`
`48`	`48`	`this.lru.Policy.ExpireAfterWrite.Value.TimeToLive.Should().Be(timeToLive);`
`49`	`49`	`}`
`50`	`50`
`51`		`- [Fact]`
	`51`	`+ [RetryFact]`
`52`	`52`	`public void WhenItemIsNotExpiredItIsNotRemoved()`
`53`	`53`	`{`
`54`	`54`	`lru.GetOrAdd(1, valueFactory.Create);`