Fixed static cache reset

Author: linkdotnet

LinkDotNet.Blog.Infrastructure/Persistence/CachedRepository.cs

@@ -9,22 +9,20 @@
 
 namespace LinkDotNet.Blog.Infrastructure.Persistence;
 
-public class CachedRepository<T> : IRepository<T>
+public sealed class CachedRepository<T> : IRepository<T>, IDisposable
     where T : Entity
 {
-    private static CancellationTokenSource resetToken = new();
-
     private readonly IRepository<T> repository;
-
     private readonly IMemoryCache memoryCache;
+    private CancellationTokenSource resetToken = new();
 
     public CachedRepository(IRepository<T> repository, IMemoryCache memoryCache)
     {
         this.repository = repository;
         this.memoryCache = memoryCache;
     }
 
-    private static MemoryCacheEntryOptions Options => new()
+    private MemoryCacheEntryOptions Options => new()
     {
         ExpirationTokens = { new CancellationChangeToken(resetToken.Token) },
         AbsoluteExpirationRelativeToNow = TimeSpan.FromDays(7),
@@ -70,7 +68,12 @@ public async Task DeleteAsync(string id)
         await repository.DeleteAsync(id);
     }
 
-    private static void ResetCache()
+    public void Dispose()
+    {
+        resetToken?.Dispose();
+    }
+
+    private void ResetCache()
     {
         if (resetToken is { IsCancellationRequested: false, Token: { CanBeCanceled: true } })
         {
@@ -80,4 +83,4 @@ private static void ResetCache()
 
         resetToken = new CancellationTokenSource();
     }
-}
+}

LinkDotNet.Blog.UnitTests/Infrastructure/Persistence/CachedRepositoryTests.cs

@@ -12,7 +12,7 @@
 
 namespace LinkDotNet.Blog.UnitTests.Infrastructure.Persistence;
 
-public class CachedRepositoryTests
+public sealed class CachedRepositoryTests : IDisposable
 {
     private readonly Mock<IRepository<BlogPost>> repositoryMock;
     private readonly CachedRepository<BlogPost> sut;
@@ -140,6 +140,11 @@ public async Task ShouldGetFreshDataAfterDelete()
             Times.Exactly(2));
     }
 
+    public void Dispose()
+    {
+        sut.Dispose();
+    }
+
     private void SetupRepository()
     {
         var blogPost = new BlogPostBuilder().Build();

stylecop.analyzers.ruleset

@@ -367,7 +367,7 @@
     <Rule Id="S3898" Action="None" />
     <Rule Id="S3900" Action="None" />
     <Rule Id="S3902" Action="None" />
-    <Rule Id="S3903" Action="Info" />
+    <Rule Id="S3903" Action="None" />
     <Rule Id="S3904" Action="Info" />
     <Rule Id="S3906" Action="None" />
     <Rule Id="S3908" Action="None" />