Custom canonicalize method on Filesystem Shell. (#1485)

* Custom canonicalize method for FilesystemShell.

* Use custom canonicalize method.
Fixed missing import.

* Move function body to already impl body.

* Create test that aims to resolve.

crates/nu-cli/src/shell/filesystem_shell.rs

@@ -14,7 +14,7 @@ use nu_parser::ExpandContext;
 use nu_protocol::{Primitive, ReturnSuccess, UntaggedValue};
 use rustyline::completion::FilenameCompleter;
 use rustyline::hint::{Hinter, HistoryHinter};
-use std::path::{Path, PathBuf};
+use std::path::{Component, Path, PathBuf};
 use std::sync::atomic::Ordering;
 use trash as SendToTrash;
 
@@ -78,6 +78,29 @@ impl FilesystemShell {
             hinter: HistoryHinter {},
         }
     }
+
+    fn canonicalize(&self, path: impl AsRef<Path>) -> std::io::Result<PathBuf> {
+        let path = if path.as_ref().is_relative() {
+            let components = path.as_ref().components();
+            let mut result = PathBuf::from(self.path());
+            for component in components {
+                match component {
+                    Component::CurDir => { /* ignore current dir */ }
+                    Component::ParentDir => {
+                        result.pop();
+                    }
+                    Component::Normal(normal) => result.push(normal),
+                    _ => {}
+                }
+            }
+
+            result
+        } else {
+            path.as_ref().into()
+        };
+
+        dunce::canonicalize(path)
+    }
 }
 
 impl Shell for FilesystemShell {
@@ -201,11 +224,10 @@ impl Shell for FilesystemShell {
             Some(v) => {
                 let target = v.as_path()?;
 
-                if PathBuf::from("-") == target {
+                if target == Path::new("-") {
                     PathBuf::from(&self.last_path)
                 } else {
-                    let path = PathBuf::from(self.path());
-                    let path = dunce::canonicalize(path.join(&target)).map_err(|_| {
+                    let path = self.canonicalize(target).map_err(|_| {
                         ShellError::labeled_error(
                             "Cannot change to directory",
                             "directory not found",