ShouldThrowValidationExceptionIfNoReceiverWasAdded -> PASS

mabroukmahdhi · Aug 11, 2024 · f04a093 · f04a093
1 parent 5026f76
commit f04a093
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 12 deletions.
diff --git a/FlexiMail.Tests.Unit/Services/FlexiExchangeServiceTests.Validations.cs b/FlexiMail.Tests.Unit/Services/FlexiExchangeServiceTests.Validations.cs
@@ -74,15 +74,7 @@ public async void ShouldThrowValidationExceptionIfNoReceiverWasAdded()
 
             invalidFlexiMessageException.AddData(
                 key: nameof(FlexiMessage.To),
-                values: "Value is not set");
-
-            invalidFlexiMessageException.AddData(
-                key: nameof(FlexiMessage.Cc),
-                values: "Value is not set");
-
-            invalidFlexiMessageException.AddData(
-                key: nameof(FlexiMessage.Bcc),
-                values: "Value is not set");
+                values: "Value is not set"); 
 
             var expectedFlexiMessageValidationException =
                 new FlexiMessageValidationException(

diff --git a/FlexiMail/Services/FlexiExchangeService.Validations.cs b/FlexiMail/Services/FlexiExchangeService.Validations.cs
@@ -14,9 +14,9 @@ private static void ValidFlexiMessage(FlexiMessage flexiMessage)
         {
             ValidFlexiMessageIsNotNull(flexiMessage);
 
-            Validate((Rule: IsInvalid(flexiMessage.To), Parameter: nameof(FlexiMessage.To)),
-                (Rule: IsInvalid(flexiMessage.Cc), Parameter: nameof(FlexiMessage.Cc)),
-                (Rule: IsInvalid(flexiMessage.Bcc), Parameter: nameof(FlexiMessage.Bcc)));
+            Validate((Rule: IsInvalid(flexiMessage.To), Parameter: nameof(FlexiMessage.To)));
+            Validate((Rule: IsInvalid(flexiMessage.Cc), Parameter: nameof(FlexiMessage.Cc)));
+            Validate((Rule: IsInvalid(flexiMessage.Bcc), Parameter: nameof(FlexiMessage.Bcc)));
         }
 
         private static void ValidFlexiMessageIsNotNull(FlexiMessage flexiMessage)