Merge pull request #3 from ryan-williams/uri

use URI repr as toString

Author: ryan-williams

build.sbt

@@ -1,4 +1,4 @@
 name := "paths"
-version := "1.0.0"
+version := "1.0.1"
 deps += libs.value('commons_io)
 addScala212

project/plugins.sbt

@@ -1 +1 @@
-addSbtPlugin("org.hammerlab" % "sbt-parent" % "1.7.2")
+addSbtPlugin("org.hammerlab" % "sbt-parent" % "1.7.3")

src/main/scala/org/hammerlab/paths/Path.scala

@@ -13,11 +13,11 @@ class Path(val path: JPath)
   extends AnyVal
     with Serializable {
 
-  override def toString: String = path.toString
+  override def toString: String = uri.toString
 
   def uri: URI = path.toUri
 
-  def extension: String = getExtension(path.toString)
+  def extension: String = getExtension(toString)
 
   def exists: Boolean = Files.exists(path)
 
@@ -80,7 +80,7 @@ class Path(val path: JPath)
   /**
    * Append `suffix` to the basename of this [[Path]].
    */
-  def +(suffix: String): Path = Path(path.toString + suffix)
+  def +(suffix: String): Path = Path(toString + suffix)
 
   def /(basename: String): Path = Path(path.resolve(basename))
 
@@ -108,7 +108,15 @@ class Path(val path: JPath)
 
 object Path {
   def apply(path: JPath): Path = new Path(path)
-  def apply(pathStr: String): Path = new Path(Paths.get(pathStr))
+
+  def apply(pathStr: String): Path = {
+    val uri = new URI(pathStr)
+    if (uri.getScheme == null)
+      new Path(Paths.get(pathStr))
+    else
+      Path(uri)
+  }
+
   def apply(uri: URI): Path = new Path(Paths.get(uri))
 
   implicit def toJava(path: Path): JPath = path.path

src/test/scala/org/hammerlab/paths/PathTest.scala

@@ -12,7 +12,7 @@ class PathTest
   test("extensions") {
     "abc.def".extension should be("def")
     "/abc/def.gh.ij".extension should be("ij")
-    "hdfs://foo/bar.baz".extension should be("baz")
+    "file:///foo/bar.baz".extension should be("baz")
   }
 
   test("removals") {