Fix code quality and format

Author: sashirestela

src/main/java/io/github/sashirestela/cleverclient/retry/RetryableRequest.java

@@ -83,7 +83,8 @@ private <T> void executeAsyncAttempt(Supplier<Object> operation,
 
     private boolean isRetryable(Throwable exception) {
         exception = (exception instanceof CompletionException) ? exception.getCause() : exception;
-        if (exception == null) return false;
+        if (exception == null)
+            return false;
         Throwable theException = (exception instanceof CleverClientException) ? exception.getCause() : exception;
         if (theException != null) {
             return Arrays.stream(config.getRetryableExceptions())
@@ -102,16 +103,15 @@ private long calculateDelayWithJitter(int attempt) {
         double multiplier = Math.pow(config.getBackoffMultiplier(), attempt - 1.0);
         long baseDelay = (long) (config.getInitialDelayMs() * multiplier);
         baseDelay = Math.min(baseDelay, config.getMaxDelayMs());
-
         if (config.getJitterFactor() > 0) {
-            double jitterRange = baseDelay * config.getJitterFactor();
+            long jitterRange = (long) (baseDelay * config.getJitterFactor());
             // Generate random jitter between -jitterRange/2 and +jitterRange/2
-            long jitter = (long) ((random.nextDouble() - 0.5) * jitterRange);
+            long halfRange = jitterRange / 2;
+            long jitter = -halfRange + (random.nextLong() % (halfRange + 1));
             baseDelay = Math.max(0, baseDelay + jitter);
             // Ensure we don't exceed maxDelayMs even with jitter
             baseDelay = Math.min(baseDelay, config.getMaxDelayMs());
         }
-
         return baseDelay;
     }
 

src/test/java/io/github/sashirestela/cleverclient/http/HttpProcessorTest.java

@@ -420,7 +420,8 @@ default void shouldReturnAnObjectAsyncWhenIsPostMethodAndBodyIsEmpty() throws IO
     default void shouldReturnAnObjectSyncWhenConfiguredRetry() throws IOException, InterruptedException {
         setMocksForString(SyncType.SYNC, "{\"id\":100,\"description\":\"Description\",\"active\":true}");
 
-        var service = getHttpProcessor(new RetryableRequest(RetryConfig.defaultValues())).createProxy(ITest.SyncService.class);
+        var service = getHttpProcessor(new RetryableRequest(RetryConfig.defaultValues()))
+                .createProxy(ITest.SyncService.class);
         var actualDemo = service.getDemo(100);
         var expectedDemo = new ITest.Demo(100, "Description", true);
 
@@ -431,7 +432,8 @@ default void shouldReturnAnObjectSyncWhenConfiguredRetry() throws IOException, I
     default void shouldReturnAnObjectAsyncWhenConfiguredRetry() throws IOException, InterruptedException {
         setMocksForString(SyncType.ASYNC, "{\"id\":100,\"description\":\"Description\",\"active\":true}");
 
-        var service = getHttpProcessor(new RetryableRequest(RetryConfig.defaultValues())).createProxy(ITest.AsyncService.class);
+        var service = getHttpProcessor(new RetryableRequest(RetryConfig.defaultValues()))
+                .createProxy(ITest.AsyncService.class);
         var actualDemo = service.getDemo(100).join();
         var expectedDemo = new ITest.Demo(100, "Description", true);