Removed caching from GetAll

Author: linkdotnet

LinkDotNet.Blog.Infrastructure/Persistence/CachedRepository.cs

@@ -46,12 +46,7 @@ public async ValueTask<IPagedList<T>> GetAllAsync(
         int page = 1,
         int pageSize = int.MaxValue)
     {
-        var key = $"{filter?.GetHashCode()}-{orderBy?.GetHashCode()}-{descending}-{page}-{pageSize}";
-        return await memoryCache.GetOrCreate(key, async e =>
-        {
-            e.SetOptions(Options);
-            return await repository.GetAllAsync(filter, orderBy, descending, page, pageSize);
-        });
+        return await repository.GetAllAsync(filter, orderBy, descending, page, pageSize);
     }
 
     public async ValueTask StoreAsync(T entity)

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

@@ -37,33 +37,6 @@ public async Task ShouldGetFromCacheWhenLoaded()
         repositoryMock.Verify(r => r.GetByIdAsync("id"), Times.Once);
     }
 
-    [Fact]
-    public async Task ShouldGetAllFromCacheWhenLoaded()
-    {
-        var blogPost = new BlogPostBuilder().Build();
-        repositoryMock.Setup(r => r.GetAllAsync(
-                It.IsAny<Expression<Func<BlogPost, bool>>>(),
-                It.IsAny<Expression<Func<BlogPost, object>>>(),
-                It.IsAny<bool>(),
-                It.IsAny<int>(),
-                It.IsAny<int>()))
-            .ReturnsAsync(new PagedList<BlogPost>(new[] { blogPost }, 1, 1));
-        var firstCall = await sut.GetAllAsync();
-
-        var secondCall = await sut.GetAllAsync();
-
-        firstCall.Count.Should().Be(1);
-        secondCall.Count.Should().Be(1);
-        repositoryMock.Verify(
-            r => r.GetAllAsync(
-            It.IsAny<Expression<Func<BlogPost, bool>>>(),
-            It.IsAny<Expression<Func<BlogPost, object>>>(),
-            It.IsAny<bool>(),
-            It.IsAny<int>(),
-            It.IsAny<int>()),
-            Times.Once);
-    }
-
     [Fact]
     public async Task ShouldNotCacheWhenParameterDifferent()
     {
@@ -87,7 +60,7 @@ await sut.GetAllAsync(
             2,
             30);
 
-        repositoryMock.Verify(
+            repositoryMock.Verify(
             r => r.GetAllAsync(
                 It.IsAny<Expression<Func<BlogPost, bool>>>(),
                 It.IsAny<Expression<Func<BlogPost, object>>>(),
@@ -156,4 +129,4 @@ private void SetupRepository()
                 It.IsAny<int>()))
             .ReturnsAsync(new PagedList<BlogPost>(new[] { blogPost }, 1, 1));
     }
-}
+}