feat(fs): add .path(path).read_link()

src/fs/path.rs

@@ -229,25 +229,6 @@ impl<T: PathType> PathReal<T> {
         std::fs::rename(self.as_pathbuf(), dest.as_pathbuf()).map_err(Error::Io)
     }
 
-    /// Returns the canonical, absolute form of the path with all intermediate
-    /// components normalized and symbolic links resolved.
-    ///
-    /// ```
-    /// # use kxio::fs::Result;
-    /// # fn main() -> Result<()> {
-    /// let fs = kxio::fs::temp()?;
-    /// let path = fs.base().join("foo");
-    /// #    fs.dir(&path).create()?;
-    /// let dir = fs.path(&path);
-    /// let canonical = dir.canonicalize()?;
-    /// #    Ok(())
-    /// # }
-    /// ```
-    pub fn canonicalize(&self) -> Result<PathBuf> {
-        self.check_error()?;
-        self.as_pathbuf().canonicalize().map_err(Error::Io)
-    }
-
     /// Returns the metadata for a path.
     ///
     /// Wrapper for [std::fs::metadata]
@@ -306,6 +287,25 @@ impl<T: PathType> PathReal<T> {
         Ok(self.as_pathbuf().is_symlink())
     }
 
+    /// Returns the canonical, absolute form of the path with all intermediate
+    /// components normalized and symbolic links resolved.
+    ///
+    /// ```
+    /// # use kxio::fs::Result;
+    /// # fn main() -> Result<()> {
+    /// let fs = kxio::fs::temp()?;
+    /// let path = fs.base().join("foo");
+    /// #    fs.dir(&path).create()?;
+    /// let dir = fs.path(&path);
+    /// let canonical = dir.canonicalize()?;
+    /// #    Ok(())
+    /// # }
+    /// ```
+    pub fn canonicalize(&self) -> Result<PathBuf> {
+        self.check_error()?;
+        self.as_pathbuf().canonicalize().map_err(Error::Io)
+    }
+
     /// Returns the metadata for a path without following symlinks.
     ///
     /// Wrapper for [std::fs::symlink_metadata]

tests/fs.rs

@@ -785,14 +785,14 @@ mod canonicalize {
     #[test]
     fn should_resolve_symlinks() -> TestResult {
         let fs = fs::temp().expect("temp fs");
+
         let file_path = fs.base().join("foo");
         let file = fs.file(&file_path);
         file.write("bar").expect("write");
 
         let link_path = fs.base().join("link");
         let link = fs.path(&link_path);
-        file.soft_link(&link).expect("soft_link");
-
+        file.soft_link(&link).expect("create");
         let canonical = link.canonicalize().expect("canonicalize");
         // macos puts all temp files under /private
         let canonical = Path::new("/").join(canonical.strip_prefix("/private").unwrap());