add depth, uri-encoding handling

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

@@ -1,7 +1,7 @@
 package org.hammerlab.paths
 
 import java.io.{ InputStream, ObjectStreamException, OutputStream, PrintWriter }
-import java.net.URI
+import java.net.{ URI, URISyntaxException }
 import java.nio.file.Files.{ newDirectoryStream, newInputStream, newOutputStream, readAllBytes }
 import java.nio.file.{ DirectoryStream, Files, Paths, Path ⇒ JPath }
 
@@ -35,6 +35,14 @@ case class Path(path: JPath) {
   def parent: Path = parentOpt.getOrElse(this)
   def parentOpt: Option[Path] = Option(path.getParent).map(Path(_))
 
+  def depth: Int =
+    if (path.isAbsolute)
+      parentOpt
+        .map(_.depth + 1)
+        .getOrElse(0)
+    else
+      Path(path.toAbsolutePath).depth
+
   def basename: String = path.getFileName.toString
 
   def size: Long = Files.size(path)
@@ -134,13 +142,17 @@ object Path {
     Paths.get(uri)
   }
 
-  def apply(pathStr: String): Path = {
-    val uri = new URI(pathStr)
-    if (uri.getScheme == null)
-      new Path(get(pathStr))
-    else
-      Path(uri)
-  }
+  def apply(pathStr: String): Path =
+    try {
+      val uri = new URI(pathStr)
+      if (uri.getScheme == null)
+        new Path(get(pathStr))
+      else
+        Path(uri)
+    } catch {
+      case _: URISyntaxException ⇒
+        new Path(get(pathStr))
+    }
 
   def apply(uri: URI): Path = Path(get(uri))
 

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

@@ -176,4 +176,25 @@ class PathTest
     Path("a").parent should be(Path("a"))
     Path("a/b").parent should be(Path("a"))
   }
+
+  val testDirDepth = Path(".").depth
+
+  test("depth") {
+    Path("/").depth should be(0)
+    Path("/a").depth should be(1)
+    Path("/a/b").depth should be(2)
+
+    Path("a").depth should be(testDirDepth)
+    Path("a/b").depth should be(testDirDepth + 1)
+  }
+
+  test("with spaces") {
+    Path("a b").depth should be(testDirDepth)
+    Path("a b/c d").depth should be(testDirDepth + 1)
+    Path("/a b").depth should be(1)
+    Path("/a b/c d").depth should be(2)
+
+    // Invalid characters for URI, interpret as local file
+    Path("hdfs:///a b").uri.getScheme should be("file")
+  }
 }