Do not double decode query string (closes #73)

Author: BenFradet

src/main/scala/com/snowplowanalytics/refererparser/Parser.scala

@@ -91,7 +91,7 @@ class Parser private[refererparser] (referers: Map[String, RefererLookup]) {
     val scheme = refererUri.getScheme
     val host   = refererUri.getHost
     val path   = refererUri.getPath
-    val query  = Option(refererUri.getQuery)
+    val query  = Option(refererUri.getRawQuery)
 
     val validUri = (scheme == "http" || scheme == "https") && host != null && path != null
 

src/test/scala/com/snowplowanalytics/refererparser/ParseTest.scala

@@ -25,7 +25,8 @@ class ParseTest extends Specification with DataTables { def is =
   s2"""This is a specification to test the parse function
     usual               $e1
     unknown referer uri $e2
-    false positives     $e3"""
+    false positives     $e3
+    no double decoding  $e4"""
 
   val resource = getClass.getResource("/referers.json").getPath
   val ioParser = CreateParser[IO].create(resource).unsafeRunSync().fold(throw _, identity)
@@ -103,6 +104,14 @@ class ParseTest extends Specification with DataTables { def is =
         evalParser.parse(refererUri, pageHost) must_== Some(genExpected(medium, source, term))
     }
 
+  def e4 =
+    "SPEC NAME" || "REFERER URI"           	                      | "REFERER MEDIUM" | "REFERER SOURCE" | "REFERER TERM"     |
+    "% encoded" !! "https://www.google.com/search?q=keyword+1%25" ! Medium.Search    ! Some("Google")   ! Some("keyword 1%") |> {
+      (_, refererUri, medium, source, term) =>
+        ioParser.parse(refererUri, pageHost) must_== Some(genExpected(medium, source, term))
+        evalParser.parse(refererUri, pageHost) must_== Some(genExpected(medium, source, term))
+    }
+
   def genExpected(medium: Medium, source: Option[String], term: Option[String]) = medium match {
     case UnknownMedium  => UnknownReferer(UnknownMedium)
     case SearchMedium   => SearchReferer(SearchMedium, source.get, term)