coverage improvements. Removed unnecessary line from DefaultClaims

lookuptable · Apr 17, 2016 · 29f980c · 29f980c
1 parent e392524
commit 29f980c
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 7 deletions.
diff --git a/pom.xml b/pom.xml
@@ -302,11 +302,6 @@
                                 <lineRate>96</lineRate>
                                 <branchRate>100</branchRate>
                             </regex>
-                            <regex>
-                                <pattern>io.jsonwebtoken.impl.DefaultJwtBuilder</pattern>
-                                <lineRate>100</lineRate>
-                                <branchRate>95</branchRate>
-                            </regex>
                             <regex>
                                 <pattern>io.jsonwebtoken.impl.DefaultJwtParser</pattern>
                                 <lineRate>100</lineRate>

diff --git a/src/main/java/io/jsonwebtoken/impl/DefaultClaims.java b/src/main/java/io/jsonwebtoken/impl/DefaultClaims.java
@@ -23,8 +23,6 @@
 
 public class DefaultClaims extends JwtMap implements Claims {
 
-    private static final DefaultClaims INSTANCE = new DefaultClaims();
-
     public DefaultClaims() {
         super();
     }

diff --git a/src/test/groovy/io/jsonwebtoken/impl/DefaultJwtBuilderTest.groovy b/src/test/groovy/io/jsonwebtoken/impl/DefaultJwtBuilderTest.groovy
@@ -279,4 +279,25 @@ class DefaultJwtBuilderTest {
         assertNull b.claims
     }
 
+    @Test
+    void testSetNullExpirationWithNullClaims() {
+        def b = new DefaultJwtBuilder()
+        b.setExpiration(null)
+        assertNull b.claims
+    }
+
+    @Test
+    void testSetNullNotBeforeWithNullClaims() {
+        def b = new DefaultJwtBuilder()
+        b.setNotBefore(null)
+        assertNull b.claims
+    }
+
+    @Test
+    void testSetNullIssuedAtWithNullClaims() {
+        def b = new DefaultJwtBuilder()
+        b.setIssuedAt(null)
+        assertNull b.claims
+    }
+
 }