#152 Do not set E-Mail to null when not permission

braunau-mobil · Mar 7, 2020 · 1c04054 · 1c04054
1 parent 54eab50
commit 1c04054
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 10 deletions.
diff --git a/src/BraunauMobil.VeloBasar/Models/Seller.cs b/src/BraunauMobil.VeloBasar/Models/Seller.cs
@@ -95,10 +95,6 @@ public void UpdateNewsletterPermissions()
             {
                 NewsletterPermissionTimesStamp = DateTime.Now;
             }
-            else
-            {
-                EMail = null;
-            }
         }
     }
 }
diff --git a/test/BraunauMobil.VeloBasar.Tests/Logic/SellerContextTests/CreateAsync.cs b/test/BraunauMobil.VeloBasar.Tests/Logic/SellerContextTests/CreateAsync.cs
@@ -24,7 +24,6 @@ await RunOnInitializedDb(async () =>
                 Assert.NotNull(createdSeller);
                 Assert.NotEqual(0, createdSeller.Id);
                 Assert.NotNull(createdSeller.Token);
-                Assert.Null(createdSeller.EMail);
             });
         }
 

diff --git a/test/BraunauMobil.VeloBasar.Tests/Logic/SellerContextTests/UpdateAsync.cs b/test/BraunauMobil.VeloBasar.Tests/Logic/SellerContextTests/UpdateAsync.cs
@@ -26,13 +26,11 @@ await RunOnInitializedDb(async () =>
                 Assert.NotNull(createdSeller);
                 Assert.NotEqual(0, createdSeller.Id);
                 Assert.NotNull(createdSeller.Token);
-                Assert.Null(createdSeller.EMail);
                 Assert.Equal(timestamp, createdSeller.NewsletterPermissionTimesStamp);
 
                 await SellerContext.UpdateAsync(createdSeller);
 
                 Assert.False(createdSeller.HasNewsletterPermission);
-                Assert.Null(createdSeller.EMail);
                 Assert.Equal(timestamp, createdSeller.NewsletterPermissionTimesStamp);
             });
         }
@@ -55,7 +53,6 @@ await RunOnInitializedDb(async () =>
                 Assert.NotNull(createdSeller);
                 Assert.NotEqual(0, createdSeller.Id);
                 Assert.NotNull(createdSeller.Token);
-                Assert.Null(createdSeller.EMail);
                 Assert.Equal(timestamp, createdSeller.NewsletterPermissionTimesStamp);
 
                 createdSeller.HasNewsletterPermission = true;
@@ -122,7 +119,6 @@ await RunOnInitializedDb(async () =>
                 sellerToCreate.HasNewsletterPermission = false;
                 await SellerContext.UpdateAsync(createdSeller);
 
-                Assert.Null(createdSeller.EMail);
                 Assert.False(createdSeller.HasNewsletterPermission);
             });
         }

diff --git a/test/BraunauMobil.VeloBasar.Tests/Models/SellerTests/UpdateNewsletterPermissions.cs b/test/BraunauMobil.VeloBasar.Tests/Models/SellerTests/UpdateNewsletterPermissions.cs
@@ -38,7 +38,6 @@ public void HasNoPermission()
 
             Assert.False(seller.HasNewsletterPermission);
             Assert.Equal(timestamp, seller.NewsletterPermissionTimesStamp);
-            Assert.Null(seller.EMail);
         }
     }
 }
-Original file line number
+Diff line change
@@ Expand Up / @@ -95,10 +95,6 @@ public void UpdateNewsletterPermissions() @@
                 {
                     NewsletterPermissionTimesStamp = DateTime.Now;
                 }
-                else
-                {
-                    EMail = null;
-                }
             }
         }
     }