Enhance email error handling and expand test coverage (#3561 )

# Description of Changes Please provide a summary of the changes, including: - **What was changed** - **EmailController**: Added a `catch (MailSendException)` block to handle invalid-address errors, log the exception, and return a 500 response with the raw error message. - **EmailServiceTest**: Added unit tests for attachment-related error cases (missing filename, null filename, missing file, null file) and invalid “to” address (null or empty), expecting `MessagingException` or `MailSendException`. - **MailConfigTest**: New test class verifying `MailConfig.java` correctly initializes `JavaMailSenderImpl` with host, port, username, password, default encoding, and SMTP properties. - **EmailControllerTest**: Refactored into a parameterized test (`shouldHandleEmailRequests`) covering four scenarios: success, generic messaging error, missing `to` parameter, and invalid address formatting. - **Why the change was made** - To ensure invalid email addresses and missing attachments are handled gracefully at the controller layer, providing clearer feedback to API clients. - To improve overall test coverage and guard against regressions in email functionality. - To enforce correct mail configuration via automated tests. --- ## Checklist ### General - [x] I have read the [Contribution Guidelines](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/CONTRIBUTING.md) - [x] I have read the [Stirling-PDF Developer Guide](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/DeveloperGuide.md) (if applicable) - [ ] I have read the [How to add new languages to Stirling-PDF](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/HowToAddNewLanguage.md) (if applicable) - [x] I have performed a self-review of my own code - [x] My changes generate no new warnings ### Documentation - [ ] I have updated relevant docs on [Stirling-PDF's doc repo](https://github.com/Stirling-Tools/Stirling-Tools.github.io/blob/main/docs/) (if functionality has heavily changed) - [ ] I have read the section [Add New Translation Tags](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/HowToAddNewLanguage.md#add-new-translation-tags) (for new translation tags only) ### UI Changes (if applicable) - [ ] Screenshots or videos demonstrating the UI changes are attached (e.g., as comments or direct attachments in the PR) ### Testing (if applicable) - [ ] I have tested my changes locally. Refer to the [Testing Guide](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/DeveloperGuide.md#6-testing) for more details.
Ukrainian translation (#3567 )
2025-06-23 07:55:07 +00:00 · 2025-05-21 15:42:08 +01:00 · 2025-05-21 15:41:51 +01:00 · 2025-05-21 15:41:11 +01:00
8 changed files with 379 additions and 108 deletions
--- a/src/main/java/stirling/software/SPDF/config/security/mail/EmailService.java
+++ b/src/main/java/stirling/software/SPDF/config/security/mail/EmailService.java
@ -37,8 +37,21 @@ public class EmailService {
     */
    @Async
    public void sendEmailWithAttachment(Email email) throws MessagingException {
-        ApplicationProperties.Mail mailProperties = applicationProperties.getMail();
        MultipartFile file = email.getFileInput();
+        // 1) Validate recipient email address
+        if (email.getTo() == null || email.getTo().trim().isEmpty()) {
+            throw new MessagingException("Invalid Addresses");
+        }
+
+        // 2) Validate attachment
+        if (file == null
+                || file.isEmpty()
+                || file.getOriginalFilename() == null
+                || file.getOriginalFilename().isEmpty()) {
+            throw new MessagingException("An attachment is required to send the email.");
+        }
+
+        ApplicationProperties.Mail mailProperties = applicationProperties.getMail();

        // Creates a MimeMessage to represent the email
        MimeMessage message = mailSender.createMimeMessage();
--- a/src/main/java/stirling/software/SPDF/controller/api/EmailController.java
+++ b/src/main/java/stirling/software/SPDF/controller/api/EmailController.java
@ -3,6 +3,7 @@ package stirling.software.SPDF.controller.api;
 import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
+import org.springframework.mail.MailSendException;
 import org.springframework.web.bind.annotation.ModelAttribute;
 import org.springframework.web.bind.annotation.PostMapping;
 import org.springframework.web.bind.annotation.RequestMapping;
@ -53,6 +54,11 @@ public class EmailController {
            // Calls the service to send the email with attachment
            emailService.sendEmailWithAttachment(email);
            return ResponseEntity.ok("Email sent successfully");
+        } catch (MailSendException ex) {
+            // handles your "Invalid Addresses" case
+            String errorMsg = ex.getMessage();
+            log.error("MailSendException: {}", errorMsg, ex);
+            return ResponseEntity.status(HttpStatus.INTERNAL_SERVER_ERROR).body(errorMsg);
        } catch (MessagingException e) {
            // Catches any messaging exception (e.g., invalid email address, SMTP server issues)
            String errorMsg = "Failed to send email: " + e.getMessage();
--- a/src/main/java/stirling/software/SPDF/controller/web/UploadLimitService.java
+++ b/src/main/java/stirling/software/SPDF/controller/web/UploadLimitService.java
@ -1,5 +1,6 @@
 package stirling.software.SPDF.controller.web;

+import java.util.Locale;
 import java.util.regex.Pattern;

 import org.springframework.beans.factory.annotation.Autowired;
@ -52,6 +53,6 @@ public class UploadLimitService {
        if (bytes < 1024) return bytes + " B";
        int exp = (int) (Math.log(bytes) / Math.log(1024));
        String pre = "KMGTPE".charAt(exp - 1) + "B";
-        return String.format("%.1f %s", bytes / Math.pow(1024, exp), pre);
+        return String.format(Locale.US, "%.1f %s", bytes / Math.pow(1024, exp), pre);
    }
 }
--- a/src/main/resources/messages_uk_UA.properties
+++ b/src/main/resources/messages_uk_UA.properties
@ -10,9 +10,9 @@ multiPdfPrompt=Оберіть PDFи (2+)
 multiPdfDropPrompt=Оберіть (або перетягніть) всі необхідні PDFи
 imgPrompt=Оберіть зображення(я)
 genericSubmit=Надіслати
-uploadLimit=Maximum file size:
-uploadLimitExceededSingular=is too large. Maximum allowed size is
-uploadLimitExceededPlural=are too large. Maximum allowed size is
+uploadLimit=Максимальний розмір файлу:
+uploadLimitExceededSingular=занадто великий. Максимально дозволений розмір -
+uploadLimitExceededPlural=занадто великі. Максимально дозволений розмір -
 processTimeWarning=Увага: Цей процес може тривати до хвилини в залежності від розміру файлу.
 pageOrderPrompt=Порядок сторінок (введіть список номерів сторінок через кому):
 pageSelectionPrompt=Користувацький вибір сторінки (введіть список номерів сторінок через кому 1,5,6 або функції типу 2n+1) :
@ -86,14 +86,14 @@ loading=Завантаження...
 addToDoc=Додати до документу
 reset=Скинути
 apply=Застосувати
-noFileSelected=No file selected. Please upload one.
+noFileSelected=Файл не вибрано. Будь ласка, завантажте один.

 legal.privacy=Політика конфіденційності
 legal.terms=Правила та умови
 legal.accessibility=Доступність
 legal.cookie=Політика використання файлів cookie
 legal.impressum=Вихідні дані
-legal.showCookieBanner=Cookie Preferences
+legal.showCookieBanner=Налаштування файлів cookie

 ###############
 #   Pipeline  #
@ -237,7 +237,7 @@ adminUserSettings.activeUsers=Активні користувачі:
 adminUserSettings.disabledUsers=Заблоковані користувачі:
 adminUserSettings.totalUsers=Всього користувачів:
 adminUserSettings.lastRequest=Останній запит
-adminUserSettings.usage=View Usage
+adminUserSettings.usage=Переглянути використання

 endpointStatistics.title=Статистика кінцевих точок
 endpointStatistics.header=Статистика кінцевих точок
@ -364,9 +364,9 @@ home.compressPdfs.title=Стиснути
 home.compressPdfs.desc=Стискайте PDF-файли, щоб зменшити їх розмір.
 compressPdfs.tags=стиск,маленький,крихітний

-home.unlockPDFForms.title=Unlock PDF Forms
-home.unlockPDFForms.desc=Remove read-only property of form fields in a PDF document.
-unlockPDFForms.tags=remove,delete,form,field,readonly
+home.unlockPDFForms.title=Розблокувати PDF форми
+home.unlockPDFForms.desc=Видалити властивість "тільки для читання" з полів форми у PDF-документі.
+unlockPDFForms.tags=видалити,розблокувати,форма,поле,тільки для читання

 home.changeMetadata.title=Змінити метадані
 home.changeMetadata.desc=Змінити/видалити/додати метадані з документа PDF
@ -609,7 +609,7 @@ login.userIsDisabled=Користувач деактивовано, вхід з
 login.alreadyLoggedIn=Ви вже увійшли до
 login.alreadyLoggedIn2=пристроїв (а). Будь ласка, вийдіть із цих пристроїв і спробуйте знову.
 login.toManySessions=У вас дуже багато активних сесій
-login.logoutMessage=You have been logged out.
+login.logoutMessage=Ви вийшли з системи.

 #auto-redact
 autoRedact.title=Автоматичне редагування
@ -742,10 +742,10 @@ sanitizePDF.title=Дезінфекція PDF
 sanitizePDF.header=Дезінфекція PDF файлу
 sanitizePDF.selectText.1=Видалити JavaScript
 sanitizePDF.selectText.2=Видалити вбудовані файли
-sanitizePDF.selectText.3=Remove XMP metadata
+sanitizePDF.selectText.3=Видалити XMP метадані
 sanitizePDF.selectText.4=Видалити посилання
 sanitizePDF.selectText.5=Видалити шрифти
-sanitizePDF.selectText.6=Remove Document Info Metadata
+sanitizePDF.selectText.6=Видалити метадані інформації про документ
 sanitizePDF.submit=Дезінфекція


@ -1071,7 +1071,7 @@ rotate.submit=Повернути
 split.title=Розділити PDF
 split.header=Розділити PDF
 split.desc.1=Числа, які ви вибрали, це номери сторінок, на яких ви хочете зробити розділ.
-split.desc.2=Таким чином, вибір 1,3,7-8 розділить 10-сторінковий документ на 6 окремих PDF-файлів з:
+split.desc.2=Таким чином, вибір 1,3,7-8 розділіть 10-сторінковий документ на 6 окремих PDF-файлів з:
 split.desc.3=Документ #1: Сторінка 1
 split.desc.4=Документ #2: Сторінки 2 і 3
 split.desc.5=Документ #3: Сторінки 4, 5 і 6
@ -1372,68 +1372,68 @@ fileChooser.extractPDF=Видобування...

 #release notes
 releases.footer=Релізи
-releases.title=Примечания к релизу
-releases.header=Примечания к релизу
-releases.current.version=Текущий релиз
-releases.note=Примітка до релізу доступна тільки на англійській мові
+releases.title=Примітки до релізу
+releases.header=Примітки до релізу
+releases.current.version=Поточний реліз
+releases.note=Примітки до релізу доступні лише англійською мовою

 #Validate Signature
 validateSignature.title=Перевірка підписів PDF
 validateSignature.header=Перевірка цифрових підписів
 validateSignature.selectPDF=Виберіть підписаний PDF-файл
 validateSignature.submit=Перевірити підписи
-validateSignature.results=Результаты проверки
+validateSignature.results=Результати перевірки
 validateSignature.status=Статус
 validateSignature.signer=Підписант
 validateSignature.date=Дата
 validateSignature.reason=Причина
-validateSignature.location=Местоположение
-validateSignature.noSignatures=В цьому документі не знайдено цифрових підписів
-validateSignature.status.valid=Дійна
-validateSignature.status.invalid=Недійсна
-validateSignature.chain.invalid=Перевірка цепочки сертифікатів не удалась - неможливо перевірити особистість підписанта
+validateSignature.location=Місцезнаходження
+validateSignature.noSignatures=У цьому документі не знайдено цифрових підписів
+validateSignature.status.valid=Дійсний
+validateSignature.status.invalid=Недійсний
+validateSignature.chain.invalid=Перевірка ланцюга сертифікатів не вдалася - неможливо перевірити особу підписанта
 validateSignature.trust.invalid=Сертифікат відсутній у довіреному сховищі - джерело не може бути перевірено
-validateSignature.cert.expired=Срок дії сертифіката істеку
-validateSignature.cert.revoked=Сертифікат був отозван
-validateSignature.signature.info=Інформація про підписи
-validateSignature.signature=Подпись
-validateSignature.signature.mathValid=Подпись математически корректна, НО:
-validateSignature.selectCustomCert=Користувачський файл сертифіката X.509 (Необов'язково)
-validateSignature.cert.info=Сведения про сертифікати
-validateSignature.cert.issuer=Издатель
-validateSignature.cert.subject=суб'єкт
-validateSignature.cert.serialNumber=Серийний номер
+validateSignature.cert.expired=Термін дії сертифіката закінчився
+validateSignature.cert.revoked=Сертифікат було відкликано
+validateSignature.signature.info=Інформація про підпис
+validateSignature.signature=Підпис
+validateSignature.signature.mathValid=Підпис математично коректний, АЛЕ:
+validateSignature.selectCustomCert=Користувацький файл сертифіката X.509 (Необов'язково)
+validateSignature.cert.info=Інформація про сертифікат
+validateSignature.cert.issuer=Видавець
+validateSignature.cert.subject=Суб'єкт
+validateSignature.cert.serialNumber=Серійний номер
 validateSignature.cert.validFrom=Дійсний з
 validateSignature.cert.validUntil=Дійсний до
 validateSignature.cert.algorithm=Алгоритм
 validateSignature.cert.keySize=Розмір ключа
 validateSignature.cert.version=Версія
 validateSignature.cert.keyUsage=Використання ключа
-validateSignature.cert.selfSigned=Самоподписанный
+validateSignature.cert.selfSigned=Самопідписаний
 validateSignature.cert.bits=біт

 ####################
 #  Cookie banner   #
 ####################
-cookieBanner.popUp.title=How we use Cookies
-cookieBanner.popUp.description.1=We use cookies and other technologies to make Stirling PDF work better for you—helping us improve our tools and keep building features you'll love.
-cookieBanner.popUp.description.2=If you’d rather not, clicking 'No Thanks' will only enable the essential cookies needed to keep things running smoothly.
-cookieBanner.popUp.acceptAllBtn=Okay
-cookieBanner.popUp.acceptNecessaryBtn=No Thanks
-cookieBanner.popUp.showPreferencesBtn=Manage preferences
-cookieBanner.preferencesModal.title=Consent Preferences Center
-cookieBanner.preferencesModal.acceptAllBtn=Accept all
-cookieBanner.preferencesModal.acceptNecessaryBtn=Reject all
-cookieBanner.preferencesModal.savePreferencesBtn=Save preferences
-cookieBanner.preferencesModal.closeIconLabel=Close modal
-cookieBanner.preferencesModal.serviceCounterLabel=Service|Services
-cookieBanner.preferencesModal.subtitle=Cookie Usage
-cookieBanner.preferencesModal.description.1=Stirling PDF uses cookies and similar technologies to enhance your experience and understand how our tools are used. This helps us improve performance, develop the features you care about, and provide ongoing support to our users.
-cookieBanner.preferencesModal.description.2=Stirling PDF cannot—and will never—track or access the content of the documents you use.
-cookieBanner.preferencesModal.description.3=Your privacy and trust are at the core of what we do.
-cookieBanner.preferencesModal.necessary.title.1=Strictly Necessary Cookies
-cookieBanner.preferencesModal.necessary.title.2=Always Enabled
-cookieBanner.preferencesModal.necessary.description=These cookies are essential for the website to function properly. They enable core features like setting your privacy preferences, logging in, and filling out forms—which is why they can’t be turned off.
-cookieBanner.preferencesModal.analytics.title=Analytics
-cookieBanner.preferencesModal.analytics.description=These cookies help us understand how our tools are being used, so we can focus on building the features our community values most. Rest assured—Stirling PDF cannot and will never track the content of the documents you work with.
+cookieBanner.popUp.title=Як ми використовуємо файли cookie
+cookieBanner.popUp.description.1=Ми використовуємо файли cookie та інші технології, щоб Stirling PDF працював краще для вас — допомагаючи нам покращувати наші інструменти та створювати функції, які вам сподобаються.
+cookieBanner.popUp.description.2=Якщо ви не хочете, натискання «Ні, дякую» увімкне лише необхідні файли cookie, потрібні для безперебійної роботи.
+cookieBanner.popUp.acceptAllBtn=Добре
+cookieBanner.popUp.acceptNecessaryBtn=Ні, дякую
+cookieBanner.popUp.showPreferencesBtn=Керувати налаштуваннями
+cookieBanner.preferencesModal.title=Центр налаштувань згоди
+cookieBanner.preferencesModal.acceptAllBtn=Прийняти всі
+cookieBanner.preferencesModal.acceptNecessaryBtn=Відхилити всі
+cookieBanner.preferencesModal.savePreferencesBtn=Зберегти налаштування
+cookieBanner.preferencesModal.closeIconLabel=Закрити модальне вікно
+cookieBanner.preferencesModal.serviceCounterLabel=Сервіс|Сервіси
+cookieBanner.preferencesModal.subtitle=Використання файлів cookie
+cookieBanner.preferencesModal.description.1=Stirling PDF використовує файли cookie та подібні технології, щоб покращити ваш досвід і зрозуміти, як використовуються наші інструменти. Це допомагає нам покращувати продуктивність, розробляти функції, які вас цікавлять, і надавати постійну підтримку нашим користувачам.
+cookieBanner.preferencesModal.description.2=Stirling PDF не може — і ніколи не буде — відстежувати або отримувати доступ до вмісту документів, які ви використовуєте.
+cookieBanner.preferencesModal.description.3=Ваша конфіденційність і довіра є основою того, що ми робимо.
+cookieBanner.preferencesModal.necessary.title.1=Суворо необхідні файли cookie
+cookieBanner.preferencesModal.necessary.title.2=Завжди увімкнені
+cookieBanner.preferencesModal.necessary.description=Ці файли cookie є необхідними для правильного функціонування вебсайту. Вони забезпечують основні функції, такі як налаштування ваших уподобань конфіденційності, вхід у систему та заповнення форм — тому їх не можна вимкнути.
+cookieBanner.preferencesModal.analytics.title=Аналітика
+cookieBanner.preferencesModal.analytics.description=Ці файли cookie допомагають нам зрозуміти, як використовуються наші інструменти, щоб ми могли зосередитися на створенні функцій, які найбільше цінує наша спільнота. Будьте впевнені — Stirling PDF не може і ніколи не буде відстежувати вміст документів, з якими ви працюєте.

--- a/src/test/java/stirling/software/SPDF/config/security/mail/EmailServiceTest.java
+++ b/src/test/java/stirling/software/SPDF/config/security/mail/EmailServiceTest.java
@ -1,5 +1,7 @@
 package stirling.software.SPDF.config.security.mail;

+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.fail;
 import static org.mockito.Mockito.*;

 import org.junit.jupiter.api.Test;
@ -57,4 +59,111 @@ public class EmailServiceTest {
        // Verify that the email was sent using mailSender
        verify(mailSender).send(mimeMessage);
    }
+
+    @Test
+    void testSendEmailWithAttachmentThrowsExceptionForMissingFilename() throws MessagingException {
+        Email email = new Email();
+        email.setTo("test@example.com");
+        email.setSubject("Test Email");
+        email.setBody("This is a test email.");
+        email.setFileInput(fileInput);
+
+        when(fileInput.isEmpty()).thenReturn(false);
+        when(fileInput.getOriginalFilename()).thenReturn("");
+
+        try {
+            emailService.sendEmailWithAttachment(email);
+            fail("Expected MessagingException to be thrown");
+        } catch (MessagingException e) {
+            assertEquals("An attachment is required to send the email.", e.getMessage());
+        }
+    }
+
+    @Test
+    void testSendEmailWithAttachmentThrowsExceptionForMissingFilenameNull()
+            throws MessagingException {
+        Email email = new Email();
+        email.setTo("test@example.com");
+        email.setSubject("Test Email");
+        email.setBody("This is a test email.");
+        email.setFileInput(fileInput);
+
+        when(fileInput.isEmpty()).thenReturn(false);
+        when(fileInput.getOriginalFilename()).thenReturn(null);
+
+        try {
+            emailService.sendEmailWithAttachment(email);
+            fail("Expected MessagingException to be thrown");
+        } catch (MessagingException e) {
+            assertEquals("An attachment is required to send the email.", e.getMessage());
+        }
+    }
+
+    @Test
+    void testSendEmailWithAttachmentThrowsExceptionForMissingFile() throws MessagingException {
+        Email email = new Email();
+        email.setTo("test@example.com");
+        email.setSubject("Test Email");
+        email.setBody("This is a test email.");
+        email.setFileInput(fileInput);
+
+        when(fileInput.isEmpty()).thenReturn(true);
+
+        try {
+            emailService.sendEmailWithAttachment(email);
+            fail("Expected MessagingException to be thrown");
+        } catch (MessagingException e) {
+            assertEquals("An attachment is required to send the email.", e.getMessage());
+        }
+    }
+
+    @Test
+    void testSendEmailWithAttachmentThrowsExceptionForMissingFileNull() throws MessagingException {
+        Email email = new Email();
+        email.setTo("test@example.com");
+        email.setSubject("Test Email");
+        email.setBody("This is a test email.");
+        email.setFileInput(null); // Missing file
+
+        try {
+            emailService.sendEmailWithAttachment(email);
+            fail("Expected MessagingException to be thrown");
+        } catch (MessagingException e) {
+            assertEquals("An attachment is required to send the email.", e.getMessage());
+        }
+    }
+
+    @Test
+    void testSendEmailWithAttachmentThrowsExceptionForInvalidAddressNull()
+            throws MessagingException {
+        Email email = new Email();
+        email.setTo(null); // Invalid address
+        email.setSubject("Test Email");
+        email.setBody("This is a test email.");
+        email.setFileInput(fileInput);
+
+        try {
+            emailService.sendEmailWithAttachment(email);
+            fail("Expected MailSendException to be thrown");
+        } catch (MessagingException e) {
+            assertEquals("Invalid Addresses", e.getMessage());
+        }
+    }
+
+    @Test
+    void testSendEmailWithAttachmentThrowsExceptionForInvalidAddressEmpty()
+            throws MessagingException {
+        Email email = new Email();
+        email.setTo(""); // Invalid address
+        email.setSubject("Test Email");
+        email.setBody("This is a test email.");
+        email.setFileInput(fileInput);
+
+        try {
+            emailService.sendEmailWithAttachment(email);
+            fail("Expected MailSendException to be thrown");
+        } catch (MessagingException e) {
+            assertEquals("Invalid Addresses", e.getMessage());
+        }
+    }
 }
--- a/src/test/java/stirling/software/SPDF/config/security/mail/MailConfigTest.java
+++ b/src/test/java/stirling/software/SPDF/config/security/mail/MailConfigTest.java
@ -0,0 +1,54 @@
+package stirling.software.SPDF.config.security.mail;
+
+import static org.junit.jupiter.api.Assertions.assertAll;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertInstanceOf;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.when;
+
+import java.util.Properties;
+
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.springframework.mail.javamail.JavaMailSender;
+import org.springframework.mail.javamail.JavaMailSenderImpl;
+
+import stirling.software.SPDF.model.ApplicationProperties;
+
+class MailConfigTest {
+
+    private ApplicationProperties.Mail mailProps;
+
+    @BeforeEach
+    void initMailProperties() {
+        mailProps = mock(ApplicationProperties.Mail.class);
+        when(mailProps.getHost()).thenReturn("smtp.example.com");
+        when(mailProps.getPort()).thenReturn(587);
+        when(mailProps.getUsername()).thenReturn("user@example.com");
+        when(mailProps.getPassword()).thenReturn("password");
+    }
+
+    @Test
+    void shouldConfigureJavaMailSenderWithCorrectProperties() {
+        ApplicationProperties appProps = mock(ApplicationProperties.class);
+        when(appProps.getMail()).thenReturn(mailProps);
+
+        MailConfig config = new MailConfig(appProps);
+        JavaMailSender sender = config.javaMailSender();
+
+        assertInstanceOf(JavaMailSenderImpl.class, sender);
+        JavaMailSenderImpl impl = (JavaMailSenderImpl) sender;
+
+        Properties props = impl.getJavaMailProperties();
+
+        assertAll(
+                "SMTP configuration",
+                () -> assertEquals("smtp.example.com", impl.getHost()),
+                () -> assertEquals(587, impl.getPort()),
+                () -> assertEquals("user@example.com", impl.getUsername()),
+                () -> assertEquals("password", impl.getPassword()),
+                () -> assertEquals("UTF-8", impl.getDefaultEncoding()),
+                () -> assertEquals("true", props.getProperty("mail.smtp.auth")),
+                () -> assertEquals("true", props.getProperty("mail.smtp.starttls.enable")));
+    }
+}
--- a/src/test/java/stirling/software/SPDF/controller/api/EmailControllerTest.java
+++ b/src/test/java/stirling/software/SPDF/controller/api/EmailControllerTest.java
@ -1,18 +1,25 @@
 package stirling.software.SPDF.controller.api;

-import static org.mockito.Mockito.*;
-import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.*;
-import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.*;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.doNothing;
+import static org.mockito.Mockito.doThrow;
+import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.multipart;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.content;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;
+
+import java.util.stream.Stream;

 import org.junit.jupiter.api.BeforeEach;
-import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
+import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.Arguments;
+import org.junit.jupiter.params.provider.MethodSource;
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
 import org.mockito.junit.jupiter.MockitoExtension;
+import org.springframework.mail.MailSendException;
 import org.springframework.test.web.servlet.MockMvc;
 import org.springframework.test.web.servlet.setup.MockMvcBuilders;
-import org.springframework.web.multipart.MultipartFile;

 import jakarta.mail.MessagingException;

@ -20,7 +27,7 @@ import stirling.software.SPDF.config.security.mail.EmailService;
 import stirling.software.SPDF.model.api.Email;

@ExtendWith(MockitoExtension.class)
-public class EmailControllerTest {
+class EmailControllerTest {

    private MockMvc mockMvc;

@ -28,59 +35,61 @@ public class EmailControllerTest {

    @InjectMocks private EmailController emailController;

-    @Mock private MultipartFile fileInput;
-
    @BeforeEach
    void setUp() {
-        // Set up the MockMvc instance for testing
        mockMvc = MockMvcBuilders.standaloneSetup(emailController).build();
    }

-    @Test
-    void testSendEmailWithAttachmentSuccess() throws Exception {
-        // Create a mock Email object
-        Email email = new Email();
-        email.setTo("test@example.com");
-        email.setSubject("Test Email");
-        email.setBody("This is a test email.");
-        email.setFileInput(fileInput);
+    @ParameterizedTest(name = "Case {index}: exception={0}, includeTo={1}")
+    @MethodSource("emailParams")
+    void shouldHandleEmailRequests(
+            Exception serviceException,
+            boolean includeTo,
+            int expectedStatus,
+            String expectedContent)
+            throws Exception {
+        if (serviceException == null) {
+            doNothing().when(emailService).sendEmailWithAttachment(any(Email.class));
+        } else {
+            doThrow(serviceException).when(emailService).sendEmailWithAttachment(any(Email.class));
+        }

-        // Mock the service to not throw any exception
-        doNothing().when(emailService).sendEmailWithAttachment(any(Email.class));
+        var request =
+                multipart("/api/v1/general/send-email")
+                        .file("fileInput", "dummy-content".getBytes())
+                        .param("subject", "Test Email")
+                        .param("body", "This is a test email.");

-        // Perform the request and verify the response
-        mockMvc.perform(
-                        multipart("/api/v1/general/send-email")
-                                .file("fileInput", "dummy-content".getBytes())
-                                .param("to", email.getTo())
-                                .param("subject", email.getSubject())
-                                .param("body", email.getBody()))
-                .andExpect(status().isOk())
-                .andExpect(content().string("Email sent successfully"));
+        if (includeTo) {
+            request = request.param("to", "test@example.com");
+        }
+
+        mockMvc.perform(request)
+                .andExpect(status().is(expectedStatus))
+                .andExpect(content().string(expectedContent));
    }

-    @Test
-    void testSendEmailWithAttachmentFailure() throws Exception {
-        // Create a mock Email object
-        Email email = new Email();
-        email.setTo("test@example.com");
-        email.setSubject("Test Email");
-        email.setBody("This is a test email.");
-        email.setFileInput(fileInput);
-
-        // Mock the service to throw a MessagingException
-        doThrow(new MessagingException("Failed to send email"))
-                .when(emailService)
-                .sendEmailWithAttachment(any(Email.class));
-
-        // Perform the request and verify the response
-        mockMvc.perform(
-                        multipart("/api/v1/general/send-email")
-                                .file("fileInput", "dummy-content".getBytes())
-                                .param("to", email.getTo())
-                                .param("subject", email.getSubject())
-                                .param("body", email.getBody()))
-                .andExpect(status().isInternalServerError())
-                .andExpect(content().string("Failed to send email: Failed to send email"));
+    static Stream<Arguments> emailParams() {
+        return Stream.of(
+                // success case
+                Arguments.of(null, true, 200, "Email sent successfully"),
+                // generic messaging error
+                Arguments.of(
+                        new MessagingException("Failed to send email"),
+                        true,
+                        500,
+                        "Failed to send email: Failed to send email"),
+                // missing 'to' results in MailSendException
+                Arguments.of(
+                        new MailSendException("Invalid Addresses"),
+                        false,
+                        500,
+                        "Invalid Addresses"),
+                // invalid email address formatting
+                Arguments.of(
+                        new MessagingException("Invalid Addresses"),
+                        true,
+                        500,
+                        "Failed to send email: Invalid Addresses"));
    }
 }
--- a/src/test/java/stirling/software/SPDF/controller/web/UploadLimitServiceTest.java
+++ b/src/test/java/stirling/software/SPDF/controller/web/UploadLimitServiceTest.java
@ -0,0 +1,79 @@
+package stirling.software.SPDF.controller.web;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.when;
+
+import java.util.stream.Stream;
+
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.Arguments;
+import org.junit.jupiter.params.provider.MethodSource;
+
+import stirling.software.SPDF.model.ApplicationProperties;
+
+class UploadLimitServiceTest {
+
+    private UploadLimitService uploadLimitService;
+    private ApplicationProperties applicationProperties;
+    private ApplicationProperties.System systemProps;
+
+    @BeforeEach
+    void setUp() {
+        applicationProperties = mock(ApplicationProperties.class);
+        systemProps = mock(ApplicationProperties.System.class);
+        when(applicationProperties.getSystem()).thenReturn(systemProps);
+
+        uploadLimitService = new UploadLimitService();
+        // inject mock
+        try {
+            var field = UploadLimitService.class.getDeclaredField("applicationProperties");
+            field.setAccessible(true);
+            field.set(uploadLimitService, applicationProperties);
+        } catch (ReflectiveOperationException e) {
+            throw new RuntimeException(e);
+        }
+    }
+
+    @ParameterizedTest(name = "getUploadLimit case #{index}: input={0}, expected={1}")
+    @MethodSource("uploadLimitParams")
+    void shouldComputeUploadLimitCorrectly(String input, long expected) {
+        when(systemProps.getFileUploadLimit()).thenReturn(input);
+
+        long result = uploadLimitService.getUploadLimit();
+        assertEquals(expected, result);
+    }
+
+    static Stream<Arguments> uploadLimitParams() {
+        return Stream.of(
+                // empty or null input yields 0
+                Arguments.of(null, 0L),
+                Arguments.of("", 0L),
+                // invalid formats
+                Arguments.of("1234MB", 0L),
+                Arguments.of("5TB", 0L),
+                // valid formats
+                Arguments.of("10KB", 10 * 1024L),
+                Arguments.of("2MB", 2 * 1024 * 1024L),
+                Arguments.of("1GB", 1L * 1024 * 1024 * 1024),
+                Arguments.of("5mb", 5 * 1024 * 1024L),
+                Arguments.of("0MB", 0L));
+    }
+
+    @ParameterizedTest(name = "getReadableUploadLimit case #{index}: rawValue={0}, expected={1}")
+    @MethodSource("readableLimitParams")
+    void shouldReturnReadableFormat(String rawValue, String expected) {
+        when(systemProps.getFileUploadLimit()).thenReturn(rawValue);
+        String result = uploadLimitService.getReadableUploadLimit();
+        assertEquals(expected, result);
+    }
+
+    static Stream<Arguments> readableLimitParams() {
+        return Stream.of(
+                Arguments.of(null, "0 B"),
+                Arguments.of("", "0 B"),
+                Arguments.of("1KB", "1.0 KB"),
+                Arguments.of("2MB", "2.0 MB"));
+    }
+}