#8 Destination fixed to respect fragment

yegor256 · Apr 17, 2016 · c22c880 · c22c880
1 parent dc5e4dd
commit c22c880
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/src/main/java/io/jare/tk/Destination.java b/src/main/java/io/jare/tk/Destination.java
@@ -86,6 +86,9 @@ public String path() throws HttpException {
         if (this.uri.getQuery() != null) {
             path.append('?').append(this.uri.getQuery());
         }
+        if (this.uri.getFragment() != null) {
+            path.append('#').append(this.uri.getFragment());
+        }
         return path.toString();
     }
 

diff --git a/src/test/java/io/jare/tk/DestinationTest.java b/src/test/java/io/jare/tk/DestinationTest.java
@@ -72,4 +72,16 @@ public void buildsPathWithParams() throws Exception {
         );
     }
 
+    /**
+     * Destination can build path with fragment.
+     * @throws Exception If some problem inside
+     */
+    @Test
+    public void buildsPathWithFragment() throws Exception {
+        MatcherAssert.assertThat(
+            new Destination(new URI("http://www.google.com/a?x=1#t")).path(),
+            Matchers.equalTo("/a?x=1#t")
+        );
+    }
+
 }