Fix push/pop for shell manager

src/commands/classified.rs

@@ -140,19 +140,21 @@ impl InternalCommand {
                     }
                     CommandAction::Exit => std::process::exit(0),
                     CommandAction::EnterValueShell(value) => {
-                        context.shell_manager.push(Box::new(ValueShell::new(value)));
+                        context
+                            .shell_manager
+                            .insert_at_current(Box::new(ValueShell::new(value)));
                     }
                     CommandAction::EnterShell(location) => {
                         let path = std::path::Path::new(&location);
 
                         if path.is_dir() {
                             // If it's a directory, add a new filesystem shell
-                            context
-                                .shell_manager
-                                .push(Box::new(FilesystemShell::with_location(
+                            context.shell_manager.insert_at_current(Box::new(
+                                FilesystemShell::with_location(
                                     location,
                                     context.registry().clone(),
-                                )?));
+                                )?,
+                            ));
                         } else {
                             // If it's a file, attempt to open the file as a value and enter it
                             let cwd = context.shell_manager.path();
@@ -180,11 +182,13 @@ impl InternalCommand {
                                         Span::unknown(),
                                     )?;
 
-                                    context.shell_manager.push(Box::new(ValueShell::new(value)));
-                                }
-                                value => context
+                                    context
                                         .shell_manager
-                                    .push(Box::new(ValueShell::new(value.tagged(contents_tag)))),
+                                        .insert_at_current(Box::new(ValueShell::new(value)));
+                                }
+                                value => context.shell_manager.insert_at_current(Box::new(
+                                    ValueShell::new(value.tagged(contents_tag)),
+                                )),
                             }
                         }
                     }
@@ -195,7 +199,7 @@ impl InternalCommand {
                         context.shell_manager.next();
                     }
                     CommandAction::LeaveShell => {
-                        context.shell_manager.pop();
+                        context.shell_manager.remove_at_current();
                         if context.shell_manager.is_empty() {
                             std::process::exit(0);
                         }

src/shell/shell_manager.rs

@@ -23,19 +23,30 @@ impl ShellManager {
         })
     }
 
-    pub fn push(&mut self, shell: Box<dyn Shell + Send>) {
+    pub fn insert_at_current(&mut self, shell: Box<dyn Shell + Send>) {
         self.shells.lock().unwrap().push(shell);
         self.current_shell = self.shells.lock().unwrap().len() - 1;
         self.set_path(self.path());
     }
 
-    pub fn pop(&mut self) {
-        self.shells.lock().unwrap().pop();
-        let new_len = self.shells.lock().unwrap().len();
+    pub fn remove_at_current(&mut self) {
+        {
+            let mut shells = self.shells.lock().unwrap();
+            if shells.len() > 0 {
+                if self.current_shell == shells.len() - 1 {
+                    shells.pop();
+                    let new_len = shells.len();
                     if new_len > 0 {
                         self.current_shell = new_len - 1;
-            self.set_path(self.path());
+                    } else {
+                        return;
                     }
+                } else {
+                    shells.remove(self.current_shell);
+                }
+            }
+        }
+        self.set_path(self.path());
     }
 
     pub fn is_empty(&self) -> bool {

tests/command_enter_test.rs

@@ -45,9 +45,11 @@ fn knows_the_filesystems_entered() {
             n
             mv andres.nu ../expected/andres.nu
             exit
+            n
             cd ..
             rm red_pill --recursive
             exit
+            n
             rm blue_pill --recursive
             exit
         "#