Apply formatting/analyzers

calledude · calledude · commit 66d9d5584a61 · 2024-11-13T18:02:38.000+01:00
diff --git a/AsyncMemoryCache.Tests/AsyncMemoryCacheTests.cs b/AsyncMemoryCache.Tests/AsyncMemoryCacheTests.cs
@@ -87,7 +87,7 @@ public void EvictionBehaviorIsStarted()
 		var evictionBehavior = Substitute.For<IEvictionBehavior>();
 		var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>
 		{
-			EvictionBehavior = evictionBehavior
+			EvictionBehavior = evictionBehavior,
 		};
 
 		_ = AsyncMemoryCache<string, IAsyncDisposable>.Create(config);
@@ -124,11 +124,11 @@ public async Task DisposeAsync()
 		var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>
 		{
 			EvictionBehavior = evictionBehavior,
-			CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>()
+			CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>
 			{
 				{ "test1", new CacheEntity<string, IAsyncDisposable>("test1", () => Task.FromResult(cacheObject1), AsyncLazyFlags.None) },
-				{ "test2", new CacheEntity<string, IAsyncDisposable>("test2", () => Task.FromResult(cacheObject2), AsyncLazyFlags.None ) }
-			}
+				{ "test2", new CacheEntity<string, IAsyncDisposable>("test2", () => Task.FromResult(cacheObject2), AsyncLazyFlags.None ) },
+			},
 		};
 
 		var target = AsyncMemoryCache<string, IAsyncDisposable>.Create(config);
@@ -150,7 +150,7 @@ public void NewCacheEntityReferenceReturnedOnEveryCall()
 		var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>
 		{
 			EvictionBehavior = evictionBehavior,
-			CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>()
+			CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>(),
 		};
 
 		var target = AsyncMemoryCache<string, IAsyncDisposable>.Create(config);
@@ -180,7 +180,7 @@ public void NewCacheEntityIsCreatedIfUsesIsBelowZero()
 		var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>
 		{
 			EvictionBehavior = evictionBehavior,
-			CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>()
+			CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>(),
 		};
 
 		var target = AsyncMemoryCache<string, IAsyncDisposable>.Create(config);
@@ -207,7 +207,7 @@ public void TryGetValue_ReturnsFalseIfItemDoesNotExist()
 		var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>
 		{
 			EvictionBehavior = evictionBehavior,
-			CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>()
+			CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>(),
 		};
 
 		var target = AsyncMemoryCache<string, IAsyncDisposable>.Create(config);
@@ -227,10 +227,10 @@ public void TryGetValue_ReturnsFalseWhenBeingDisposed()
 		var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>
 		{
 			EvictionBehavior = evictionBehavior,
-			CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>()
+			CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>
 			{
-				{ key, new(key, () => Task.FromResult(Substitute.For<IAsyncDisposable>()), AsyncLazyFlags.None) }
-			}
+				{ key, new(key, () => Task.FromResult(Substitute.For<IAsyncDisposable>()), AsyncLazyFlags.None) },
+			},
 		};
 
 		config.CacheBackingStore[key].References = -1;
@@ -253,10 +253,10 @@ public void TryGetValue_ReturnsTrueIfItemExistsAndIncrementsReferenceCount()
 		var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>
 		{
 			EvictionBehavior = evictionBehavior,
-			CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>()
+			CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>
 			{
-				{ key, new(key, () => Task.FromResult(Substitute.For<IAsyncDisposable>()), AsyncLazyFlags.None) }
-			}
+				{ key, new(key, () => Task.FromResult(Substitute.For<IAsyncDisposable>()), AsyncLazyFlags.None) },
+			},
 		};
 
 		var target = AsyncMemoryCache<string, IAsyncDisposable>.Create(config);
