byebug 1.8.2 → 2.0.0

data/test/method_test.rb

@@ -1,7 +1,7 @@
 require_relative 'test_helper'
 
 class TestMethod < TestDsl::TestCase
-  temporary_change_hash Byebug::Command.settings, :autolist, 0
+  temporary_change_hash Byebug.settings, :autolist, 0
 
   describe 'show instance method of a class' do
     it 'must show using full command name' do
@@ -58,13 +58,4 @@ class TestMethod < TestDsl::TestCase
       check_output_includes '@a = "b"', '@c = "d"'
     end
   end
-
-  describe 'Post Mortem' do
-    it 'must work in post-mortem mode' do
-      enter 'cont', 'm i self'
-      debug_file 'post_mortem'
-      check_output_includes /to_s/
-    end
-  end
-
 end

data/test/post_mortem_test.rb

@@ -2,24 +2,257 @@ require_relative 'test_helper'
 
 class TestPostMortem < TestDsl::TestCase
 
-  it 'must enter into post mortem mode' do
-    enter 'cont'
-    debug_file('post_mortem') { Byebug.post_mortem?.must_equal true }
-  end
+  describe 'Features' do
+    it 'must enter into post-mortem mode' do
+      enter 'cont'
+      debug_file('post_mortem') { Byebug.post_mortem?.must_equal true }
+    end
+
+    it 'must stop at the correct line' do
+      enter 'cont'
+      debug_file('post_mortem') { $state.line.must_equal 8 }
+    end
+
+    it 'must exit from post-mortem mode after stepping command' do
+      enter 'cont', 'break 12', 'cont'
+      debug_file('post_mortem') { Byebug.post_mortem?.must_equal false }
+    end
 
-  it 'must stop at the correct line' do
-    enter 'cont'
-    debug_file('post_mortem') { $state.line.must_equal 8 }
+    it 'must save the raised exception' do
+      enter 'cont'
+      debug_file('post_mortem') {
+        Byebug.last_exception.must_be_kind_of RuntimeError }
+    end
   end
 
-  it 'must exit from post mortem mode after stepping command' do
-    enter 'cont', 'break 12', 'cont'
-    debug_file('post_mortem') { Byebug.post_mortem?.must_equal false }
+  describe 'Unavailable commands' do
+    temporary_change_hash Byebug.settings, :autoeval, false
+
+    describe 'step' do
+      it 'must not work in post-mortem mode' do
+        enter 'cont', 'step'
+        debug_file 'post_mortem'
+        check_error_includes 'Unknown command: "step".  Try "help".'
+      end
+    end
+
+    describe 'next' do
+      it 'must not work in post-mortem mode' do
+        enter 'cont', 'next'
+        debug_file 'post_mortem'
+        check_error_includes 'Unknown command: "next".  Try "help".'
+      end
+    end
+
+    describe 'finish' do
+      it 'must not work in post-mortem mode' do
+        enter 'cont', 'finish'
+        debug_file 'post_mortem'
+        check_error_includes 'Unknown command: "finish".  Try "help".'
+      end
+    end
   end
 
