Email delivery Unit Tests (#72)

* Email Delivery Unit Test

---------

Signed-off-by: Anders Swanson <anders.swanson@oracle.com>

Author: anders-swanson

docs/src/main/asciidoc/email.adoc

@@ -4,7 +4,7 @@
 [#email-delivery]
 == Email Delivery
 
-https://docs.oracle.com/en-us/iaas/Content/Email/home.htm[Email Delivery] provides an implementation of the Spring Mail interface, providing MailSender and JavaMailSender beans to send email using OCI Email Delivery.
+https://docs.oracle.com/en-us/iaas/Content/Email/home.htm[Email Delivery] module provides an implementation of the Spring Mail interface, implementing MailSender and JavaMailSender interfaces to send email using OCI Email Delivery.
 
 Maven coordinates:
 
@@ -74,7 +74,7 @@ public class EmailService {
 
 === Using JavaMailSender
 
-For more complex emails, such as those with attachments, use the JavaMailSender interface. The followign example shows how to send an email with the JavaMailSender API.
+For more complex emails, such as those with attachments, use the JavaMailSender interface. The following example shows how to send an email with the JavaMailSender API.
 
 [source,java]
 ----

spring-cloud-oci-email/pom.xml

@@ -67,11 +67,5 @@ Licensed under the Universal Permissive License v 1.0 as shown at https://oss.or
             <groupId>org.eclipse.angus</groupId>
             <artifactId>jakarta.mail</artifactId>
         </dependency>
-
-        <dependency>
-            <groupId>org.projectlombok</groupId>
-            <artifactId>lombok</artifactId>
-            <scope>provided</scope>
-        </dependency>
     </dependencies>
 </project>

spring-cloud-oci-email/src/main/java/com/oracle/cloud/spring/email/EmailDeliveryMailSender.java

@@ -14,14 +14,12 @@
 import jakarta.mail.internet.AddressException;
 import jakarta.mail.internet.InternetAddress;
 import jakarta.mail.internet.MimeMessage;
-import lombok.Builder;
 import org.springframework.mail.MailException;
 import org.springframework.mail.MailPreparationException;
 import org.springframework.mail.MailSendException;
 import org.springframework.mail.MailSender;
 import org.springframework.mail.SimpleMailMessage;
 
-@Builder
 public class EmailDeliveryMailSender implements MailSender {
     protected final Session session;
     private final String smtpHost;

spring-cloud-oci-email/src/test/java/com/oracle/cloud/spring/email/EmailDeliveryJavaMailSenderTest.java

@@ -0,0 +1,39 @@
+package com.oracle.cloud.spring.email;
+
+import jakarta.mail.Message;
+import jakarta.mail.MessagingException;
+import jakarta.mail.internet.MimeMessage;
+import org.junit.jupiter.api.Test;
+
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.times;
+import static org.mockito.Mockito.verify;
+
+public class EmailDeliveryJavaMailSenderTest extends EmailTest {
+    @Test
+    void sendMessage() throws MessagingException {
+        EmailDeliveryJavaMailSender sender = getSender();
+        sender.send(getMimeMessage(sender));
+        verify(transport, times(1)).sendMessage(any(), any());
+    }
+
+    @Test
+    void sendMessages() throws MessagingException {
+        EmailDeliveryJavaMailSender sender = getSender();
+        sender.send(getMimeMessage(sender), getMimeMessage(sender), getMimeMessage(sender));
+        verify(transport, times(3)).sendMessage(any(), any());
+    }
+
+    private EmailDeliveryJavaMailSender getSender() {
+        return new EmailDeliveryJavaMailSender(session, "x", "y", "z");
+    }
+
+    private MimeMessage getMimeMessage(EmailDeliveryJavaMailSender sender) throws MessagingException {
+        MimeMessage message = sender.createMimeMessage();
+        message.setFrom(testEmail);
+        message.setRecipients(Message.RecipientType.TO, testEmail2);
+        message.setText(testBody);
+        message.setSubject(testSubject);
+        return message;
+    }
+}

spring-cloud-oci-email/src/test/java/com/oracle/cloud/spring/email/EmailDeliveryMailSenderTest.java

@@ -0,0 +1,51 @@
+package com.oracle.cloud.spring.email;
+
+import jakarta.mail.MessagingException;
+import org.junit.jupiter.api.Test;
+import org.springframework.mail.SimpleMailMessage;
+
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.times;
+import static org.mockito.Mockito.verify;
+
+public class EmailDeliveryMailSenderTest extends EmailTest {
+    @Test
+    void sendNoFrom() throws MessagingException {
+        SimpleMailMessage message = getMessage();
+        message.setFrom(null);
+        assertThatExceptionOfType(NullPointerException.class)
+                .isThrownBy(() -> getSender().send(message))
+                .withMessage("From address is null");
+        verify(transport, times(0)).sendMessage(any(), any());
+    }
+
+    @Test
+    void sendMessage() throws MessagingException {
+        getSender().send(getMessage());
+        verify(transport, times(1)).sendMessage(any(), any());
+    }
+
+    @Test
+    void sendMessages() throws MessagingException {
+        getSender().send(getMessage(), getMessage(), getMessage());
+        verify(transport, times(3)).sendMessage(any(), any());
+    }
+
+    private EmailDeliveryMailSender getSender() {
+        return new EmailDeliveryMailSender(session, "x", "y", "z");
+    }
+
+    private SimpleMailMessage getMessage() {
+        SimpleMailMessage message = new SimpleMailMessage();
+        message.setFrom(testEmail);
+        message.setTo(testEmail);
+        message.setCc(testEmail2);
+        message.setBcc(testEmail3);
+        message.setSubject(testSubject);
+        message.setText(testBody);
+        message.setReplyTo(testEmail2);
+        return message;
+    }
+}

spring-cloud-oci-email/src/test/java/com/oracle/cloud/spring/email/EmailTest.java

@@ -0,0 +1,29 @@
+package com.oracle.cloud.spring.email;
+
+import java.util.Properties;
+
+import jakarta.mail.NoSuchProviderException;
+import jakarta.mail.Session;
+import jakarta.mail.Transport;
+import org.junit.jupiter.api.BeforeEach;
+
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.when;
+
+public class EmailTest {
+    final String testEmail = "test@test.xyz";
+    final String testEmail2 = "test2@test.xyz";
+    final String testEmail3 = "test3@test.xyz";
+    final String testSubject = "test subject";
+    final String testBody = "test body";
+    Session session;
+    Transport transport;
+
+    @BeforeEach
+    void setUpEach() throws NoSuchProviderException {
+        session = mock(Session.class);
+        transport = mock(Transport.class);
+        when(session.getTransport()).thenReturn(transport);
+        when(session.getProperties()).thenReturn(new Properties());
+    }
+}