@@ -277,14 +277,14 @@ public void TryGetValue_ExistingItemWithExpirationStrategy_CallsCacheEntityAcces
 
 		var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>
 		{
-			CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>()
+			CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>
 			{
 				{
 					key,
 					new CacheEntity<string, IAsyncDisposable>(key, () => Task.FromResult(Substitute.For<IAsyncDisposable>()), AsyncLazyFlags.None)
 						.WithExpirationStrategy(expirationStrategy)
-				}
-			}
+				},
+			},
 		};
 
 		var target = AsyncMemoryCache<string, IAsyncDisposable>.Create(config);
@@ -303,14 +303,14 @@ public void Indexer_ExistingItemWithExpirationStrategy_CallsCacheEntityAccessed(
 
 		var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>
 		{
-			CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>()
+			CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>
 			{
 				{
 					key,
 					new CacheEntity<string, IAsyncDisposable>(key, () => Task.FromResult(Substitute.For<IAsyncDisposable>()), AsyncLazyFlags.None)
 						.WithExpirationStrategy(expirationStrategy)
-				}
-			}
+				},
+			},
 		};
 
 		var target = AsyncMemoryCache<string, IAsyncDisposable>.Create(config);
@@ -324,7 +324,7 @@ private static AsyncMemoryCacheConfiguration<string, IAsyncDisposable> CreateCon
 	{
 		return new()
 		{
-			EvictionBehavior = EvictionBehavior.Disabled
+			EvictionBehavior = EvictionBehavior.Disabled,
 		};
 	}
 }
diff --git a/AsyncMemoryCache.Tests/CacheEntityReferenceExtensionTests.cs b/AsyncMemoryCache.Tests/CacheEntityReferenceExtensionTests.cs
@@ -54,7 +54,7 @@ public void WithExpirationStrategyExtension()
 	[Fact]
 	public void WithExpirationCallbackExtension()
 	{
-		var expirationCallback = (string key, IAsyncDisposable obj) => { };
+		var expirationCallback = (string _, IAsyncDisposable _) => { };
 
 		var cacheEntity = new CacheEntity<string, IAsyncDisposable>("test", () => Task.FromResult((IAsyncDisposable)null!), AsyncLazyFlags.None);
 		var cacheEntityReference = new CacheEntityReference<string, IAsyncDisposable>(cacheEntity);
diff --git a/AsyncMemoryCache.Tests/CacheEntityReferenceTests.cs b/AsyncMemoryCache.Tests/CacheEntityReferenceTests.cs
@@ -10,7 +10,7 @@ public class CacheEntityReferenceTests
 	[Fact]
 	public void IncrementsAndDecrementsProperly()
 	{
-		var cacheEntity = new CacheEntity<string, IAsyncDisposable>("", () => Task.FromResult((IAsyncDisposable)null!), AsyncLazyFlags.None);
+		var cacheEntity = new CacheEntity<string, IAsyncDisposable>(string.Empty, () => Task.FromResult((IAsyncDisposable)null!), AsyncLazyFlags.None);
 		Assert.Equal(0, cacheEntity.References);
 
 		var cacheEntityReference = new CacheEntityReference<string, IAsyncDisposable>(cacheEntity);
@@ -23,7 +23,7 @@ public void IncrementsAndDecrementsProperly()
 	[Fact]
 	public void MultipleDisposes_DecrementsOnlyOnce()
 	{
-		var cacheEntity = new CacheEntity<string, IAsyncDisposable>("", () => Task.FromResult((IAsyncDisposable)null!), AsyncLazyFlags.None);
+		var cacheEntity = new CacheEntity<string, IAsyncDisposable>(string.Empty, () => Task.FromResult((IAsyncDisposable)null!), AsyncLazyFlags.None);
 		Assert.Equal(0, cacheEntity.References);
 
 		var cacheEntityReference = new CacheEntityReference<string, IAsyncDisposable>(cacheEntity);
@@ -34,7 +34,7 @@ public void MultipleDisposes_DecrementsOnlyOnce()
 		cacheEntityReference.Dispose();
 		cacheEntityReference.Dispose();
 		cacheEntityReference.Dispose();
-#pragma warning enable
+#pragma warning restore
 		Assert.Equal(0, cacheEntity.References);
 	}
 }
diff --git a/AsyncMemoryCache.Tests/EvictionBehaviorTests.cs b/AsyncMemoryCache.Tests/EvictionBehaviorTests.cs
@@ -34,7 +34,7 @@ public void DefaultEvictionBehavior_TicksAccordingToConfig(int? seconds, int exp
 
 		var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>
 		{
-			CacheBackingStore = cache
+			CacheBackingStore = cache,
 		};
 
 		var timeProvider = new FakeTimeProvider();
@@ -113,7 +113,7 @@ private static async Task Internal_DefaultEvictionBehavior_RemovesExpiredItem_Le
 
 		var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>
 		{
-			CacheBackingStore = cache
+			CacheBackingStore = cache,
 		};
 
 		target.Start(config, _logger);
@@ -143,7 +143,7 @@ public async Task UseSystemTimer()
 
 		var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>
 		{
-			CacheBackingStore = cache
+			CacheBackingStore = cache,
 		};
 
 		var target = new DefaultEvictionBehavior(null, TimeSpan.FromMilliseconds(1));
@@ -180,7 +180,7 @@ public async Task ExpiredCacheItem_AlwaysDisposedIfRefCountBelowZero()
 		var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>
 		{
 			CacheItemExpired = null,
-			CacheBackingStore = cache
+			CacheBackingStore = cache,
 		};
 
 		target.Start(config, _logger);
@@ -203,7 +203,7 @@ public async Task ExpiredCacheItem_NeverDisposedIfRefCountAboveZero()
 			{
 				expiredKey, new CacheEntity<string, IAsyncDisposable>(expiredKey, () => Task.FromResult(expiredCacheObject), AsyncLazyFlags.None)
 					.WithAbsoluteExpiration(DateTimeOffset.UtcNow)
-			}
+			},
 		};
 
 		var resetEvent = new ManualResetEvent(false);
