Skip body building when HTTP call body is absent (#1649)

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

Author: mathecruz

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

@@ -140,8 +140,10 @@ private RequestExecutor buildRequestExecutor() {
       case HttpMethod.POST:
       case HttpMethod.PUT:
       case HttpMethod.PATCH:
-        return new WithBodyRequestExecutor(
-            httpMethod, redirect, auth, definition.application(), body);
+        return body != null
+            ? new WithBodyRequestExecutor(
+                httpMethod, redirect, auth, definition.application(), body)
+            : new WithoutBodyRequestExecutor(httpMethod, redirect, auth);
       case HttpMethod.DELETE:
       case HttpMethod.HEAD:
       case HttpMethod.OPTIONS:

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

@@ -324,6 +324,27 @@ void callHttpPost_should_return_created_firstName() throws Exception {
         });
   }
 
+  @Test
+  void callHttpPost_without_body_should_not_throw() throws Exception {
+    mockServer.enqueue(new MockResponse(204, Headers.of(), ""));
+
+    assertDoesNotThrow(
+        () ->
+            appl.workflowDefinition(
+                    readWorkflowFromClasspath("workflows-samples/call-http-post-no-body.yaml"))
+                .instance(Map.of())
+                .start()
+                .join());
+
+    RecordedRequest recordedRequest = mockServer.takeRequest();
+    SoftAssertions.assertSoftly(
+        softly -> {
+          softly.assertThat(recordedRequest.getMethod()).isEqualTo("POST");
+          softly.assertThat(recordedRequest.getUrl()).asString().contains("/api/v1/authors");
+          softly.assertThat(recordedRequest.getBodySize()).isEqualTo(0);
+        });
+  }
+
   @Test
   void testCallHttpDelete() throws IOException, InterruptedException {
     mockServer.enqueue(new MockResponse(204, Headers.of(), ""));

impl/test/src/test/resources/workflows-samples/call-http-post-no-body.yaml

@@ -0,0 +1,12 @@
+document:
+  dsl: 1.0.0-alpha1
+  namespace: test
+  name: call-http-post-no-body
+  version: 1.0.0
+do:
+  - postAuthors:
+      call: http
+      with:
+       method: post
+       endpoint:
+         uri: http://localhost:9876/api/v1/authors