add getToken() to obtain string representation of the token

lbalmaceda · lbalmaceda · commit ff49aea76e25 · 2016-11-30T17:33:27.000-03:00
diff --git a/.travis.yml b/.travis.yml
@@ -15,7 +15,7 @@ script:
 after_success:
   - bash <(curl -s https://codecov.io/bash)
 after_failure:
-  - cat file:///home/travis/build/auth0/java-jwt/lib/build/reports/tests/index.html
+  - cat $HOME/travis/build/auth0/java-jwt/lib/build/reports/tests/index.html
 branches:
   only:
   - v3
diff --git a/lib/src/main/java/com/auth0/jwt/JWT.java b/lib/src/main/java/com/auth0/jwt/JWT.java
@@ -117,4 +117,9 @@ public String getContentType() {
     public String getKeyId() {
         return jwt.getKeyId();
     }
+
+    @Override
+    public String getToken() {
+        return jwt.getToken();
+    }
 }
diff --git a/lib/src/main/java/com/auth0/jwt/JWTDecoder.java b/lib/src/main/java/com/auth0/jwt/JWTDecoder.java
@@ -18,11 +18,13 @@
 @SuppressWarnings("WeakerAccess")
 final class JWTDecoder implements JWT {
 
+    private final String token;
     private Header header;
     private Payload payload;
     private String signature;
 
     private JWTDecoder(String jwt) throws JWTDecodeException {
+        this.token = jwt;
         parseToken(jwt);
     }
 
@@ -124,4 +126,8 @@ public String getSignature() {
         return signature;
     }
 
+    @Override
+    public String getToken() {
+        return token;
+    }
 }
diff --git a/lib/src/main/java/com/auth0/jwt/interfaces/JWT.java b/lib/src/main/java/com/auth0/jwt/interfaces/JWT.java
@@ -4,4 +4,11 @@
  * The JWT class represents a Json Web Token.
  */
 public interface JWT extends Payload, Header, Signature {
+
+    /**
+     * Returns the String representation of the token.
+     *
+     * @return the String representation of the token.
+     */
+    String getToken();
 }
diff --git a/lib/src/test/java/com/auth0/jwt/JWTDecoderTest.java b/lib/src/test/java/com/auth0/jwt/JWTDecoderTest.java
@@ -7,7 +7,6 @@
 import org.apache.commons.codec.binary.Base64;
 import org.hamcrest.collection.IsCollectionWithSize;
 import org.hamcrest.core.IsCollectionContaining;
-import org.junit.Ignore;
 import org.junit.Rule;
 import org.junit.Test;
 import org.junit.rules.ExpectedException;
@@ -61,14 +60,13 @@ public void shouldThrowIfHeaderHasInvalidJSONFormat() throws Exception {
         customJWT(invalidJson, validJson, "signature");
     }
 
-    // toString
+    // getToken
     @Test
-    @Ignore("Pending implementation")
     public void shouldGetStringToken() throws Exception {
         JWT jwt = JWTDecoder.decode("eyJhbGciOiJIUzI1NiJ9.e30.XmNK3GpH3Ys_7wsYBfq4C3M6goz71I7dTgUkuIa5lyQ");
         assertThat(jwt, is(notNullValue()));
-        assertThat(jwt.toString(), is(notNullValue()));
-        assertThat(jwt.toString(), is("eyJhbGciOiJIUzI1NiJ9.e30.XmNK3GpH3Ys_7wsYBfq4C3M6goz71I7dTgUkuIa5lyQ"));
+        assertThat(jwt.getToken(), is(notNullValue()));
+        assertThat(jwt.getToken(), is("eyJhbGciOiJIUzI1NiJ9.e30.XmNK3GpH3Ys_7wsYBfq4C3M6goz71I7dTgUkuIa5lyQ"));
     }
 
     // Parts
diff --git a/lib/src/test/java/com/auth0/jwt/JWTTest.java b/lib/src/test/java/com/auth0/jwt/JWTTest.java
@@ -42,6 +42,15 @@ public void shouldDecodeAStringToken() throws Exception {
         assertThat(jwt, is(notNullValue()));
     }
 
+    // getToken
+    @Test
+    public void shouldGetStringToken() throws Exception {
+        JWT jwt = JWT.decode("eyJhbGciOiJIUzI1NiJ9.e30.XmNK3GpH3Ys_7wsYBfq4C3M6goz71I7dTgUkuIa5lyQ");
+        assertThat(jwt, is(notNullValue()));
+        assertThat(jwt.getToken(), is(notNullValue()));
+        assertThat(jwt.getToken(), is("eyJhbGciOiJIUzI1NiJ9.e30.XmNK3GpH3Ys_7wsYBfq4C3M6goz71I7dTgUkuIa5lyQ"));
+    }
+
 
     // Verify
 

Original file line number	Diff line number	Diff line change
`@@ -117,4 +117,9 @@ public String getContentType() {`
`117`	`117`	`public String getKeyId() {`
`118`	`118`	`return jwt.getKeyId();`
`119`	`119`	`}`
	`120`	`+`
	`121`	`+ @Override`
	`122`	`+ public String getToken() {`
	`123`	`+ return jwt.getToken();`
	`124`	`+ }`
`120`	`125`	`}`
Original file line number	Diff line number	Diff line change
`@@ -18,11 +18,13 @@`
`18`	`18`	`@SuppressWarnings("WeakerAccess")`
`19`	`19`	`final class JWTDecoder implements JWT {`
`20`	`20`
	`21`	`+ private final String token;`
`21`	`22`	`private Header header;`
`22`	`23`	`private Payload payload;`
`23`	`24`	`private String signature;`
`24`	`25`
`25`	`26`	`private JWTDecoder(String jwt) throws JWTDecodeException {`
	`27`	`+ this.token = jwt;`
`26`	`28`	`parseToken(jwt);`
`27`	`29`	`}`
`28`	`30`
`@@ -124,4 +126,8 @@ public String getSignature() {`
`124`	`126`	`return signature;`
`125`	`127`	`}`
`126`	`128`
	`129`	`+ @Override`
	`130`	`+ public String getToken() {`
	`131`	`+ return token;`
	`132`	`+ }`
`127`	`133`	`}`