Hardened Skill domain object

Author: linkdotnet

LinkDotNet.Blog.Domain/Skill.cs

@@ -16,13 +16,19 @@ private Skill(string name, string iconUrl, string capability, ProficiencyLevel p
         ProficiencyLevel = proficiencyLevel;
     }
 
-    public string IconUrl { get; set; }
+    public string IconUrl { get; private set; }
 
-    public string Name { get; set; }
+    public string Name { get; private set; }
 
-    public string Capability { get; set; }
+    public string Capability { get; private set; }
 
-    public ProficiencyLevel ProficiencyLevel { get; set; }
+    public ProficiencyLevel ProficiencyLevel { get; private set; }
+
+    public void SetProficiencyLevel(ProficiencyLevel level)
+    {
+        ArgumentNullException.ThrowIfNull(level);
+        ProficiencyLevel = level;
+    }
 
     public static Skill Create(string name, string iconUrl, string capability, string proficiencyLevel)
     {

LinkDotNet.Blog.UnitTests/Domain/SkillTests.cs

@@ -57,4 +57,24 @@ public void ShouldSetUrlToNullWhenWhitespace(string url)
 
         skill.IconUrl.Should().BeNull();
     }
+
+    [Fact]
+    public void ShouldSetProficiencyLevel()
+    {
+        var skill = Skill.Create("name", null, "cap", ProficiencyLevel.Familiar.Key);
+
+        skill.SetProficiencyLevel(ProficiencyLevel.Proficient);
+
+        skill.ProficiencyLevel.Should().Be(ProficiencyLevel.Proficient);
+    }
+
+    [Fact]
+    public void ShouldThrowWhenProficiencyIsNull()
+    {
+        var skill = Skill.Create("name", null, "cap", ProficiencyLevel.Familiar.Key);
+
+        Action result = () => skill.SetProficiencyLevel(null);
+
+        result.Should().Throw<ArgumentNullException>();
+    }
 }

LinkDotNet.Blog.Web/Shared/Skills/SkillTable.razor

@@ -83,7 +83,7 @@
             return;
         }
 
-        currentDragItem.ProficiencyLevel = proficiencyLevel;
+        currentDragItem.SetProficiencyLevel(proficiencyLevel);
         await repository.StoreAsync(currentDragItem);
         currentDragItem = null;
     }