From 5809da5683849ce93124ccc398e2e9d0373bd3e8 Mon Sep 17 00:00:00 2001
From: Javier <ing.javierlg@gmail.com>
Date: Fri, 24 Jan 2020 12:33:01 -0600
Subject: [PATCH] Adding more unit test to reach 80% Coverage

---
 sdk-actors/pom.xml                            |  9 ++-----
 sdk/pom.xml                                   |  9 ++-----
 .../io/dapr/client/DaprClientBuilderTest.java | 27 +++++++++++++++++++
 .../io/dapr/client/DaprHttpBuilderTest.java   | 19 +++++++++++++
 4 files changed, 50 insertions(+), 14 deletions(-)
 create mode 100644 sdk/src/test/java/io/dapr/client/DaprClientBuilderTest.java
 create mode 100644 sdk/src/test/java/io/dapr/client/DaprHttpBuilderTest.java

diff --git a/sdk-actors/pom.xml b/sdk-actors/pom.xml
index 0e0cf570a7..7f6868eca9 100644
--- a/sdk-actors/pom.xml
+++ b/sdk-actors/pom.xml
@@ -138,17 +138,12 @@
             <configuration>
               <rules>
                 <rule>
-                  <element>CLASS</element>
+                  <element>BUNDLE</element>
                   <limits>
                    <limit>
                       <counter>LINE</counter>
                       <value>COVEREDRATIO</value>
-                      <minimum>0.00</minimum>
-                    </limit>
-                    <limit>
-                      <counter>BRANCH</counter>
-                      <value>COVEREDRATIO</value>
-                      <minimum>0.00</minimum>
+                      <minimum>80%</minimum>
                     </limit>
                   </limits>
                 </rule>
diff --git a/sdk/pom.xml b/sdk/pom.xml
index ac83d76628..e209c6cc4c 100644
--- a/sdk/pom.xml
+++ b/sdk/pom.xml
@@ -152,17 +152,12 @@
             <configuration>
               <rules>
                 <rule>
-                  <element>CLASS</element>
+                  <element>BUNDLE</element>
                   <limits>
                    <limit>
                       <counter>LINE</counter>
                       <value>COVEREDRATIO</value>
-                      <minimum>0.00</minimum>
-                    </limit>
-                    <limit>
-                      <counter>BRANCH</counter>
-                      <value>COVEREDRATIO</value>
-                      <minimum>0.00</minimum>
+                      <minimum>80%</minimum>
                     </limit>
                   </limits>
                   <excludes>
diff --git a/sdk/src/test/java/io/dapr/client/DaprClientBuilderTest.java b/sdk/src/test/java/io/dapr/client/DaprClientBuilderTest.java
new file mode 100644
index 0000000000..60dc6ac82b
--- /dev/null
+++ b/sdk/src/test/java/io/dapr/client/DaprClientBuilderTest.java
@@ -0,0 +1,27 @@
+package io.dapr.client;
+
+import io.dapr.serializer.DaprObjectSerializer;
+import org.junit.Test;
+import static org.junit.jupiter.api.Assertions.*;
+import static org.mockito.Mockito.mock;
+
+public class DaprClientBuilderTest {
+
+  @Test
+  public void build() {
+    DaprObjectSerializer objectSerializer = mock(DaprObjectSerializer.class);
+    DaprObjectSerializer stateSerializer = mock(DaprObjectSerializer.class);
+    DaprClientBuilder daprClientBuilder = new DaprClientBuilder(objectSerializer, stateSerializer);
+    DaprClient daprClient = daprClientBuilder.build();
+    assertNotNull(daprClient);
+  }
+
+  @Test(expected = IllegalArgumentException.class)
+  public void buildException() {
+    DaprClientBuilder daprClientBuilder = new DaprClientBuilder(null,null);
+    DaprClient daprClient = daprClientBuilder.build();
+    assertNotNull(daprClient);
+  }
+
+
+}
\ No newline at end of file
diff --git a/sdk/src/test/java/io/dapr/client/DaprHttpBuilderTest.java b/sdk/src/test/java/io/dapr/client/DaprHttpBuilderTest.java
new file mode 100644
index 0000000000..9a26db2fc7
--- /dev/null
+++ b/sdk/src/test/java/io/dapr/client/DaprHttpBuilderTest.java
@@ -0,0 +1,19 @@
+package io.dapr.client;
+
+import org.junit.Test;
+import java.time.Duration;
+import static org.junit.Assert.assertNotNull;
+import static org.mockito.Mockito.mock;
+
+public class DaprHttpBuilderTest {
+
+  @Test
+  public void withReadTimeout() throws Exception {
+    DaprHttpBuilder daprHttpBuilder = new DaprHttpBuilder();
+    Duration duration = mock(Duration.class);
+    daprHttpBuilder.build();
+    DaprHttpBuilder dapr = daprHttpBuilder.withReadTimeout(duration);
+    assertNotNull(dapr);
+  }
+
+}
\ No newline at end of file