Apply pull request suggestions

Signed-off-by: Matheus Cruz <matheuscruz.dev@gmail.com>

Author: mcruzdev

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

@@ -40,37 +40,22 @@ public WorkflowModel apply(
       Builder request, WorkflowContext workflow, TaskContext task, WorkflowModel model) {
     HttpModelConverter converter = HttpConverterResolver.converter(workflow, task);
 
-    if (!redirect) {
-      request.property("jersey.config.client.followRedirects", false);
-    }
-
     Response response = invokeRequest(request, converter, workflow, task, model);
-    validateStatus(task, response);
+    validateStatus(task, response, converter);
     return workflow
         .definition()
         .application()
         .modelFactory()
         .fromAny(response.readEntity(converter.responseType()));
   }
 
-  private void validateStatus(TaskContext task, Response response) {
+  private void validateStatus(TaskContext task, Response response, HttpModelConverter converter) {
     Family statusFamily = response.getStatusInfo().getFamily();
-    boolean isSuccess = statusFamily.equals(SUCCESSFUL);
-    boolean isRedirect = statusFamily.equals(REDIRECTION);
-    boolean valid = redirect ? (isSuccess || isRedirect) : isSuccess;
 
-    if (!valid) {
-      String expectedRange = redirect ? "200-399" : "200-299";
+    if (statusFamily != SUCCESSFUL || (!this.redirect && statusFamily == REDIRECTION)) {
       throw new WorkflowException(
-          WorkflowError.communication(
-                  response.getStatus(),
-                  task,
-                  String.format(
-                      "The property 'redirect' is set to %s but received status %d (%s); expected status in the %s range",
-                      redirect,
-                      response.getStatus(),
-                      response.getStatusInfo().getReasonPhrase(),
-                      expectedRange))
+          converter
+              .errorFromResponse(WorkflowError.communication(response.getStatus(), task), response)
               .build());
     }
   }

impl/test/src/test/java/io/serverlessworkflow/impl/test/HTTPWorkflowDefinitionTest.java

@@ -35,6 +35,7 @@
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 
 public class HTTPWorkflowDefinitionTest {
@@ -415,6 +416,27 @@ void testOptionsCall() throws IOException, InterruptedException {
   }
 
   @Test
+  @Disabled(
+      value =
+          "See the following discussion: https://github.com/serverlessworkflow/sdk-java/pull/1013/files#r2562919102")
+  void testPatchCall() throws IOException, InterruptedException {
+    mockServer.enqueue(new MockResponse(204, Headers.of(), ""));
+    appl.workflowDefinition(readWorkflowFromClasspath("workflows-samples/call-http-patch.yaml"))
+        .instance(Map.of())
+        .start()
+        .join();
+
+    RecordedRequest recordedRequest = mockServer.takeRequest();
+    SoftAssertions.assertSoftly(
+        softly -> {
+          softly.assertThat(recordedRequest.getMethod()).isEqualTo("PATCH");
+          softly.assertThat(recordedRequest.getUrl().toString()).contains("/users/1");
+        });
+  }
+
+  @Test
+  @Disabled(
+      "See the following discussion: https://github.com/serverlessworkflow/sdk-java/pull/1013/files#r2566152233")
   void testRedirect_should_throws_when_redirect_is_false_and_response_status_is_not_2xx() {
     mockServer.enqueue(
         new MockResponse(301, Headers.of("Location", "http://localhost:9876/redirected"), ""));
@@ -429,8 +451,6 @@ void testRedirect_should_throws_when_redirect_is_false_and_response_status_is_no
                     .instance(Map.of())
                     .start()
                     .join());
-    assertThat(exception.getCause().getMessage())
-        .contains(
-            "The property 'redirect' is set to false but received status 301 (Redirection); expected status in the 200-299 range");
+    assertThat(exception.getCause().getMessage()).contains("status=301");
   }
 }