#146 Switched BankAccountInfo to be a record

SimonGeering · Nov 24, 2020 · 75d5646 · 75d5646
1 parent b5b26e5
commit 75d5646
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 16 deletions.
diff --git a/src/AdminAssistant/DomainModel/Modules/AccountsModule/BankAccountInfo.cs b/src/AdminAssistant/DomainModel/Modules/AccountsModule/BankAccountInfo.cs
@@ -2,14 +2,14 @@
 
 namespace AdminAssistant.DomainModel.Modules.AccountsModule
 {
-    public class BankAccountInfo : IDatabasePersistable
+    public record BankAccountInfo : IDatabasePersistable
     {
-        public int BankAccountID { get; set; }
-        public string AccountName { get; set; } = string.Empty;
-        public int CurrentBalance { get; set; }
-        public string Symbol { get; set; } = string.Empty;
-        public string DecimalFormat { get; set; } = string.Empty;
-        public bool IsBudgeted { get; set; }
+        public int BankAccountID { get; init; }
+        public string AccountName { get; init; } = string.Empty;
+        public int CurrentBalance { get; init; }
+        public string Symbol { get; init; } = string.Empty;
+        public string DecimalFormat { get; init; } = string.Empty;
+        public bool IsBudgeted { get; init; }
 
         public int PrimaryKey => BankAccountID;
     }

diff --git a/src/AdminAssistant/DomainModel/Modules/AccountsModule/Builders/BankAccountInfoBuilder.cs b/src/AdminAssistant/DomainModel/Modules/AccountsModule/Builders/BankAccountInfoBuilder.cs
@@ -5,21 +5,25 @@ public interface IBankAccountInfoBuilder
         BankAccountInfo Build();
         IBankAccountInfoBuilder WithTestData(int bankAccountInfoID = Constants.UnknownRecordID);
     }
-    internal class BankAccountInfoBuilder : BankAccountInfo, IBankAccountInfoBuilder
+    internal class BankAccountInfoBuilder : IBankAccountInfoBuilder
     {
+        private BankAccountInfo _bankAccountInfo = new BankAccountInfo();
+
         public static BankAccountInfo Default(IBankAccountInfoBuilder builder) => builder.Build();
 
-        public BankAccountInfo Build() => this;
+        public BankAccountInfo Build() => _bankAccountInfo;
 
         public IBankAccountInfoBuilder WithTestData(int bankAccountID = 0)
         {
-            BankAccountID = bankAccountID;
-            AccountName = "A valid account name";
-            CurrentBalance = 0;
-            Symbol = "GBP";
-            DecimalFormat = "2.2-2";
-            IsBudgeted = false;
-
+            _bankAccountInfo = _bankAccountInfo with 
+            {
+                BankAccountID = bankAccountID,
+                AccountName = "A valid account name",
+                CurrentBalance = 0,
+                Symbol = "GBP",
+                DecimalFormat = "2.2-2",
+                IsBudgeted = false
+            };
             return this;
         }
     }