Fix old_stack behavior on error

lib/pry/commands/cd.rb

@@ -20,9 +20,9 @@ class Pry
     BANNER
 
     def process
-      stack, old_stack = context_from_object_path(arg_string, _pry_, state.old_stack||[])
+      state.old_stack ||= []
-      state.old_stack = old_stack
+      stack, state.old_stack = context_from_object_path(arg_string, _pry_, state.old_stack)
-      _pry_.binding_stack = stack unless stack.nil?
+      _pry_.binding_stack = stack if stack
     end
   end
 end

test/test_default_commands/test_cd.rb

@@ -20,9 +20,9 @@ describe 'Pry::DefaultCommands::Cd' do
         eval '_pry_.command_state["cd"]'
       end
 
-      # def old_stack
+      def old_stack
-      #   eval '_pry_.command_state["cd"].old_stack.dup'
+        eval '_pry_.command_state["cd"].old_stack.dup'
-      # end
+      end
     end
   end
 
@@ -42,20 +42,20 @@ describe 'Pry::DefaultCommands::Cd' do
       end
     end
 
-    # describe 'when an error was raised' do
+    describe 'when an error was raised' do
-    #   it 'should not toggle and should keep correct stacks' do
+      it 'should not toggle and should keep correct stacks' do
-    #     proc {
+        proc {
-    #       @t.eval 'cd @'
+          @t.eval 'cd @'
-    #     }.should.raise(Pry::CommandError)
+        }.should.raise(Pry::CommandError)
 
-    #     @t.old_stack.should == []
+        @t.old_stack.should == []
-    #     @t.assert_binding_stack [@o]
+        @t.assert_binding_stack [@o]
 
-    #     @t.eval 'cd -'
+        @t.eval 'cd -'
-    #     @t.old_stack.should == []
+        @t.old_stack.should == []
-    #     @t.assert_binding_stack [@o]
+        @t.assert_binding_stack [@o]
-    #   end
+      end
-    # end
+    end
 
     describe 'when using simple cd syntax' do
       it 'should toggle' do