-  it 'must save the raised exception' do
-    enter 'cont'
-    debug_file('post_mortem') {
-      Byebug.last_exception.must_be_kind_of RuntimeError }
+  describe 'Available commands' do
+    before { @tst_file = fullpath('post_mortem') }
+
+    describe 'restart' do
+      it 'must work in post-mortem mode' do
+        must_restart
+        enter 'cont', 'restart'
+        debug_file 'post_mortem'
+      end
+    end
+
+    describe 'display' do
+      it 'must be able to set display expressions in post-mortem mode' do
+        enter 'cont', 'display 2 + 2'
+        debug_file 'post_mortem'
+        check_output_includes '1:', '2 + 2 = 4'
+      end
+    end
+
+    describe 'frame' do
+      it 'must work in post-mortem mode' do
+        enter 'cont', 'frame'
+        debug_file('post_mortem') { $state.line.must_equal 8 }
+        check_output_includes \
+          /--> #0  block in CatchExample\.a\s+at #{@tst_file}:8/
+      end
+    end
+
+    describe 'condition' do
+      it 'must be able to set conditions in post-mortem mode' do
+        enter 'cont', 'break 12',
+              ->{ "cond #{Byebug.breakpoints.first.id} true" }, 'cont'
+        debug_file('post_mortem') { $state.line.must_equal 12 }
+      end
+    end
+
+    describe 'break' do
+      it 'must be able to set breakpoints in post-mortem mode' do
+        enter 'cont', 'break 12', 'cont'
+        debug_file('post_mortem') { $state.line.must_equal 12 }
+      end
+    end
+
+    describe 'exit' do
+      it 'must work in post-mortem mode' do
+        Byebug::QuitCommand.any_instance.expects(:exit!)
+        enter 'cont', 'exit!'
+        debug_file 'post_mortem'
+      end
+    end
+
+    describe 'reload' do
+      after { change_line_in_file(@tst_file, 7, '        z = 4') }
+
+      it 'must work in post-mortem mode' do
+        enter 'cont', -> do
+          change_line_in_file(@tst_file, 7, 'z = 100')
+          'reload'
+        end, 'l 7-7'
+        debug_file 'post_mortem'
+        check_output_includes '7: z = 100'
+      end
+    end
+
+    describe 'edit' do
+      temporary_change_hash ENV, 'EDITOR', 'editr'
+
+      it 'must work in post-mortem mode' do
+        Byebug::Edit.any_instance.
+                     expects(:system).with("editr +2 #{fullpath('edit')}")
+        enter 'cont', "edit #{fullpath('edit')}:2", 'cont'
+        debug_file 'post_mortem'
+      end
+    end
+
+    describe 'info' do
+      it 'must work in post-mortem mode' do
+        enter 'cont', 'info line'
+        debug_file 'post_mortem'
+        check_output_includes "Line 8 of \"#{@tst_file}\""
+      end
+    end
+
+    describe 'irb' do
+      let(:irb) { stub(context: ->{}) }
+
+      it 'must work in post-mortem mode' do
+        irb.stubs(:eval_input).throws(:IRB_EXIT, :cont)
+        enter 'cont', 'break 12', 'irb'
+        debug_file('post_mortem') { $state.line.must_equal 12 }
+      end
+    end
+
+    describe 'source' do
+      before { File.open(filename, 'w') do |f|
+                 f.puts 'break 2'
+                 f.puts 'break 3 if true'
+               end }
+      after { FileUtils.rm(filename) }
+
+      let(:filename) { 'source_example.txt' }
+
+      it 'must work in post-mortem mode' do
+        enter 'cont', "so #{filename}"
+        debug_file('post_mortem') { Byebug.breakpoints[0].pos.must_equal 3 }
+      end
+    end
+
+    describe 'help' do
+      it 'must work in post-mortem mode' do
+        enter 'cont', 'help'
+        debug_file 'post_mortem'
+        check_output_includes 'Available commands:'
+      end
+    end
+
+    describe 'var' do
+      it 'must work in post-mortem mode' do
+        enter 'cont', 'var local'
+        debug_file 'post_mortem'
+        check_output_includes 'x => nil', 'z => 4'
+      end
+    end
+
+    describe 'list' do
+      it 'must work in post-mortem mode' do
+        enter 'cont'
+        debug_file 'post_mortem'
+        check_output_includes "[3, 12] in #{@tst_file}"
+      end
+    end
+
+    describe 'method' do
+      it 'must work in post-mortem mode' do
+        enter 'cont', 'm i self'
+        debug_file 'post_mortem'
+        check_output_includes /to_s/
+      end
+    end
+
+    describe 'kill' do
+      it 'must work in post-mortem mode' do
+        Process.expects(:kill).with('USR1', Process.pid)
+        enter 'cont', 'kill USR1'
+        debug_file 'post_mortem'
+      end
+    end
+
+    describe 'eval' do
+      it 'must work in post-mortem mode' do
+        enter 'cont', 'eval 2 + 2'
+        debug_file 'post_mortem'
+        check_output_includes '4'
+      end
+    end
+
+    describe 'set' do
+      temporary_change_hash Byebug.settings, :autolist, 0
+
+      it 'must work in post-mortem mode' do
+        enter 'cont', 'set autolist on'
+        debug_file 'post_mortem'
+        check_output_includes 'autolist is on.'
+      end
+    end
+
+    describe 'save' do
+      let(:file_name) { 'save_output.txt' }
+      let(:file_contents) { File.read(file_name) }
+      after { FileUtils.rm(file_name) }
+
+      it 'must work in post-mortem mode' do
+        enter 'cont', "save #{file_name}"
+        debug_file 'post_mortem'
+        file_contents.must_include 'set autoirb off'
+      end
+    end
+
+    describe 'show' do
+      it 'must work in post-mortem mode' do
+        enter 'cont', 'show autolist'
+        debug_file 'post_mortem'
+        check_output_includes 'autolist is on.'
+      end
+    end
+
+    describe 'trace' do
+      it 'must work in post-mortem mode' do
+        enter 'cont', 'trace on'
+        debug_file 'post_mortem'
+        check_output_includes 'line tracing is on.'
+      end
+    end
+
+    describe 'thread' do
+      it "must work in post-mortem mode" do
+        enter 'cont', 'thread list'
+        debug_file('post_mortem')
+        check_output_includes /\+ \d+ #<Thread:(\S+) run/
+      end
+    end
   end
 end

