add Test

Author: larsgrefer

autoconfigure/build.gradle

@@ -1,6 +1,7 @@
 plugins {
     id "findbugs"
     id "checkstyle"
+    id "jacoco"
     id "io.spring.dependency-management" version "1.0.3.RELEASE"
 }
 apply plugin: "io.freefair.maven-jars"
@@ -21,6 +22,8 @@ dependencyManagement {
     }
 }
 
+jacoco.toolVersion = "0.8.0"
+
 sourceCompatibility = 1.8
 targetCompatibility = 1.8
 

autoconfigure/src/main/java/io/freefair/spring/okhttp/OkHttpProperties.java

@@ -25,12 +25,12 @@ public class OkHttpProperties {
     /**
      * The default read timeout for new connections.
      */
-    private Duration readTimeout = Duration.ofSeconds(10);
+    private Duration readTimeout;
 
     /**
      * The default write timeout for new connections.
      */
-    private Duration writeTimeout = Duration.ofSeconds(10);
+    private Duration writeTimeout;
 
     /**
      * The interval between web socket pings initiated by this client. Use this to

autoconfigure/src/test/java/io/freefair/spring/okhttp/DurationPropertiesIT.java

@@ -0,0 +1,51 @@
+package io.freefair.spring.okhttp;
+
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.boot.SpringBootConfiguration;
+import org.springframework.boot.context.properties.EnableConfigurationProperties;
+import org.springframework.boot.test.context.SpringBootTest;
+import org.springframework.test.context.junit4.SpringRunner;
+
+import java.time.Duration;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+
+/**
+ * @author Lars Grefer
+ */
+@RunWith(SpringRunner.class)
+@SpringBootTest(
+        properties = {
+                "okhttp.connectTimeout=10s",
+                "okhttp.readTimeout=1d",
+                "okhttp.writeTimeout=5"
+        }
+)
+public class DurationPropertiesIT {
+
+    @Autowired
+    private OkHttpProperties okHttpProperties;
+
+    @Test
+    public void getConnectTimeout() {
+        assertThat(okHttpProperties.getConnectTimeout()).isEqualTo(Duration.ofSeconds(10));
+    }
+
+    @Test
+    public void getReadTimeout() {
+        assertThat(okHttpProperties.getReadTimeout()).isEqualTo(Duration.ofDays(1));
+    }
+
+    @Test
+    public void getWriteTimeout() {
+        assertThat(okHttpProperties.getWriteTimeout()).isEqualTo(Duration.ofMillis(5));
+    }
+
+    @SpringBootConfiguration
+    @EnableConfigurationProperties(OkHttpProperties.class)
+    public static class Config {
+    }
+}

autoconfigure/src/test/java/io/freefair/spring/okhttp/OkHttp3AutoConfigurationTest.java

@@ -28,7 +28,7 @@ public class OkHttp3AutoConfigurationTest {
     private OkHttpClient okHttpClient;
 
     @Test
-    public void testContextLoads() throws Exception {
+    public void testContextLoads() {
         assertThat(okHttpClient).isNotNull();
     }