Don't update blogpost when same reference

linkdotnet · Jan 15, 2022 · 39b7df9 · 39b7df9
1 parent e3a014d
commit 39b7df9
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 2 deletions.
diff --git a/src/LinkDotNet.Blog.Domain/BlogPost.cs b/src/LinkDotNet.Blog.Domain/BlogPost.cs
@@ -51,6 +51,11 @@ private BlogPost()
 
     public void Update(BlogPost from)
     {
+        if (from == this)
+        {
+            return;
+        }
+
         Title = from.Title;
         ShortDescription = from.ShortDescription;
         Content = from.Content;
@@ -75,4 +80,4 @@ private void ReplaceTags(IEnumerable<Tag> tags)
             }
         }
     }
-}
+}
diff --git a/tests/LinkDotNet.Blog.UnitTests/Domain/BlogPostTests.cs b/tests/LinkDotNet.Blog.UnitTests/Domain/BlogPostTests.cs
@@ -59,4 +59,15 @@ public void ShouldSetDateWhenGiven()
 
         blog.UpdatedDate.Should().Be(somewhen);
     }
-}
+
+    [Fact]
+    public void ShouldNotDeleteTagsWhenSameReference()
+    {
+        var bp = new BlogPostBuilder().WithTags("tag 1").Build();
+
+        bp.Update(bp);
+
+        bp.Tags.Should().HaveCount(1);
+        bp.Tags.Single().Content.Should().Be("tag 1");
+    }
+}