data/test/quit_test.rb

@@ -42,13 +42,4 @@ class TestQuit < TestDsl::TestCase
     enter 'exit!'
     debug_file 'quit'
   end
-
-  describe 'Post Mortem' do
-    it 'must work in post-mortem mode' do
-      Byebug::QuitCommand.any_instance.expects(:exit!)
-      enter 'cont', 'exit!'
-      debug_file 'post_mortem'
-    end
-  end
-
 end

data/test/reload_test.rb

@@ -3,7 +3,7 @@ require_relative 'test_helper'
 class TestReload < TestDsl::TestCase
 
   describe 'autoreloading' do
-    after { Byebug::Command.settings[:autoreload] = true }
+    after { Byebug.settings[:autoreload] = true }
 
     it 'must notify that automatic reloading is on by default' do
       enter 'reload'
@@ -31,18 +31,4 @@ class TestReload < TestDsl::TestCase
       check_output_includes '4: a = 100'
     end
   end
-
-  describe 'Post Mortem' do
-    after { change_line_in_file(fullpath('post_mortem'), 7, '        z = 4') }
-
-    it 'must work in post-mortem mode' do
-      enter 'cont', -> do
-        change_line_in_file(fullpath('post_mortem'), 7, 'z = 100')
-        'reload'
-      end, 'l 7-7'
-      debug_file 'post_mortem'
-      check_output_includes '7: z = 100'
-    end
-  end
-
 end

data/test/repl_test.rb

@@ -35,14 +35,6 @@ class TestRepl < TestDsl::TestCase
         debug_file 'repl'
       end
     end
-
-    describe 'Post Mortem' do
-      it 'must work in post-mortem mode' do
-        irb.stubs(:eval_input).throws(:IRB_EXIT, :cont)
-        enter 'cont', 'break 12', 'irb'
-        debug_file('post_mortem') { $state.line.must_equal 12 }
-      end
-    end
   end
 
   @has_pry = false
@@ -66,7 +58,7 @@ class TestRepl < TestDsl::TestCase
       end
     end
 
-    describe 'Post Mortem' do
+    describe 'post-mortem' do
       it 'must work in post-mortem mode' do
         skip 'TODO'
       end

data/test/restart_test.rb

@@ -1,12 +1,6 @@
 require_relative 'test_helper'
 
 class TestRestart < TestDsl::TestCase
-
-  def must_restart
-    Byebug::RestartCommand.any_instance.unstub(:exec)
-    Byebug::RestartCommand.any_instance.expects(:exec)
-  end
-
   describe 'usual restarting' do
     temporary_change_const Byebug, 'BYEBUG_SCRIPT', 'byebug_script'
 
@@ -18,7 +12,7 @@ class TestRestart < TestDsl::TestCase
     end
 
     describe 'when arguments have spaces' do
