1390 - SF

bytechefhq · Nov 13, 2024 · 718f937 · 718f937
1 parent 490bed5
commit 718f937
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 10 deletions.
diff --git a/...-service/src/main/java/com/bytechef/platform/component/definition/HttpClientExecutor.java b/...-service/src/main/java/com/bytechef/platform/component/definition/HttpClientExecutor.java
@@ -383,9 +383,10 @@ private BodyPublisher getFormUrlEncodedBodyPublisher(Body body) {
 
     private void processParameters(String prefix, Map<?, ?> parameters, FormBodyPublisher.Builder builder) {
         parameters.forEach((key, value) -> {
-            String newKey = prefix.isEmpty() ? key.toString() : prefix + "[" + key + "]";
             Validate.notNull(value, "Expected value for " + key);
 
+            String newKey = prefix.isEmpty() ? key.toString() : prefix + "[" + key + "]";
+
             if (value instanceof Map<?, ?> nestedMap) {
                 processParameters(newKey, nestedMap, builder);
             } else if (value instanceof List<?> list) {

diff --git a/...vice/src/test/java/com/bytechef/platform/component/definition/HttpClientExecutorTest.java b/...vice/src/test/java/com/bytechef/platform/component/definition/HttpClientExecutorTest.java
@@ -17,6 +17,8 @@
 package com.bytechef.platform.component.definition;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertNull;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import com.bytechef.component.definition.ActionContext;
@@ -51,7 +53,6 @@
 import java.util.Map;
 import java.util.Optional;
 import javax.net.ssl.SSLSession;
-import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 import org.mockito.Mockito;
@@ -215,7 +216,7 @@ public void testCreateHTTPClient() {
         assertTrue(httpClient.authenticator()
             .isEmpty());
 
-        Assertions.assertNotNull(httpClient.sslContext());
+        assertNotNull(httpClient.sslContext());
 
         //
 
@@ -352,7 +353,7 @@ public void testCreateHTTPClient() {
             new ComponentConnection("componentName", 1, -1, Map.of(), Authorization.AuthorizationType.NONE.name()),
             Mockito.mock(Context.class));
 
-        Assertions.assertNotNull(httpClient.followRedirects());
+        assertNotNull(httpClient.followRedirects());
 
         //
 
@@ -363,7 +364,7 @@ public void testCreateHTTPClient() {
             new ComponentConnection("componentName", 1, -1, Map.of(), Authorization.AuthorizationType.NONE.name()),
             Mockito.mock(Context.class));
 
-        Assertions.assertNotNull(httpClient.followRedirects());
+        assertNotNull(httpClient.followRedirects());
 
         //
 
@@ -411,7 +412,7 @@ public void testCreateHTTPRequest() {
     @Disabled
     @Test
     public void testHandleResponse() {
-        Assertions.assertNull(
+        assertNull(
             httpClientExecutor
                 .handleResponse(new TestHttpResponse(null), configuration)
                 .getBody());
@@ -502,15 +503,15 @@ public void testHandleResponseWithIncompatibleResponseType() {
         Http.Response response =
             httpClientExecutor.handleResponse(testHttpResponse, configurationBuilder.build());
 
-        Assertions.assertNull(response.getBody());
+        assertNull(response.getBody());
 
         testHttpResponse = new TestHttpResponse(
             "{\"key1\":\"value1\", \"key2\":\"value2\"}",
             HttpHeaders.of(Map.of("content-type", List.of("application/json")), (s, s2) -> true), 404);
 
         response = httpClientExecutor.handleResponse(testHttpResponse, configurationBuilder.build());
 
-        Assertions.assertNotNull(response.getBody());
+        assertNotNull(response.getBody());
 
         assertEquals(Map.of("key1", "value1", "key2", "value2"), response.getBody());
 
@@ -520,7 +521,7 @@ public void testHandleResponseWithIncompatibleResponseType() {
 
         response = httpClientExecutor.handleResponse(testHttpResponse, configurationBuilder.build());
 
-        Assertions.assertNull(response.getBody());
+        assertNull(response.getBody());
 
         testHttpResponse = new TestHttpResponse(
             "<root><key1>value1</key1><key2>value2</key2></root>",
@@ -530,7 +531,7 @@ public void testHandleResponseWithIncompatibleResponseType() {
             testHttpResponse, configurationBuilder.responseType(Http.ResponseType.XML)
                 .build());
 
-        Assertions.assertNotNull(response.getBody());
+        assertNotNull(response.getBody());
 
         assertEquals(Map.of("key1", "value1", "key2", "value2"), response.getBody());
     }