Add input validation for duration parameters with 1-second minimum requirement

Author: aakash007s

sdk/src/main/java/com/amazonaws/lambda/durable/validation/ParameterValidator.java

@@ -29,9 +29,8 @@ public static void validateDuration(Duration duration, String parameterName) {
             throw new IllegalArgumentException(parameterName + " cannot be null");
         }
         if (duration.toSeconds() < MIN_DURATION_SECONDS) {
-            throw new IllegalArgumentException(
-                    parameterName + " must be at least " + MIN_DURATION_SECONDS + " second, got: "
-                            + duration.toSeconds() + " seconds");
+            throw new IllegalArgumentException(parameterName + " must be at least " + MIN_DURATION_SECONDS
+                    + " second, got: " + duration.toSeconds() + " seconds");
         }
     }
 
@@ -44,9 +43,8 @@ public static void validateDuration(Duration duration, String parameterName) {
      */
     public static void validateOptionalDuration(Duration duration, String parameterName) {
         if (duration != null && duration.toSeconds() < MIN_DURATION_SECONDS) {
-            throw new IllegalArgumentException(
-                    parameterName + " must be at least " + MIN_DURATION_SECONDS + " second, got: "
-                            + duration.toSeconds() + " seconds");
+            throw new IllegalArgumentException(parameterName + " must be at least " + MIN_DURATION_SECONDS
+                    + " second, got: " + duration.toSeconds() + " seconds");
         }
     }
 

sdk/src/test/java/com/amazonaws/lambda/durable/DurationValidationIntegrationTest.java

@@ -21,9 +21,9 @@ void callbackConfig_withInvalidTimeout_shouldThrow() {
 
     @Test
     void callbackConfig_withInvalidHeartbeatTimeout_shouldThrow() {
-        var exception = assertThrows(
-                IllegalArgumentException.class,
-                () -> CallbackConfig.builder().heartbeatTimeout(Duration.ofMillis(999)).build());
+        var exception = assertThrows(IllegalArgumentException.class, () -> CallbackConfig.builder()
+                .heartbeatTimeout(Duration.ofMillis(999))
+                .build());
 
         assertTrue(exception.getMessage().contains("Heartbeat timeout"));
         assertTrue(exception.getMessage().contains("at least 1 second"));
@@ -39,10 +39,8 @@ void callbackConfig_withValidTimeouts_shouldPass() {
 
     @Test
     void callbackConfig_withNullTimeouts_shouldPass() {
-        assertDoesNotThrow(() -> CallbackConfig.builder()
-                .timeout(null)
-                .heartbeatTimeout(null)
-                .build());
+        assertDoesNotThrow(() ->
+                CallbackConfig.builder().timeout(null).heartbeatTimeout(null).build());
     }
 
     @Test
@@ -57,7 +55,8 @@ void invokeConfig_withInvalidTimeout_shouldThrow() {
 
     @Test
     void invokeConfig_withValidTimeout_shouldPass() {
-        assertDoesNotThrow(() -> InvokeConfig.builder().timeout(Duration.ofSeconds(30)).build());
+        assertDoesNotThrow(
+                () -> InvokeConfig.builder().timeout(Duration.ofSeconds(30)).build());
     }
 
     @Test
@@ -67,9 +66,9 @@ void invokeConfig_withNullTimeout_shouldPass() {
 
     @Test
     void durableConfig_withInvalidPollingInterval_shouldThrow() {
-        var exception = assertThrows(
-                IllegalArgumentException.class,
-                () -> DurableConfig.builder().withPollingInterval(Duration.ofMillis(500)).build());
+        var exception = assertThrows(IllegalArgumentException.class, () -> DurableConfig.builder()
+                .withPollingInterval(Duration.ofMillis(500))
+                .build());
 
         assertTrue(exception.getMessage().contains("Polling interval"));
         assertTrue(exception.getMessage().contains("at least 1 second"));
@@ -84,8 +83,7 @@ void durableConfig_withValidPollingInterval_shouldPass() {
 
     @Test
     void durableConfig_withNullPollingInterval_shouldPass() {
-        assertDoesNotThrow(() -> DurableConfig.builder()
-                .withPollingInterval(null)
-                .build());
+        assertDoesNotThrow(
+                () -> DurableConfig.builder().withPollingInterval(null).build());
     }
 }

sdk/src/test/java/com/amazonaws/lambda/durable/retry/RetryStrategiesTest.java

@@ -218,8 +218,7 @@ void fixedDelay_withSubSecondDelay_shouldThrow() {
 
     @Test
     void fixedDelay_withNullDelay_shouldThrow() {
-        var exception =
-                assertThrows(IllegalArgumentException.class, () -> RetryStrategies.fixedDelay(3, null));
+        var exception = assertThrows(IllegalArgumentException.class, () -> RetryStrategies.fixedDelay(3, null));
 
         assertTrue(exception.getMessage().contains("fixedDelay"));
         assertTrue(exception.getMessage().contains("cannot be null"));