Skip to content

Commit ca1eab2

Browse files
fix: validationUtil
1 parent 9e39e75 commit ca1eab2

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

src/main/java/dev/resms/util/ValidationUtil.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,10 +12,14 @@ public class ValidationUtil {
1212
* @throws ReSMSException if the phone number is invalid
1313
*/
1414
public static void validatePhoneNumber(String phoneNumber) throws ReSMSException {
15-
if (phoneNumber == null || phoneNumber.trim().isEmpty()) {
15+
if (phoneNumber == null) {
1616
throw new ReSMSException("Phone number is required");
1717
}
1818

19+
if (phoneNumber.trim().isEmpty()) {
20+
throw new ReSMSException("Phone number cannot be empty");
21+
}
22+
1923
String cleanNumber = phoneNumber.trim();
2024
if (!cleanNumber.matches("^\\+?[1-9]\\d{1,14}$")) {
2125
throw new ReSMSException("Invalid phone number format");

src/test/java/dev/resms/util/ValidationUtilTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ void testValidatePhoneNumber_validNumber() {
1616
@Test
1717
void testValidatePhoneNumber_emptyNumber() {
1818
ReSMSException ex = assertThrows(ReSMSException.class, () -> ValidationUtil.validatePhoneNumber(" "));
19-
assertEquals("Phone number is required", ex.getMessage());
19+
assertEquals("Phone number cannot be empty", ex.getMessage());
2020
}
2121

2222
@Test

0 commit comments

Comments
 (0)