test: add Unit Tests to core package (#243)

crowdin · May 9, 2024 · 049b9ff · 049b9ff
1 parent acb303e
commit 049b9ff
Show file tree

Hide file tree

Showing 16 changed files with 736 additions and 11 deletions.
diff --git a/build.gradle b/build.gradle
@@ -29,7 +29,7 @@ dependencies {
     implementation 'org.apache.httpcomponents:httpclient:4.5.13'
     implementation 'com.fasterxml.jackson.core:jackson-databind:2.14.0'
 
-    testImplementation 'org.junit.jupiter:junit-jupiter-api:5.9.0'
+    testImplementation 'org.junit.jupiter:junit-jupiter-api:5.9.0','org.mockito:mockito-core:4.11.0'
     testRuntimeOnly 'org.junit.jupiter:junit-jupiter-engine:5.9.0'
     testAnnotationProcessor 'org.projectlombok:lombok:1.18.24'
 }

diff --git a/src/main/java/com/crowdin/client/core/CrowdinApi.java b/src/main/java/com/crowdin/client/core/CrowdinApi.java
@@ -6,6 +6,9 @@
 import com.crowdin.client.core.http.impl.json.JacksonJsonTransformer;
 import com.crowdin.client.core.model.ClientConfig;
 import com.crowdin.client.core.model.Credentials;
+import org.apache.http.client.config.CookieSpecs;
+import org.apache.http.client.config.RequestConfig;
+import org.apache.http.impl.client.HttpClientBuilder;
 
 import java.util.Collections;
 import java.util.HashMap;
@@ -18,7 +21,12 @@ public abstract class CrowdinApi {
     protected final String url;
 
     public CrowdinApi(Credentials credentials) {
-        this(credentials, ClientConfig.builder().httpClient(new ApacheHttpClient(credentials, new JacksonJsonTransformer(), Collections.emptyMap())).build());
+        this(credentials, ClientConfig.builder()
+                .httpClient(new ApacheHttpClient(credentials, new JacksonJsonTransformer(), Collections.emptyMap(), HttpClientBuilder
+                        .create()
+                        .setDefaultRequestConfig(RequestConfig.custom().setCookieSpec(CookieSpecs.STANDARD).build())
+                        .build()))
+                .build());
     }
 
     public CrowdinApi(Credentials credentials, ClientConfig clientConfig) {

diff --git a/src/main/java/com/crowdin/client/core/http/impl/http/ApacheHttpClient.java b/src/main/java/com/crowdin/client/core/http/impl/http/ApacheHttpClient.java
@@ -49,14 +49,11 @@ public class ApacheHttpClient implements HttpClient {
 
     private final CloseableHttpClient httpClient;
 
-    public ApacheHttpClient(Credentials credentials, JsonTransformer jsonTransformer, Map<String, ?> defaultHeaders) {
+    public ApacheHttpClient(Credentials credentials, JsonTransformer jsonTransformer, Map<String, ?> defaultHeaders, CloseableHttpClient httpClient) {
         this.credentials = credentials;
         this.jsonTransformer = jsonTransformer;
         this.defaultHeaders = defaultHeaders;
-        this.httpClient = HttpClientBuilder
-            .create()
-            .setDefaultRequestConfig(RequestConfig.custom().setCookieSpec(CookieSpecs.STANDARD).build())
-            .build();
+        this.httpClient = httpClient;
     }
 
     public ApacheHttpClient(Credentials credentials, JsonTransformer jsonTransformer, Map<String, ?> defaultHeaders, ClientConfig.Host proxy, ClientConfig.UsernamePasswordCredentials proxyCreds, Integer timeoutMs) {

diff --git a/src/main/java/com/crowdin/client/core/http/impl/json/EmptyArrayToNullDeserializer.java b/src/main/java/com/crowdin/client/core/http/impl/json/EmptyArrayToNullDeserializer.java
@@ -1,6 +1,5 @@
 package com.crowdin.client.core.http.impl.json;
 
-import com.fasterxml.jackson.core.JacksonException;
 import com.fasterxml.jackson.core.JsonParser;
 import com.fasterxml.jackson.core.JsonToken;
 import com.fasterxml.jackson.databind.*;
@@ -23,7 +22,7 @@ public EmptyArrayToNullDeserializer(JavaType type) {
     }
 
     @Override
-    public Object deserialize(JsonParser p, DeserializationContext ctxt) throws IOException, JacksonException {
+    public Object deserialize(JsonParser p, DeserializationContext ctxt) throws IOException {
         if (p.getCurrentToken() == JsonToken.VALUE_NULL) {
             return null;
         }
@@ -43,11 +42,11 @@ public Object deserialize(JsonParser p, DeserializationContext ctxt) throws IOEx
     }
 
     private static boolean isCollectionType(Class<?> type) {
-        return type.isArray() && Collection.class.isAssignableFrom(type);
+        return type.isArray() || Collection.class.isAssignableFrom(type);
     }
 
     @Override
-    public JsonDeserializer<?> createContextual(DeserializationContext ctxt, BeanProperty property) throws JsonMappingException {
+    public JsonDeserializer<?> createContextual(DeserializationContext ctxt, BeanProperty property) {
         return new EmptyArrayToNullDeserializer(property.getType());
     }
 }
diff --git a/src/test/java/com/crowdin/client/core/http/CrowdinApiConstructorTest.java b/src/test/java/com/crowdin/client/core/http/CrowdinApiConstructorTest.java
@@ -0,0 +1,24 @@
+package com.crowdin.client.core.http;
+
+import com.crowdin.client.Client;
+import com.crowdin.client.core.model.ClientConfig;
+import com.crowdin.client.core.model.Credentials;
+
+public class CrowdinApiConstructorTest extends Client {
+
+    public CrowdinApiConstructorTest(Credentials credentials, ClientConfig clientConfig) {
+        super(credentials, clientConfig);
+    }
+
+    public CrowdinApiConstructorTest(Credentials credentials) {
+        super(credentials);
+    }
+
+    public HttpClient getHttpClient() {
+        return this.httpClient;
+    }
+
+    public String geturl() {
+        return this.url;
+    }
+}
diff --git a/src/test/java/com/crowdin/client/core/http/CrowdinApiTest.java b/src/test/java/com/crowdin/client/core/http/CrowdinApiTest.java
@@ -0,0 +1,79 @@
+package com.crowdin.client.core.http;
+
+import com.crowdin.client.core.http.impl.http.ApacheHttpClient;
+import com.crowdin.client.core.http.impl.json.JacksonJsonTransformer;
+import com.crowdin.client.core.model.ClientConfig;
+import com.crowdin.client.core.model.Credentials;
+import com.crowdin.client.framework.TestHttpClient;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.TestInstance;
+
+import static org.junit.jupiter.api.Assertions.*;
+
+@TestInstance(TestInstance.Lifecycle.PER_CLASS)
+public class CrowdinApiTest {
+
+    private final String organization = "testOrganization";
+    private final String token = "testToken";
+    private final String regularUrl = "https://api.crowdin.com/api/v2";
+    private final String organizationUrl = "https://testOrganization.api.crowdin.com/api/v2";
+
+    @Test
+    public void testCrowdinApiWithNullInputs() {
+        CrowdinApiConstructorTest client = new CrowdinApiConstructorTest(new Credentials(null, null, null), ClientConfig.builder()
+                .userAgent("User-Agent")
+                .build());
+        assertNotNull(client);
+        assertInstanceOf(ApacheHttpClient.class, client.getHttpClient());
+    }
+
+    @Test
+    public void testCrowdinApiWithAgent() {
+        CrowdinApiConstructorTest client = new CrowdinApiConstructorTest(new Credentials(null, null, null), ClientConfig.builder()
+                .integrationUserAgent("X-Crowdin-Integrations-User-Agent")
+                .jsonTransformer(new JacksonJsonTransformer())
+                .build());
+        assertNotNull(client);
+        assertInstanceOf(ApacheHttpClient.class, client.getHttpClient());
+        assertEquals(client.geturl(), regularUrl);
+    }
+
+    @Test
+    public void testCrowdinApiWithBaseUrl() {
+        CrowdinApiConstructorTest client = new CrowdinApiConstructorTest(new Credentials(token, organization, "/"), ClientConfig.builder()
+                .httpClient(new TestHttpClient()).build());
+        assertEquals(client.geturl(), "/api/v2");
+    }
+
+    @Test
+    public void testCrowdinApiWithBaseUrlEmpty() {
+        CrowdinApiConstructorTest client = new CrowdinApiConstructorTest(new Credentials(token, organization, ""), ClientConfig.builder()
+                .userAgent("User-Agent")
+                .httpClient(new TestHttpClient()).build());
+        assertEquals(client.geturl(), "/api/v2");
+    }
+
+    @Test
+    public void testCrowdinApiForRegularUrl() {
+        CrowdinApiConstructorTest client = new CrowdinApiConstructorTest(new Credentials(token, null, null), ClientConfig.builder()
+                .userAgent("User-Agent")
+                .httpClient(new TestHttpClient()).build());
+        assertEquals(client.geturl(), regularUrl);
+    }
+
+    @Test
+    public void testCrowdinApiForOrganizationUrl() {
+        CrowdinApiConstructorTest client = new CrowdinApiConstructorTest(new Credentials(token, organization, null), ClientConfig.builder()
+                .userAgent("User-Agent")
+                .httpClient(new TestHttpClient()).build());
+        assertEquals(client.geturl(), organizationUrl);
+    }
+
+    @Test
+    public void testCrowdinApiConstructor() {
+        CrowdinApiConstructorTest client = new CrowdinApiConstructorTest(new Credentials(token, organization, null));
+        assertInstanceOf(ApacheHttpClient.class, client.getHttpClient());
+        assertEquals(client.geturl(), organizationUrl);
+    }
+
+}
diff --git a/src/test/java/com/crowdin/client/core/http/exception/HttpExceptionTest.java b/src/test/java/com/crowdin/client/core/http/exception/HttpExceptionTest.java
@@ -0,0 +1,15 @@
+package com.crowdin.client.core.http.exception;
+
+import com.crowdin.client.core.http.exceptions.HttpException;
+import org.junit.jupiter.api.Test;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+public class HttpExceptionTest {
+
+    @Test
+    public void httpExceptionFromTest() {
+        HttpException message = HttpException.fromMessage("Exception message!");
+        assertEquals(message.getError().getMessage(), "Exception message!");
+    }
+}