Apply refactoring

Signed-off-by: Matheus Cruz <[email protected]>

Author: mcruzdev

impl/http/src/main/java/io/serverlessworkflow/impl/executors/http/AbstractHttpExecutorBuilder.java

@@ -25,6 +25,7 @@
 import jakarta.ws.rs.HttpMethod;
 import jakarta.ws.rs.client.WebTarget;
 import jakarta.ws.rs.core.Response;
+
 import java.net.URI;
 import java.util.Map;
 import java.util.Optional;
@@ -48,18 +49,21 @@ protected static RequestSupplier buildRequestSupplier(
           HttpModelConverter converter = HttpConverterResolver.converter(w, t);
           Response res =
               request.buildPost(converter.toEntity(bodyFilter.apply(w, t, node))).invoke();
-          handleWithRedirect(redirect, t, res);
+
+          applyRedirectValidation(redirect, t, res);
 
           return w.definition()
               .application()
               .modelFactory()
-              .fromAny(res.readEntity(converter.responseType()));
+              .fromAny(
+                  request.post(
+                      converter.toEntity(bodyFilter.apply(w, t, node)), converter.responseType()));
         };
       case HttpMethod.GET:
       default:
         return (request, w, t, n) -> {
           Response res = request.buildGet().invoke();
-          handleWithRedirect(redirect, t, res);
+          applyRedirectValidation(redirect, t, res);
 
           return w.definition()
               .application()
@@ -69,32 +73,25 @@ protected static RequestSupplier buildRequestSupplier(
     }
   }
 
-  private static void handleWithRedirect(boolean redirect, TaskContext t, Response response) {
+  private static void applyRedirectValidation(boolean redirect, TaskContext t, Response response) {
     Response.Status.Family statusFamily = response.getStatusInfo().getFamily();
+    boolean isSuccess = statusFamily.equals(Response.Status.Family.SUCCESSFUL);
+    boolean isRedirect = statusFamily.equals(Response.Status.Family.REDIRECTION);
 
-    if (redirect
-        && (!statusFamily.equals(Response.Status.Family.SUCCESSFUL)
-            && !statusFamily.equals(Response.Status.Family.REDIRECTION))) {
-      throw new WorkflowException(
-          WorkflowError.communication(
-                  response.getStatus(),
-                  t,
-                  new RedirectValidationException(
-                      String.format(
-                          "The property 'redirect' is true but received status %d (%s); expected status in the 200-399 range",
-                          response.getStatus(), response.getStatusInfo().getReasonPhrase())))
-              .build());
-    }
-
-    if (!redirect && !statusFamily.equals(Response.Status.Family.SUCCESSFUL)) {
+    boolean valid = redirect ? (isSuccess || isRedirect) : isSuccess;
+    if (!valid) {
+      String expectedRange = redirect ? "200-399" : "200-299";
       throw new WorkflowException(
           WorkflowError.communication(
                   response.getStatus(),
                   t,
                   new RedirectValidationException(
                       String.format(
-                          "The property 'redirect' is false but received status %d (%s); expected status in the 200-299 range",
-                          response.getStatus(), response.getStatusInfo().getReasonPhrase())))
+                          "The property 'redirect' is %s but received status %d (%s); expected status in the %s range",
+                          redirect,
+                          response.getStatus(),
+                          response.getStatusInfo().getReasonPhrase(),
+                          expectedRange)))
               .build());
     }
   }