@@ -236,7 +236,7 @@ public async Task CacheItemWithoutExpirationStrategy_NeverDisposed()
 			{
 				// Not setting any expiration strategy
 				expiredKey, new CacheEntity<string, IAsyncDisposable>(expiredKey, () => Task.FromResult(expiredCacheObject), AsyncLazyFlags.None)
-			}
+			},
 		};
 
 		var resetEvent = new ManualResetEvent(false);
@@ -263,10 +263,8 @@ public async Task DefaultEvictionBehavior_CallsCallbacks_ForExpiredItem()
 		const string expiredKey = "expired";
 
 		var resetEvent = new ManualResetEvent(false);
-		var globalCacheItemExpiredCallbackCalled = false;
-		var globalCacheItemExpiredCallback = (string key, IAsyncDisposable value) => { globalCacheItemExpiredCallbackCalled = true; };
 		var cacheItemExpirationCallbackCalled = false;
-		var cacheItemExpirationCallback = (string key, IAsyncDisposable value) => 
+		var cacheItemExpirationCallback = (string _, IAsyncDisposable _) =>
 		{
 			cacheItemExpirationCallbackCalled = true;
 			resetEvent.Set();
@@ -288,10 +286,11 @@ public async Task DefaultEvictionBehavior_CallsCallbacks_ForExpiredItem()
 
 		var timeProvider = new FakeTimeProvider(DateTime.UtcNow);
 		var target = new DefaultEvictionBehavior(timeProvider);
+		var globalCacheItemExpiredCallbackCalled = false;
 		var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>
 		{
-			CacheItemExpired = globalCacheItemExpiredCallback,
-			CacheBackingStore = cache
+			CacheItemExpired = (string _, IAsyncDisposable _) => globalCacheItemExpiredCallbackCalled = true,
+			CacheBackingStore = cache,
 		};
 
 		target.Start(config, _logger);
diff --git a/AsyncMemoryCache.Tests/ExpirationStrategyTests.cs b/AsyncMemoryCache.Tests/ExpirationStrategyTests.cs
@@ -22,7 +22,7 @@ public void SlidingExpirationStrategy(int lastUseOffset, bool expectedExpiryStat
 	{
 		var target = new SlidingExpirationStrategy(TimeSpan.FromMinutes(1))
 		{
-			LastUse = DateTimeOffset.UtcNow.AddMinutes(lastUseOffset)
+			LastUse = DateTimeOffset.UtcNow.AddMinutes(lastUseOffset),
 		};
 
 		Assert.Equal(expectedExpiryState, target.IsExpired());
@@ -34,7 +34,7 @@ public void SlidingExpirationStrategy_CacheEntityAccessed_SetsLastUse()
 		var lastUse = DateTimeOffset.MinValue;
 		var target = new SlidingExpirationStrategy(TimeSpan.Zero)
 		{
-			LastUse = lastUse
+			LastUse = lastUse,
 		};
 
 		target.CacheEntityAccessed();
diff --git a/AsyncMemoryCache/AsyncMemoryCache.cs b/AsyncMemoryCache/AsyncMemoryCache.cs
@@ -1,5 +1,4 @@
-﻿using AsyncMemoryCache.EvictionBehaviors;
-using Microsoft.Extensions.Logging;
+﻿using Microsoft.Extensions.Logging;
 using Microsoft.Extensions.Logging.Abstractions;
 using Nito.AsyncEx;
 using System;
@@ -91,6 +90,7 @@ private AsyncMemoryCache(IAsyncMemoryCacheConfiguration<TKey, TValue> configurat
 	/// </summary>
 	/// <param name="configuration">The <see cref="IAsyncMemoryCacheConfiguration{TKey, TValue}"/>.</param>
 	/// <param name="logger">The optional <see cref="ILogger{CategoryName}">ILogger</see>&lt;<see cref="AsyncMemoryCache{TKey, TValue}"></see>&gt;</param>
+	/// <exception cref="ArgumentNullException">When <paramref name="configuration"/> is null</exception>
 	public static AsyncMemoryCache<TKey, TValue> Create(IAsyncMemoryCacheConfiguration<TKey, TValue> configuration, ILogger<AsyncMemoryCache<TKey, TValue>>? logger = null)
 	{
 		var asyncMemoryCache = new AsyncMemoryCache<TKey, TValue>(configuration, logger);
@@ -162,7 +162,7 @@ public async ValueTask DisposeAsync()
 		var diposeTasks = _configuration.CacheBackingStore
 			.Select(async x =>
 			{
-				var cachedObject = await x.Value;
+				var cachedObject = await x.Value.ConfigureAwait(false);
 				await cachedObject.DisposeAsync().ConfigureAwait(false);
 			});
 
diff --git a/AsyncMemoryCache/EvictionBehaviors/DefaultEvictionBehavior.cs b/AsyncMemoryCache/EvictionBehaviors/DefaultEvictionBehavior.cs
@@ -103,7 +103,7 @@ private static async Task CheckExpiredItems<TKey, TValue>(IAsyncMemoryCacheConfi
 		foreach (var expiredItem in expiredItems)
 		{
 			logger.LogTrace("Expiring item with key {Key}", expiredItem.Key);
-			var item = await expiredItem;
+			var item = await expiredItem.ConfigureAwait(false);
 			if (cache.Remove(expiredItem.Key))
 			{
 				configuration.CacheItemExpired?.Invoke(expiredItem.Key, item);

Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,7 @@ public void EvictionBehaviorIsStarted()`
`87`	`87`	`var evictionBehavior = Substitute.For<IEvictionBehavior>();`
`88`	`88`	`var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>`
`89`	`89`	`{`
`90`		`- EvictionBehavior = evictionBehavior`
	`90`	`+ EvictionBehavior = evictionBehavior,`
`91`	`91`	`};`
`92`	`92`
`93`	`93`	`_ = AsyncMemoryCache<string, IAsyncDisposable>.Create(config);`
`@@ -124,11 +124,11 @@ public async Task DisposeAsync()`
`124`	`124`	`var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>`
`125`	`125`	`{`
`126`	`126`	`EvictionBehavior = evictionBehavior,`
`127`		`- CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>()`
	`127`	`+ CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>`
`128`	`128`	`{`
`129`	`129`	`{ "test1", new CacheEntity<string, IAsyncDisposable>("test1", () => Task.FromResult(cacheObject1), AsyncLazyFlags.None) },`
`130`		`- { "test2", new CacheEntity<string, IAsyncDisposable>("test2", () => Task.FromResult(cacheObject2), AsyncLazyFlags.None ) }`
`131`		`- }`
	`130`	`+ { "test2", new CacheEntity<string, IAsyncDisposable>("test2", () => Task.FromResult(cacheObject2), AsyncLazyFlags.None ) },`
	`131`	`+ },`
`132`	`132`	`};`
`133`	`133`
`134`	`134`	`var target = AsyncMemoryCache<string, IAsyncDisposable>.Create(config);`
`@@ -150,7 +150,7 @@ public void NewCacheEntityReferenceReturnedOnEveryCall()`
`150`	`150`	`var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>`
`151`	`151`	`{`
`152`	`152`	`EvictionBehavior = evictionBehavior,`
`153`		`- CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>()`
	`153`	`+ CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>(),`
`154`	`154`	`};`
`155`	`155`
`156`	`156`	`var target = AsyncMemoryCache<string, IAsyncDisposable>.Create(config);`
`@@ -180,7 +180,7 @@ public void NewCacheEntityIsCreatedIfUsesIsBelowZero()`
`180`	`180`	`var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>`
`181`	`181`	`{`
`182`	`182`	`EvictionBehavior = evictionBehavior,`
`183`		`- CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>()`
	`183`	`+ CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>(),`
`184`	`184`	`};`
`185`	`185`
`186`	`186`	`var target = AsyncMemoryCache<string, IAsyncDisposable>.Create(config);`
`@@ -207,7 +207,7 @@ public void TryGetValue_ReturnsFalseIfItemDoesNotExist()`
`207`	`207`	`var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>`
`208`	`208`	`{`
`209`	`209`	`EvictionBehavior = evictionBehavior,`
`210`		`- CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>()`
	`210`	`+ CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>(),`
`211`	`211`	`};`
`212`	`212`
`213`	`213`	`var target = AsyncMemoryCache<string, IAsyncDisposable>.Create(config);`
`@@ -227,10 +227,10 @@ public void TryGetValue_ReturnsFalseWhenBeingDisposed()`
`227`	`227`	`var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>`
`228`	`228`	`{`
`229`	`229`	`EvictionBehavior = evictionBehavior,`
`230`		`- CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>()`
	`230`	`+ CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>`
`231`	`231`	`{`
`232`		`- { key, new(key, () => Task.FromResult(Substitute.For<IAsyncDisposable>()), AsyncLazyFlags.None) }`
`233`		`- }`
	`232`	`+ { key, new(key, () => Task.FromResult(Substitute.For<IAsyncDisposable>()), AsyncLazyFlags.None) },`
	`233`	`+ },`
`234`	`234`	`};`
`235`	`235`
`236`	`236`	`config.CacheBackingStore[key].References = -1;`
`@@ -253,10 +253,10 @@ public void TryGetValue_ReturnsTrueIfItemExistsAndIncrementsReferenceCount()`
`253`	`253`	`var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>`
`254`	`254`	`{`
`255`	`255`	`EvictionBehavior = evictionBehavior,`
`256`		`- CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>()`
	`256`	`+ CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>`
`257`	`257`	`{`
`258`		`- { key, new(key, () => Task.FromResult(Substitute.For<IAsyncDisposable>()), AsyncLazyFlags.None) }`
`259`		`- }`
	`258`	`+ { key, new(key, () => Task.FromResult(Substitute.For<IAsyncDisposable>()), AsyncLazyFlags.None) },`
	`259`	`+ },`
`260`	`260`	`};`
`261`	`261`
`262`	`262`	`var target = AsyncMemoryCache<string, IAsyncDisposable>.Create(config);`
`@@ -277,14 +277,14 @@ public void TryGetValue_ExistingItemWithExpirationStrategy_CallsCacheEntityAcces`
`277`	`277`
`278`	`278`	`var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>`
`279`	`279`	`{`
`280`		`- CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>()`
	`280`	`+ CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>`
`281`	`281`	`{`
`282`	`282`	`{`
`283`	`283`	`key,`
`284`	`284`	`new CacheEntity<string, IAsyncDisposable>(key, () => Task.FromResult(Substitute.For<IAsyncDisposable>()), AsyncLazyFlags.None)`
`285`	`285`	`.WithExpirationStrategy(expirationStrategy)`
`286`		`- }`
`287`		`- }`
	`286`	`+ },`
	`287`	`+ },`
`288`	`288`	`};`
`289`	`289`
`290`	`290`	`var target = AsyncMemoryCache<string, IAsyncDisposable>.Create(config);`
`@@ -303,14 +303,14 @@ public void Indexer_ExistingItemWithExpirationStrategy_CallsCacheEntityAccessed(`
`303`	`303`
`304`	`304`	`var config = new AsyncMemoryCacheConfiguration<string, IAsyncDisposable>`
`305`	`305`	`{`
`306`		`- CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>()`
	`306`	`+ CacheBackingStore = new Dictionary<string, CacheEntity<string, IAsyncDisposable>>`
`307`	`307`	`{`
`308`	`308`	`{`
`309`	`309`	`key,`
`310`	`310`	`new CacheEntity<string, IAsyncDisposable>(key, () => Task.FromResult(Substitute.For<IAsyncDisposable>()), AsyncLazyFlags.None)`
`311`	`311`	`.WithExpirationStrategy(expirationStrategy)`
`312`		`- }`
`313`		`- }`
	`312`	`+ },`
	`313`	`+ },`
`314`	`314`	`};`
`315`	`315`
`316`	`316`	`var target = AsyncMemoryCache<string, IAsyncDisposable>.Create(config);`
`@@ -324,7 +324,7 @@ private static AsyncMemoryCacheConfiguration<string, IAsyncDisposable> CreateCon`
`324`	`324`	`{`
`325`	`325`	`return new()`
`326`	`326`	`{`
`327`		`- EvictionBehavior = EvictionBehavior.Disabled`
	`327`	`+ EvictionBehavior = EvictionBehavior.Disabled,`
`328`	`328`	`};`
`329`	`329`	`}`
`330`	`330`	`}`
Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ public void WithExpirationStrategyExtension()`
`54`	`54`	`[Fact]`
`55`	`55`	`public void WithExpirationCallbackExtension()`
`56`	`56`	`{`
`57`		`- var expirationCallback = (string key, IAsyncDisposable obj) => { };`
	`57`	`+ var expirationCallback = (string _, IAsyncDisposable _) => { };`
`58`	`58`
`59`	`59`	`var cacheEntity = new CacheEntity<string, IAsyncDisposable>("test", () => Task.FromResult((IAsyncDisposable)null!), AsyncLazyFlags.None);`
`60`	`60`	`var cacheEntityReference = new CacheEntityReference<string, IAsyncDisposable>(cacheEntity);`
Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ public class CacheEntityReferenceTests`
`10`	`10`	`[Fact]`
`11`	`11`	`public void IncrementsAndDecrementsProperly()`
`12`	`12`	`{`
`13`		`- var cacheEntity = new CacheEntity<string, IAsyncDisposable>("", () => Task.FromResult((IAsyncDisposable)null!), AsyncLazyFlags.None);`
	`13`	`+ var cacheEntity = new CacheEntity<string, IAsyncDisposable>(string.Empty, () => Task.FromResult((IAsyncDisposable)null!), AsyncLazyFlags.None);`
`14`	`14`	`Assert.Equal(0, cacheEntity.References);`
`15`	`15`
`16`	`16`	`var cacheEntityReference = new CacheEntityReference<string, IAsyncDisposable>(cacheEntity);`
`@@ -23,7 +23,7 @@ public void IncrementsAndDecrementsProperly()`
`23`	`23`	`[Fact]`
`24`	`24`	`public void MultipleDisposes_DecrementsOnlyOnce()`
`25`	`25`	`{`
`26`		`- var cacheEntity = new CacheEntity<string, IAsyncDisposable>("", () => Task.FromResult((IAsyncDisposable)null!), AsyncLazyFlags.None);`
	`26`	`+ var cacheEntity = new CacheEntity<string, IAsyncDisposable>(string.Empty, () => Task.FromResult((IAsyncDisposable)null!), AsyncLazyFlags.None);`
`27`	`27`	`Assert.Equal(0, cacheEntity.References);`
`28`	`28`
`29`	`29`	`var cacheEntityReference = new CacheEntityReference<string, IAsyncDisposable>(cacheEntity);`
`@@ -34,7 +34,7 @@ public void MultipleDisposes_DecrementsOnlyOnce()`
`34`	`34`	`cacheEntityReference.Dispose();`
`35`	`35`	`cacheEntityReference.Dispose();`
`36`	`36`	`cacheEntityReference.Dispose();`
`37`		`-#pragma warning enable`
	`37`	`+#pragma warning restore`
`38`	`38`	`Assert.Equal(0, cacheEntity.References);`
`39`	`39`	`}`
`40`	`40`	`}`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ public void SlidingExpirationStrategy(int lastUseOffset, bool expectedExpiryStat`
`22`	`22`	`{`
`23`	`23`	`var target = new SlidingExpirationStrategy(TimeSpan.FromMinutes(1))`
`24`	`24`	`{`
`25`		`- LastUse = DateTimeOffset.UtcNow.AddMinutes(lastUseOffset)`
	`25`	`+ LastUse = DateTimeOffset.UtcNow.AddMinutes(lastUseOffset),`
`26`	`26`	`};`
`27`	`27`
`28`	`28`	`Assert.Equal(expectedExpiryState, target.IsExpired());`
`@@ -34,7 +34,7 @@ public void SlidingExpirationStrategy_CacheEntityAccessed_SetsLastUse()`
`34`	`34`	`var lastUse = DateTimeOffset.MinValue;`
`35`	`35`	`var target = new SlidingExpirationStrategy(TimeSpan.Zero)`
`36`	`36`	`{`
`37`		`- LastUse = lastUse`
	`37`	`+ LastUse = lastUse,`
`38`	`38`	`};`
`39`	`39`
`40`	`40`	`target.CacheEntityAccessed();`
Original file line number	Diff line number	Diff line change
`@@ -103,7 +103,7 @@ private static async Task CheckExpiredItems<TKey, TValue>(IAsyncMemoryCacheConfi`
`103`	`103`	`foreach (var expiredItem in expiredItems)`
`104`	`104`	`{`
`105`	`105`	`logger.LogTrace("Expiring item with key {Key}", expiredItem.Key);`
`106`		`- var item = await expiredItem;`
	`106`	`+ var item = await expiredItem.ConfigureAwait(false);`
`107`	`107`	`if (cache.Remove(expiredItem.Key))`
`108`	`108`	`{`
`109`	`109`	`configuration.CacheItemExpired?.Invoke(expiredItem.Key, item);`