Added URLDecoding to verifier

roger · fernandezpablo85 · commit cf843811272f · 2010-11-30T14:49:47.000-03:00
diff --git a/src/main/java/org/scribe/model/Verifier.java b/src/main/java/org/scribe/model/Verifier.java
@@ -1,12 +1,15 @@
 package org.scribe.model;
 
+import org.scribe.utils.*;
+
 /**
  * Represents an OAuth verifier code.
  * 
  * @author Pablo Fernandez
  */
 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
@@ -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);
+  }
+}

Original file line number	Diff line number	Diff line change
`@@ -1,12 +1,15 @@`
`1`	`1`	`package org.scribe.model;`
`2`	`2`
	`3`	`+import org.scribe.utils.*;`
	`4`	`+`
`3`	`5`	`/**`
`4`	`6`	`* Represents an OAuth verifier code.`
`5`	`7`	`*`
`6`	`8`	`* @author Pablo Fernandez`
`7`	`9`	`*/`
`8`	`10`	`public class Verifier`
`9`	`11`	`{`
	`12`	`+`
`10`	`13`	`private final String value;`
`11`	`14`
`12`	`15`	`/**`
`@@ -16,7 +19,7 @@ public class Verifier`
`16`	`19`	`*/`
`17`	`20`	`public Verifier(String value)`
`18`	`21`	`{`
`19`		`- this.value = value;`
	`22`	`+ this.value = URLUtils.percentDecode(value);`
`20`	`23`	`}`
`21`	`24`
`22`	`25`	`public String getValue()`