hotfix: implement HTML email sending without BCC and add unit tests f…

…or email service

src/main/java/leonardo/labutilities/qualitylabpro/services/email/EmailService.java

@@ -13,7 +13,6 @@
 import org.springframework.mail.SimpleMailMessage;
 import org.springframework.mail.javamail.JavaMailSender;
 import org.springframework.mail.javamail.MimeMessageHelper;
-import org.springframework.scheduling.annotation.Async;
 import org.springframework.scheduling.annotation.EnableAsync;
 import org.springframework.stereotype.Service;
 
@@ -61,7 +60,26 @@ public void sendPlainTextEmail(EmailDTO email) {
 		javaMailSender.send(message);
 	}
 
-	@Async
+	public void sendHtmlEmailWithoutBcc(EmailDTO emailDTO) {
+		MimeMessage mimeMessage = javaMailSender.createMimeMessage();
+		try {
+			MimeMessageHelper helper = new MimeMessageHelper(mimeMessage, true);
+			helper.setFrom(emailFrom);
+
+			helper.addTo(emailDTO.to());
+			helper.setSubject(EMAIL_SUBJECT_PREFIX + emailDTO.subject());
+			helper.setText(buildEmailBody(emailDTO.body()), true);
+
+			javaMailSender.send(mimeMessage);
+			log.info("HTML identifier sent successfully to {} client", emailDTO.to());
+
+		} catch (MessagingException e) {
+			log.error("Failed to send HTML identifier: {}", e.getMessage(), e);
+			throw new RuntimeException("Email sending failed", e);
+		}
+	}
+
+
 	public void sendHtmlEmail(EmailDTO emailDTO) {
 		MimeMessage mimeMessage = javaMailSender.createMimeMessage();
 		try {
@@ -170,7 +188,7 @@ public void sendUserActionEmail(String actionType, String username, String email
 			LocalDateTime date) {
 		String subject = String.format("User %s - %s", username, actionType);
 		String content = createUserActionEmailContent(actionType, username, email, date);
-		this.sendPlainTextEmail(new EmailDTO(email, subject, content));
+		sendHtmlEmailWithoutBcc(new EmailDTO(email, subject, content));
 	}
 
 	private String createUserActionEmailContent(String actionType, String username, String email,

src/test/java/leonardo/labutilities/qualitylabpro/services/EmailServiceTest.java

@@ -0,0 +1,70 @@
+package leonardo.labutilities.qualitylabpro.services;
+
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.verify;
+import static org.mockito.Mockito.when;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
+import org.mockito.InjectMocks;
+import org.mockito.Mock;
+import org.mockito.junit.jupiter.MockitoExtension;
+import org.springframework.mail.MailSendException;
+import org.springframework.mail.javamail.JavaMailSender;
+import org.springframework.test.util.ReflectionTestUtils;
+import jakarta.mail.MessagingException;
+import jakarta.mail.internet.MimeMessage;
+import leonardo.labutilities.qualitylabpro.dtos.email.EmailDTO;
+import leonardo.labutilities.qualitylabpro.services.email.EmailService;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.doThrow;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+@ExtendWith(MockitoExtension.class)
+class EmailServiceTest {
+
+	@Mock
+	private JavaMailSender javaMailSender;
+
+	@InjectMocks
+	private EmailService emailService;
+
+	private static final String TEST_EMAIL_FROM = "[email protected]";
+
+	@BeforeEach
+	void setUp() {
+		ReflectionTestUtils.setField(emailService, "emailFrom", TEST_EMAIL_FROM);
+	}
+
+	@Test
+	void shouldSendHtmlEmailSuccessfully() throws MessagingException {
+		// Given
+		EmailDTO emailDTO = new EmailDTO("[email protected]", "Test Subject", "Test Body");
+		MimeMessage mimeMessage = mock(MimeMessage.class);
+		when(javaMailSender.createMimeMessage()).thenReturn(mimeMessage);
+
+		// When
+		emailService.sendHtmlEmailWithoutBcc(emailDTO);
+
+		// Then
+		verify(javaMailSender).createMimeMessage();
+		verify(javaMailSender).send(mimeMessage);
+	}
+
+	@Test
+	void shouldThrowRuntimeExceptionWhenEmailFails() throws MessagingException {
+		// Given
+		EmailDTO emailDTO = new EmailDTO("[email protected]", "Test Subject", "Test Body");
+		MimeMessage mimeMessage = mock(MimeMessage.class);
+		when(javaMailSender.createMimeMessage()).thenReturn(mimeMessage);
+		doThrow(new MailSendException("Email sending failed")).when(javaMailSender)
+				.send(any(MimeMessage.class));
+
+		// Then
+		RuntimeException exception = assertThrows(RuntimeException.class, () -> {
+			emailService.sendHtmlEmailWithoutBcc(emailDTO);
+		});
+		assertEquals("Email sending failed", exception.getMessage());
+	}
+}