-      temporary_change_hash Byebug::Command.settings, :argv, ['argv1', 'argv 2']
+      temporary_change_hash Byebug.settings, :argv, ['argv1', 'argv 2']
 
       it 'must be correctly escaped' do
         Byebug::RestartCommand.any_instance.expects(:exec).with \
@@ -29,7 +23,7 @@ class TestRestart < TestDsl::TestCase
     end
 
     describe 'when arguments specified by set command' do
-      temporary_change_hash Byebug::Command.settings, :argv, []
+      temporary_change_hash Byebug.settings, :argv, []
 
       it 'must specify arguments by "set" command' do
         Byebug::RestartCommand.any_instance.expects(:exec).
@@ -47,7 +41,7 @@ class TestRestart < TestDsl::TestCase
       temporary_change_const Byebug, 'BYEBUG_SCRIPT', 'byebug_script'
 
       describe 'with set args' do
-        temporary_change_hash Byebug::Command.settings, :argv, ['argv']
+        temporary_change_hash Byebug.settings, :argv, ['argv']
 
         it 'must restart and show a message about reexecing' do
           must_restart
@@ -141,13 +135,4 @@ class TestRestart < TestDsl::TestCase
       end
     end
   end
-
-  describe 'Post Mortem' do
-    it 'must work in post-mortem mode' do
-      must_restart
-      enter 'cont', 'restart'
-      debug_file 'post_mortem'
-    end
-  end
-
 end

data/test/save_test.rb

@@ -1,6 +1,7 @@
 require_relative 'test_helper'
 
 class TestSave < TestDsl::TestCase
+
   describe 'successful saving' do
     let(:file_name) { 'save_output.txt' }
     let(:file_contents) { File.read(file_name) }
@@ -54,7 +55,6 @@ class TestSave < TestDsl::TestCase
     it 'must show a message about successful saving' do
       check_output_includes "Saved to '#{file_name}'"
     end
-
   end
 
   describe 'without filename' do
@@ -73,16 +73,4 @@ class TestSave < TestDsl::TestCase
       check_output_includes "Saved to '#{interface.restart_file}'"
     end
   end
-
-  describe 'Post Mortem' do
-    let(:file_name) { 'save_output.txt' }
-    let(:file_contents) { File.read(file_name) }
-    after { FileUtils.rm(file_name) }
-    it 'must work in post-mortem mode' do
-      enter 'cont', "save #{file_name}"
-      debug_file 'post_mortem'
-      file_contents.must_include 'set autoirb off'
-    end
-  end
-
 end

data/test/set_test.rb

@@ -3,63 +3,63 @@ require_relative 'test_helper'
 class TestSet < TestDsl::TestCase
 
   describe 'setting to on' do
-    temporary_change_hash Byebug::Command.settings, :autolist, 0
+    temporary_change_hash Byebug.settings, :autolist, 0
 
     it 'must set a setting to on' do
       enter 'set autolist on'
       debug_file 'set'
-      Byebug::Command.settings[:autolist].must_equal 1
+      Byebug.settings[:autolist].must_equal 1
     end
 
     it 'must set a setting to on by 1' do
       enter 'set autolist 1'
       debug_file 'set'
-      Byebug::Command.settings[:autolist].must_equal 1
+      Byebug.settings[:autolist].must_equal 1
     end
 
     it 'must set a setting to on by default' do
       enter 'set autolist'
       debug_file 'set'
-      Byebug::Command.settings[:autolist].must_equal 1
+      Byebug.settings[:autolist].must_equal 1
     end
 
     it 'must set a setting using shortcut' do
       enter 'set autol'
       debug_file 'set'
-      Byebug::Command.settings[:autolist].must_equal 1
+      Byebug.settings[:autolist].must_equal 1
     end
   end
 
   describe 'setting to off' do
-    temporary_change_hash Byebug::Command.settings, :autolist, 1
+    temporary_change_hash Byebug.settings, :autolist, 1
 
     it 'must set a setting to off' do
       enter 'set autolist off'
       debug_file 'set'
-      Byebug::Command.settings[:autolist].must_equal 0
+      Byebug.settings[:autolist].must_equal 0
     end
 
     it 'must set a setting to off by 0' do
       enter 'set autolist 0'
       debug_file 'set'
