Merge pull request #28 from davidanthoff/fragment-query-support-in-uri

Add support for query and fragment parts to URI conversion

Author: rofinn

src/uri.jl

@@ -1,4 +1,4 @@
-function URIParser.URI(p::AbstractPath)
+function URIParser.URI(p::AbstractPath; query="", fragment="")
     if isempty(root(p))
         throw(ArgumentError("$p is not an absolute path"))
     end
@@ -16,5 +16,5 @@ function URIParser.URI(p::AbstractPath)
         print(b, URIParser.escape(p.parts[i]))
     end
 
-    return URIParser.URI(String(take!(b)))
+    return URIParser.URI(URIParser.URI(String(take!(b))); query=query, fragment=fragment)
 end

test/test_uri.jl

@@ -6,4 +6,5 @@ using FilePaths
     @test string(URI(p"/foo foo/bar")) == "file:///foo%20foo/bar"
     @test_throws ArgumentError URI(p"foo/bar")
     @test string(URI(WindowsPath("C:\\foo\\bar"))) == "file:///C:/foo/bar"
+    @test string(URI(p"/foo/bar", query="querypart", fragment="fragmentpart")) == "file:///foo/bar?querypart#fragmentpart"
 end