refactor: Moved reading time into BlogPost

Author: linkdotnet

src/LinkDotNet.Blog.Domain/BlogPost.cs

@@ -38,6 +38,8 @@ private BlogPost()
 
     public string TagsAsString => Tags is null ? string.Empty : string.Join(", ", Tags);
 
+    public int ReadingTimeInMinutes { get; private set; }
+
     public string Slug => GenerateSlug();
 
     private string GenerateSlug()
@@ -56,13 +58,13 @@ private string GenerateSlug()
 
         var slug = Title.ToLower(CultureInfo.CurrentCulture);
 
-        // Remove all special characters from the string.  
+        // Remove all special characters from the string.
         slug = MatchIfSpecialCharactersExist().Replace(slug, "");
 
-        // Remove all additional spaces in favour of just one.  
+        // Remove all additional spaces in favour of just one.
         slug= MatchIfAdditionalSpacesExist().Replace(slug," ").Trim();
 
-        // Replace all spaces with the hyphen.  
+        // Replace all spaces with the hyphen.
         slug= MatchIfSpaceExist().Replace(slug, "-");
 
         return slug;
@@ -115,6 +117,7 @@ public static BlogPost Create(
             PreviewImageUrlFallback = previewImageUrlFallback,
             IsPublished = isPublished,
             Tags = tags?.Select(t => t.Trim()).ToImmutableArray(),
+            ReadingTimeInMinutes = ReadingTimeCalculator.CalculateReadingTime(content),
         };
 
         return blogPost;
@@ -144,5 +147,6 @@ public void Update(BlogPost from)
         PreviewImageUrlFallback = from.PreviewImageUrlFallback;
         IsPublished = from.IsPublished;
         Tags = from.Tags;
+        ReadingTimeInMinutes = from.ReadingTimeInMinutes;
     }
 }

src/LinkDotNet.Blog.Web/Features/Services/ReadingTimeCalculator.cs renamed to src/LinkDotNet.Blog.Domain/ReadingTimeCalculator.cs

@@ -1,7 +1,7 @@
 using System;
 using System.Text.RegularExpressions;
 
-namespace LinkDotNet.Blog.Web.Features.Services;
+namespace LinkDotNet.Blog.Domain;
 
 public static partial class ReadingTimeCalculator
 {

src/LinkDotNet.Blog.Web/Features/Components/ShortBlogPost.razor

@@ -5,7 +5,6 @@
 @using LinkDotNet.Blog.Web.Features.Services
 @using Microsoft.Extensions.Caching.Memory
 
-@inject IMemoryCache MemoryCache
 <article>
 	<div class="blog-card @AltCssClass">
 		<div class="meta">
@@ -24,7 +23,7 @@
 					<li class="draft">Draft</li>
 				}
 				<li class="date me-4"><span>@BlogPost.UpdatedDate.ToString("dd/MM/yyyy")</span></li>
-				@if (BlogPost.Tags != null && BlogPost.Tags.Any())
+				@if (BlogPost.Tags.Any())
 				{
 					<li class="tags me-4">
 						<ul>
@@ -35,7 +34,7 @@
 						</ul>
 					</li>
 				}
-				<li class="read-time me-4">@readingTime min</li>
+				<li class="read-time me-4">@BlogPost.ReadingTimeInMinutes min</li>
 			</ul>
 		</div>
 		<div class="description">
@@ -50,8 +49,6 @@
 </article>
 
 @code {
-	private int readingTime;
-
 	[Parameter]
 	public BlogPost BlogPost { get; set; }
 
@@ -83,14 +80,4 @@
 
 		return base.SetParametersAsync(ParameterView.Empty);
 	}
-
-	protected override void OnInitialized()
-	{
-		var key = "reading-time-" + BlogPost.Id;
-		readingTime = MemoryCache.GetOrCreate(key, entry =>
-		{
-			entry.AbsoluteExpirationRelativeToNow = TimeSpan.FromHours(1);
-			return ReadingTimeCalculator.CalculateReadingTime(BlogPost.Content);
-		});
-	}
 }

tests/LinkDotNet.Blog.UnitTests/Domain/BlogPostTests.cs

@@ -12,19 +12,20 @@ public void ShouldUpdateBlogPost()
     {
         var blogPostToUpdate = new BlogPostBuilder().Build();
         blogPostToUpdate.Id = "random-id";
-        var blogPost = BlogPost.Create("Title", "Desc", "Content", "Url", true, previewImageUrlFallback: "Url2");
+        var blogPost = BlogPost.Create("Title", "Desc", "Other Content", "Url", true, previewImageUrlFallback: "Url2");
         blogPost.Id = "something else";
 
         blogPostToUpdate.Update(blogPost);
 
         blogPostToUpdate.Title.Should().Be("Title");
         blogPostToUpdate.ShortDescription.Should().Be("Desc");
-        blogPostToUpdate.Content.Should().Be("Content");
+        blogPostToUpdate.Content.Should().Be("Other Content");
         blogPostToUpdate.PreviewImageUrl.Should().Be("Url");
         blogPostToUpdate.PreviewImageUrlFallback.Should().Be("Url2");
         blogPostToUpdate.IsPublished.Should().BeTrue();
         blogPostToUpdate.Tags.Should().BeNullOrEmpty();
         blogPostToUpdate.Slug.Should().NotBeNull();
+        blogPostToUpdate.ReadingTimeInMinutes.Should().Be(1);
     }
 
     [Theory]

tests/LinkDotNet.Blog.UnitTests/Web/Features/Services/ReadingTimeCalculatorTests.cs renamed to tests/LinkDotNet.Blog.UnitTests/Domain/ReadingTimeCalculatorTests.cs

@@ -1,7 +1,7 @@
 using System.Linq;
-using LinkDotNet.Blog.Web.Features.Services;
+using LinkDotNet.Blog.Domain;
 
-namespace LinkDotNet.Blog.UnitTests.Web.Features.Services;
+namespace LinkDotNet.Blog.UnitTests.Domain;
 
 public class ReadingTimeCalculatorTests
 {