Merge pull request #5 from xenit-eu/java-8-datetime

Added junit test for JDK8 Time-api support

Author: tgeens

opa-async-java-client/src/main/java/eu/xenit/contentcloud/opa/client/OpaClient.java

@@ -143,7 +143,7 @@ public OpaClient build() {
 
         }
 
-        private OpaHttpClient getOrCreateDefaultHttpClient() {
+        protected OpaHttpClient getOrCreateDefaultHttpClient() {
             var client = this.restClient;
 
             // configure the provided client

opa-async-java-client/src/main/java/eu/xenit/contentcloud/opa/client/rest/client/jdk/DefaultOpaHttpClient.java

@@ -27,7 +27,8 @@
 public class DefaultOpaHttpClient implements OpaHttpClient {
 
     private final HttpClient httpClient;
-    private final ConverterProcessor converterProcessor;
+
+    final ConverterProcessor converterProcessor;
 
     private URI baseUrl;
     private RequestLogger logger;

opa-async-java-client/src/test/java/eu/xenit/contentcloud/opa/client/rest/client/jdk/DefaultOpaHttpClientTest.java

@@ -0,0 +1,27 @@
+package eu.xenit.contentcloud.opa.client.rest.client.jdk;
+
+import eu.xenit.contentcloud.opa.client.OpaClient;
+import eu.xenit.contentcloud.opa.client.rest.client.jdk.converter.HttpBodyConverter;
+import eu.xenit.contentcloud.opa.client.rest.http.MediaType;
+import org.junit.jupiter.api.Test;
+
+import java.time.Instant;
+import java.util.Map;
+
+class DefaultOpaHttpClientTest {
+
+    @Test
+    void defaultOpaClient_shouldSupportJdk8TimeApi() {
+        var opaClient = new OpaClient.Builder() {
+            DefaultOpaHttpClient getHttpClient() {
+                return (DefaultOpaHttpClient) super.getOrCreateDefaultHttpClient();
+            }
+        }.getHttpClient();
+
+        opaClient.converterProcessor.write(HttpBodyConverter.SerializationContext.of(
+                Map.of("auth_time", Instant.now()),
+                new HttpHeadersBuilder().add("Content-Type", MediaType.APPLICATION_JSON.toString())
+        ));
+    }
+
+}