-      Byebug::Command.settings[:autolist].must_equal 0
+      Byebug.settings[:autolist].must_equal 0
     end
 
     it 'must set a setting to off by "no" prefix' do
       enter 'set noautolist'
       debug_file 'set'
-      Byebug::Command.settings[:autolist].must_equal 0
+      Byebug.settings[:autolist].must_equal 0
     end
 
     it 'must set a setting to off by "no" prefix and shortcut' do
       enter 'set noautol'
       debug_file 'set'
-      Byebug::Command.settings[:autolist].must_equal 0
+      Byebug.settings[:autolist].must_equal 0
     end
   end
 
   describe 'messages' do
-    temporary_change_hash Byebug::Command.settings, :autolist, 0
+    temporary_change_hash Byebug.settings, :autolist, 0
 
     it 'must show a message after setting' do
       enter 'set autolist on'
@@ -71,7 +71,7 @@ class TestSet < TestDsl::TestCase
   describe 'testing' do
     describe '$state' do
       describe 'when setting "testing" to on' do
-        temporary_change_hash Byebug::Command.settings, :testing, false
+        temporary_change_hash Byebug.settings, :testing, false
 
         it 'must get set' do
           enter 'set testing', 'break 3', 'cont'
@@ -81,7 +81,7 @@ class TestSet < TestDsl::TestCase
       end
 
       describe 'when setting "testing" to off' do
-        temporary_change_hash Byebug::Command.settings, :testing, true
+        temporary_change_hash Byebug.settings, :testing, true
 
         it 'must get unset' do
           enter 'set notesting', 'break 3', 'cont'
@@ -99,6 +99,12 @@ class TestSet < TestDsl::TestCase
         interface.history_save.must_equal true
       end
 
+      it 'must set history save to on when no param' do
+        enter 'set history save'
+        debug_file 'set'
+        interface.history_save.must_equal true
+      end
+
       it 'must show a message' do
         enter 'set history save on'
         debug_file 'set'
@@ -124,6 +130,12 @@ class TestSet < TestDsl::TestCase
         debug_file 'set'
         check_output_includes 'Byebug history size is 250'
       end
+
+      it 'must show an error message if no size provided' do
+        enter 'set history size'
+        debug_file 'set'
+        check_output_includes 'You need to specify the history size'
+      end
     end
 
     describe 'filename' do
@@ -141,29 +153,30 @@ class TestSet < TestDsl::TestCase
         debug_file 'set'
         check_output_includes "The command history file is \"#{filename}\""
       end
+
+      it 'must show an error message if no filenmae provided' do
+        enter 'set history filename'
+        debug_file 'set'
+        check_output_includes 'You need to specify a filename'
+      end
+
     end
 
     it 'must show an error message if used wrong subcommand' do
       enter 'set history bla 2'
       debug_file 'set'
       check_output_includes \
-        'Invalid history parameter bla. Should be "filename", "save" or "size".'
-    end
-
-    it 'must show an error message if provided only one argument' do
-      enter 'set history save'
-      debug_file 'set'
-      check_output_includes 'Need two parameters for "set history"; got 1.'
+        'Invalid history parameter bla. Should be "filename", "save" or "size"'
     end
   end
 
   describe 'width' do
-    temporary_change_hash Byebug::Command.settings, :width, 20
+    temporary_change_hash Byebug.settings, :width, 20
 
     it 'must get correctly set' do
       enter 'set width 10'
       debug_file('set')
-      Byebug::Command.settings[:width].must_equal 10
+      Byebug.settings[:width].must_equal 10
     end
   end
 
@@ -174,14 +187,4 @@ class TestSet < TestDsl::TestCase
       check_output_includes /List of "set" subcommands:/
     end
   end
-
-  describe 'Post Mortem' do
-    temporary_change_hash Byebug::Command.settings, :autolist, 0
-
-    it 'must work in post-mortem mode' do
-      enter 'cont', 'set autolist on'
-      debug_file 'post_mortem'
-      check_output_includes 'autolist is on.'
-    end
-  end
 end