refactor: make record

Author: linkdotnet

src/LinkDotNet.Blog.Domain/Enumeration.cs

@@ -6,34 +6,22 @@
 namespace LinkDotNet.Blog.Domain;
 
 #pragma warning disable CA1724
-public abstract class Enumeration<TEnumeration>
-   where TEnumeration : Enumeration<TEnumeration>
-#pragma warning restore
+public abstract record Enumeration<TEnumeration>(string Key)
+#pragma warning restore CA1724
+    where TEnumeration : Enumeration<TEnumeration>
 {
-    protected Enumeration(string key)
-    {
-        ArgumentException.ThrowIfNullOrWhiteSpace(key);
-        Key = key;
-    }
-
     public static FrozenSet<TEnumeration> All { get; } = GetEnumerations();
 
-    public string Key { get; }
-
-    public static bool operator ==(Enumeration<TEnumeration>? a, Enumeration<TEnumeration>? b)
-        => a is not null && b is not null && a.Key.Equals(b.Key, StringComparison.Ordinal);
+    public static bool operator ==(Enumeration<TEnumeration>? a, string? b)
+        => a is not null && b is not null && a.Key.Equals(b, StringComparison.Ordinal);
 
-    public static bool operator !=(Enumeration<TEnumeration>? a, Enumeration<TEnumeration>? b) => !(a == b);
+    public static bool operator !=(Enumeration<TEnumeration>? a, string? b) => !(a == b);
 
     public static TEnumeration Create(string key)
         => All.SingleOrDefault(p => p.Key == key)
            ?? throw new InvalidOperationException($"{key} is not a valid value for {typeof(TEnumeration).Name}");
 
-    public override int GetHashCode() => Key.GetHashCode(StringComparison.Ordinal);
-
-    public override bool Equals(object? obj) => obj?.GetType() == typeof(TEnumeration) && ((TEnumeration)obj).Key == Key;
-
-    public override string ToString() => Key;
+    public sealed override string ToString() => Key;
 
     private static FrozenSet<TEnumeration> GetEnumerations()
     {
@@ -46,3 +34,4 @@ private static FrozenSet<TEnumeration> GetEnumerations()
             .ToFrozenSet();
     }
 }
+

src/LinkDotNet.Blog.Domain/ProficiencyLevel.cs

@@ -1,6 +1,6 @@
 namespace LinkDotNet.Blog.Domain;
 
-public sealed class ProficiencyLevel : Enumeration<ProficiencyLevel>
+public sealed record ProficiencyLevel : Enumeration<ProficiencyLevel>
 {
     public static readonly ProficiencyLevel Familiar = new(nameof(Familiar));
     public static readonly ProficiencyLevel Proficient = new(nameof(Proficient));

src/LinkDotNet.Blog.Infrastructure/Persistence/PersistenceProvider.cs

@@ -2,7 +2,7 @@
 
 namespace LinkDotNet.Blog.Infrastructure.Persistence;
 
-public sealed class PersistenceProvider : Enumeration<PersistenceProvider>
+public sealed record PersistenceProvider : Enumeration<PersistenceProvider>
 {
     public static readonly PersistenceProvider SqlServer = new(nameof(SqlServer));
     public static readonly PersistenceProvider Sqlite = new(nameof(Sqlite));

src/LinkDotNet.Blog.Web/Features/Services/FileUpload/AuthenticationMode.cs

@@ -2,7 +2,7 @@
 
 namespace LinkDotNet.Blog.Web.Features.Services.FileUpload;
 
-public class AuthenticationMode : Enumeration<AuthenticationMode>
+public sealed record AuthenticationMode : Enumeration<AuthenticationMode>
 {
     public static readonly AuthenticationMode Default = new(nameof(Default));
 

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

@@ -2,7 +2,7 @@
 
 namespace LinkDotNet.Blog.UnitTests.Domain;
 
-public sealed class TestEnumeration : Enumeration<TestEnumeration>
+public sealed record TestEnumeration : Enumeration<TestEnumeration>
 {
     public static readonly TestEnumeration One = new TestEnumeration(nameof(One));