From cf843811272f086b85557dee4c5977e8ae20f1cb Mon Sep 17 00:00:00 2001 From: roger Date: Tue, 30 Nov 2010 08:56:05 +0100 Subject: [PATCH] Added URLDecoding to verifier --- src/main/java/org/scribe/model/Verifier.java | 5 ++++- .../java/org/scribe/model/VerifierTest.java | 21 +++++++++++++++++++ 2 files changed, 25 insertions(+), 1 deletion(-) create mode 100644 src/test/java/org/scribe/model/VerifierTest.java diff --git a/src/main/java/org/scribe/model/Verifier.java b/src/main/java/org/scribe/model/Verifier.java index 2a98fd6..f1dee62 100644 --- a/src/main/java/org/scribe/model/Verifier.java +++ b/src/main/java/org/scribe/model/Verifier.java @@ -1,5 +1,7 @@ package org.scribe.model; +import org.scribe.utils.*; + /** * Represents an OAuth verifier code. * @@ -7,6 +9,7 @@ package org.scribe.model; */ public class Verifier { + private final String value; /** @@ -16,7 +19,7 @@ public class Verifier */ public Verifier(String value) { - this.value = value; + this.value = URLUtils.percentDecode(value); } public String getValue() diff --git a/src/test/java/org/scribe/model/VerifierTest.java b/src/test/java/org/scribe/model/VerifierTest.java new file mode 100644 index 0000000..509ace8 --- /dev/null +++ b/src/test/java/org/scribe/model/VerifierTest.java @@ -0,0 +1,21 @@ +package org.scribe.model; + +import static org.junit.Assert.*; + +import org.junit.*; + +public class VerifierTest +{ + @Test + public void shouldParse() + { + Verifier verifier = new Verifier("p8k%2BGIjIL9PblXq%2BpH6LmT9l"); + assertEquals(verifier.getValue(), "p8k+GIjIL9PblXq+pH6LmT9l"); + } + + @Test(expected = IllegalArgumentException.class) + public void shouldThrowErrorForNullString() + { + new Verifier(null); + } +}