Added explicit cache tests and remove unused logic in delete

Author: linkdotnet

LinkDotNet.Blog.Infrastructure/Persistence/CachedRepository.cs

@@ -64,7 +64,6 @@ public async ValueTask StoreAsync(T entity)
     public async ValueTask DeleteAsync(string id)
     {
         ResetCache();
-        memoryCache.Remove(id);
         await repository.DeleteAsync(id);
     }
 

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

@@ -30,4 +30,36 @@ public async Task ShouldNotCacheWhenDifferentQueries()
         allWithTag2.Count.Should().Be(1);
         allWithTag2.Single().Tags.Single().Content.Should().Be("tag 2");
     }
+
+    [Fact]
+    public async Task ShouldResetOnDelete()
+    {
+        var bp1 = new BlogPostBuilder().WithTitle("1").Build();
+        var bp2 = new BlogPostBuilder().WithTitle("2").Build();
+        var sut = new CachedRepository<BlogPost>(Repository, new MemoryCache(new MemoryCacheOptions()));
+        await sut.StoreAsync(bp1);
+        await sut.StoreAsync(bp2);
+        await sut.GetAllAsync();
+        await sut.DeleteAsync(bp1.Id);
+
+        var db = await sut.GetAllAsync();
+
+        db.Single().Title.Should().Be("2");
+    }
+
+    [Fact]
+    public async Task ShouldResetOnSave()
+    {
+        var bp1 = new BlogPostBuilder().WithTitle("1").Build();
+        var bp2 = new BlogPostBuilder().WithTitle("2").Build();
+        var sut = new CachedRepository<BlogPost>(Repository, new MemoryCache(new MemoryCacheOptions()));
+        await sut.StoreAsync(bp1);
+        await sut.GetAllAsync();
+        bp1.Update(bp2);
+        await sut.StoreAsync(bp1);
+
+        var db = await sut.GetAllAsync();
+
+        db.Single().Title.Should().Be("2");
+    }
 }

LinkDotNet.Blog.UnitTests/Web/Pages/Admin/SitemapTests.cs

@@ -40,9 +40,9 @@ public void ShouldDisplaySitemap()
         var sitemap = new SitemapUrlSet
         {
             Urls = new List<SitemapUrl>
-                {
-                    new() { Location = "loc", LastModified = "Now" },
-                },
+            {
+                new() { Location = "loc", LastModified = "Now" },
+            },
         };
         sitemapMock.Setup(s => s.CreateSitemapAsync())
             .ReturnsAsync(sitemap);
@@ -65,9 +65,9 @@ public void ShouldShowLoadingWhenGenerating()
         var sitemap = new SitemapUrlSet
         {
             Urls = new List<SitemapUrl>
-                {
-                    new() { Location = "loc", LastModified = "Now" },
-                },
+            {
+                new() { Location = "loc", LastModified = "Now" },
+            },
         };
         sitemapMock.Setup(s => s.CreateSitemapAsync())
             .Returns(async () =>