test naming

Alex Peck · Alex Peck · commit cc1e338e91b3 · 2023-09-11T18:25:42.000-07:00
diff --git a/BitFaster.Caching.UnitTests/Lfu/ConcurrentLfuTests.cs b/BitFaster.Caching.UnitTests/Lfu/ConcurrentLfuTests.cs
@@ -582,12 +582,12 @@ public void WhenItemDoesNotExistUpdatedAddsItem()
         }
 
         [Fact]
-        public void WhenKeyIsRemovedItIsRemoved()
+        public void WhenKeyExistsTryRemoveRemovesItem()
         {
             cache.GetOrAdd(1, k => k);
 
             cache.TryRemove(1).Should().BeTrue();
-            cache.TryGet(1, out var value).Should().BeFalse();
+            cache.TryGet(1, out _).Should().BeFalse();
         }
 
         [Fact]
@@ -600,16 +600,16 @@ public void WhenKeyExistsTryRemoveReturnsValue()
         }
 
         [Fact]
-        public void WhenItemIsRemovedItIsRemoved()
+        public void WhenItemExistsTryRemoveRemovesItem()
         {
             cache.GetOrAdd(1, k => k);
 
             cache.TryRemove(new KeyValuePair<int, int>(1, 1)).Should().BeTrue();
-            cache.TryGet(1, out var value).Should().BeFalse();
+            cache.TryGet(1, out _).Should().BeFalse();
         }
 
         [Fact]
-        public void WhenItemDoesntMatchItIsNotRemoved()
+        public void WhenItemDoesntMatchTryRemoveDoesNotRemove()
         {
             cache.GetOrAdd(1, k => k);
 

Original file line number	Diff line number	Diff line change
`@@ -582,12 +582,12 @@ public void WhenItemDoesNotExistUpdatedAddsItem()`
`582`	`582`	`}`
`583`	`583`
`584`	`584`	`[Fact]`
`585`		`- public void WhenKeyIsRemovedItIsRemoved()`
	`585`	`+ public void WhenKeyExistsTryRemoveRemovesItem()`
`586`	`586`	`{`
`587`	`587`	`cache.GetOrAdd(1, k => k);`
`588`	`588`
`589`	`589`	`cache.TryRemove(1).Should().BeTrue();`
`590`		`- cache.TryGet(1, out var value).Should().BeFalse();`
	`590`	`+ cache.TryGet(1, out _).Should().BeFalse();`
`591`	`591`	`}`
`592`	`592`
`593`	`593`	`[Fact]`
`@@ -600,16 +600,16 @@ public void WhenKeyExistsTryRemoveReturnsValue()`
`600`	`600`	`}`
`601`	`601`
`602`	`602`	`[Fact]`
`603`		`- public void WhenItemIsRemovedItIsRemoved()`
	`603`	`+ public void WhenItemExistsTryRemoveRemovesItem()`
`604`	`604`	`{`
`605`	`605`	`cache.GetOrAdd(1, k => k);`
`606`	`606`
`607`	`607`	`cache.TryRemove(new KeyValuePair<int, int>(1, 1)).Should().BeTrue();`
`608`		`- cache.TryGet(1, out var value).Should().BeFalse();`
	`608`	`+ cache.TryGet(1, out _).Should().BeFalse();`
`609`	`609`	`}`
`610`	`610`
`611`	`611`	`[Fact]`
`612`		`- public void WhenItemDoesntMatchItIsNotRemoved()`
	`612`	`+ public void WhenItemDoesntMatchTryRemoveDoesNotRemove()`
`613`	`613`	`{`
`614`	`614`	`cache.GetOrAdd(1, k => k);`
`615`	`615`