debugger 1.2.4 → 1.3.0

data/test/jump_test.rb

@@ -0,0 +1,70 @@
+require_relative 'test_helper'
+
+describe "Jump Command" do
+  include TestDsl
+
+  describe "successful" do
+    it "must jump with absolute line number" do
+      enter 'break 6', 'cont', "jump 8 #{fullpath('jump')}"
+      debug_file('jump') { state.line.must_equal 8 }
+    end
+
+    it "must not initialize skipped variables during jump" do
+      enter 'break 6', 'cont', "jump 8 #{fullpath('jump')}", 'next'
+      enter 'var local'
+      debug_file('jump')
+      check_output_includes "a => 2", "b => nil", "c => nil", "d => 5"
+    end
+
+    it "must jump with relative line number (-)" do
+      enter 'break 8', 'cont', "jump -2 #{fullpath('jump')}"
+      debug_file('jump') { state.line.must_equal 6 }
+    end
+
+    it "must jump with relative line number (+)" do
+      enter 'break 8', 'cont', "jump +2 #{fullpath('jump')}"
+      debug_file('jump') { state.line.must_equal 10 }
+    end
+  end
+
+  describe "errors" do
+    it "must show an error if line number is invalid" do
+      enter 'jump bla'
+      debug_file('jump')
+      check_output_includes "Bad line number: bla", interface.error_queue
+    end
+
+    it "must show an error if line number is not specified" do
+      enter 'jump'
+      debug_file('jump')
+      check_output_includes '"jump" must be followed by a line number', interface.error_queue
+    end
+
+    describe "when there is no active code in specified line" do
+      it "must not jump to there" do
+        enter "jump 13 #{fullpath('jump')}"
+        debug_file('jump') { state.line.must_equal 3 }
+      end
+
+      it "must show an error" do
+        enter "jump 13 #{fullpath('jump')}"
+        debug_file('jump')
+        check_output_includes "Couldn't find active code at #{fullpath('jump')}:13", interface.error_queue
+      end
+    end
+  end
+
+
+  describe "Post Mortem" do
+    # TODO: This test fails with "Segmentation fault". Probably need to fix it somehow, or forbid this
+    # command in the post mortem mode. Seems like state.context.frame_file and state.context.frame_line
+    # cause that.
+    it "must work in post-mortem mode"# do
+    #  enter 'cont', 'jump 12'
+    #  debug_file 'post_mortem'
+    #  pi
+    #end
+  end
+
+
+end

data/test/kill_test.rb

@@ -0,0 +1,47 @@
+require_relative 'test_helper'
+
+describe "Kill Command" do
+  include TestDsl
+
+  it "must send signal to some pid" do
+    Process.expects(:kill).with("USR1", Process.pid)
+    enter 'kill USR1'
+    debug_file('kill')
+  end
+
+  it "must finalize interface when sending KILL signal explicitly" do
+    Process.stubs(:kill).with("KILL", Process.pid)
+    interface.expects(:finalize)
+    enter 'kill KILL'
+    debug_file('kill')
+  end
+
+  it "must ask confirmation when sending KILL implicitly" do
+    Process.expects(:kill).with("KILL", Process.pid)
+    enter 'kill', 'y'
+    debug_file('kill')
+    check_output_includes "Really kill? (y/n)", interface.confirm_queue
+  end
+
+  describe "unknown signal" do
+    it "must not send the signal" do
+      Process.expects(:kill).with("BLA", Process.pid).never
+      enter 'kill BLA'
+      debug_file('kill')
+    end
+
+    it "must show an error" do
+      enter 'kill BLA'
+      debug_file('kill')
+      check_output_includes "signal name BLA is not a signal I know about", interface.error_queue
+    end
+  end
+
+  describe "Post Mortem" 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
+end

data/test/list_test.rb

@@ -0,0 +1,145 @@
+require_relative 'test_helper'
+
+describe "List Command" do
+  include TestDsl
+  temporary_change_hash_value(Debugger::Command.settings, :listsize, 3)
+  before { LineCache.clear_file_cache }
+  after  { LineCache.clear_file_cache }
+
+  describe "listsize" do
+    it "must show lines according to :listsize setting" do
+      enter 'set listsize 4', 'break 5', 'cont', 'list'
+      debug_file 'list'
+      check_output_includes "[3, 6] in #{fullpath('list')}"
+    end
+
+    it "must not set it if the param is not an integer" do
+      enter 'set listsize 4.0', 'break 5', 'cont', 'list'
+      debug_file 'list'
+      check_output_includes "[4, 6] in #{fullpath('list')}"
+    end
+  end
+
+  describe "without arguments" do
+    it "must show surrounding lines with the first call" do
+      enter 'break 5', 'cont', 'list'
+      debug_file 'list'
+      check_output_includes "[4, 6] in #{fullpath('list')}", "4  4", "=> 5  5", "6  6"
+    end
+
+    it "must list forward after second call" do
+      enter 'break 5', 'cont', 'list', 'list'
+      debug_file 'list'
+      check_output_includes "[7, 9] in #{fullpath('list')}", "7  7", "8  8", "9  9"
+    end
+  end
+
+  describe "list backward" do
+    it "must show surrounding lines with the first call" do
+      enter 'break 5', 'cont', 'list -'
+      debug_file 'list'
+      check_output_includes "[4, 6] in #{fullpath('list')}", "4  4", "=> 5  5", "6  6"
+    end
+
+    it "must list backward after second call" do
+      enter 'break 5', 'cont', 'list -', 'list -'
+      debug_file 'list'
+      check_output_includes "[1, 3] in #{fullpath('list')}", "1  debugger", "2  2", "3  3"
+    end
+  end
+
+  it "must show the surrounding lines with =" do
+    enter 'break 5', 'cont', 'list ='
+    debug_file 'list'
+    check_output_includes "[4, 6] in #{fullpath('list')}", "4  4", "=> 5  5", "6  6"
+  end
+
+  describe "autolist" do
+    temporary_change_hash_value(Debugger::Command.settings, :autolist, 0)
+
+    it "must show the surronding lines even without 'list' command if autolist is enabled" do
+      enter 'set autolist', 'break 5', 'cont'
+      debug_file 'list'
+      check_output_includes "[4, 6] in #{fullpath('list')}", "4  4", "=> 5  5", "6  6"
+    end
+  end
+
+  describe "specified lines" do
+    it "must show with mm-nn" do
+      enter 'list 4-6'
+      debug_file 'list'
+      check_output_includes "[4, 6] in #{fullpath('list')}", "4  4", "5  5", "6  6"
+    end
+
+    it "must show with mm,nn" do
+      enter 'list 4,6'
+      debug_file 'list'
+      check_output_includes "[4, 6] in #{fullpath('list')}", "4  4", "5  5", "6  6"
+    end
+
+    it "must show surroundings with mm-" do
+      enter 'list 4-'
+      debug_file 'list'
+      check_output_includes "[3, 5] in #{fullpath('list')}", "3  3", "4  4", "5  5"
+    end
+
+    it "must show surroundings with mm," do
+      enter 'list 4,'
+      debug_file 'list'
+      check_output_includes "[3, 5] in #{fullpath('list')}", "3  3", "4  4", "5  5"
+    end
+
+    it "must show nothing if there is no such lines" do
+      enter 'list 44,44'
+      debug_file 'list'
+      check_output_includes "[44, 44] in #{fullpath('list')}"
+      check_output_doesnt_include /^44  \S/
+    end
+
+    it "must show nothing if range is incorrect" do
+      enter 'list 5,4'
+      debug_file 'list'
+      check_output_includes "[5, 4] in #{fullpath('list')}"
+      check_output_doesnt_include "5  5"
+      check_output_doesnt_include "4  4"
+    end
+  end
+
+  describe "reload source" do
+    temporary_change_hash_value(Debugger::Command.settings, :reload_source_on_change, false)
+
+    after { change_line_in_file(fullpath('list'), 4, '4') }
+    it "must not reload if setting is false" do
+      enter 'set noautoreload', -> do
+        change_line_in_file(fullpath('list'), 4, '100')
+        'list 4-4'
+      end
+      debug_file 'list'
+      check_output_includes "4  4"
+    end
+
+    it "must reload if setting is true" do
+      enter 'set autoreload', -> do
+        change_line_in_file(fullpath('list'), 4, '100')
+        'list 4-4'
+      end
+      debug_file 'list'
+      check_output_includes "4  100"
+    end
+  end
+
+  it "must show an error when there is no such file" do
+    enter ->{state.file = "blabla"; 'list 4-4'}
+    debug_file 'list'
+    check_output_includes "No sourcefile available for blabla", interface.error_queue
+  end
+
+
+  describe "Post Mortem" do
+    it "must work in post-mortem mode" do
+      enter 'cont', 'list'
+      debug_file 'post_mortem'
+      check_output_includes "[7, 9] in #{fullpath('post_mortem')}"
+    end
+  end
+end

data/test/method_test.rb

@@ -0,0 +1,70 @@
+require_relative 'test_helper'
+
+describe "Method Command" do
+  include TestDsl
+
+  # TODO: Need to write tests for 'method signature' command, but I can't install the 'ruby-internal' gem
+  # on my machine, it fails to build gem native extension.
+
+  describe "show instance method of a class" do
+    it "must show using full command name" do
+      enter 'break 15', 'cont', 'm MethodEx'
+      debug_file 'method'
+      check_output_includes /bla/
+      check_output_doesnt_include /foo/
+    end
+
+    it "must show using shortcut" do
+      enter 'break 15', 'cont', 'method MethodEx'
+      debug_file 'method'
+      check_output_includes /bla/
+    end
+
+    it "must show an error if specified object is not a class or module" do
+      enter 'break 15', 'cont', 'm a'
+      debug_file 'method'
+      check_output_includes "Should be Class/Module: a"
+    end
+  end
+
+
+  describe "show methods of an object" do
+    it "must show using full command name" do
+      enter 'break 15', 'cont', 'method instance a'
+      debug_file 'method'
+      check_output_includes /bla/
+      check_output_doesnt_include /foo/
+    end
+
+    it "must show using shortcut" do
+      enter 'break 15', 'cont', 'm i a'
+      debug_file 'method'
+      check_output_includes /bla/
+    end
+  end
+
+
+  describe "show instance variables of an object" do
+    it "must show using full name command" do
+      enter 'break 15', 'cont', 'method iv a'
+      debug_file 'method'
+      check_output_includes '@a = "b"', '@c = "d"'
+    end
+
+    it "must show using shortcut" do
+      enter 'break 15', 'cont', 'm iv a'
+      debug_file 'method'
+      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

@@ -0,0 +1,25 @@
+require_relative 'test_helper'
+
+describe "Post Mortem" do
+  include TestDsl
+
+  it "must enter into port mortem mode" do
+    enter 'cont'
+    debug_file("post_mortem") { Debugger.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") { Debugger.post_mortem?.must_equal false }
+  end
+
+  it "must save the raised exception" do
+    enter 'cont'
+    debug_file("post_mortem") { Debugger.last_exception.must_be_kind_of RuntimeError }
+  end
+end

data/test/quit_test.rb

@@ -0,0 +1,55 @@
+require_relative 'test_helper'
+
+describe "Quit Command" do
+  include TestDsl
+
+  it "must quit if user confirmed" do
+    Debugger::QuitCommand.any_instance.expects(:exit!)
+    enter 'quit', 'y'
+    debug_file 'quit'
+    check_output_includes "Really quit? (y/n)", interface.confirm_queue
+  end
+
+  it "must not quit if user didn't confirm" do
+    Debugger::QuitCommand.any_instance.expects(:exit!).never
+    enter 'quit', 'n'
+    debug_file 'quit'
+    check_output_includes "Really quit? (y/n)", interface.confirm_queue
+  end
+
+  it "must quit immediatly if used with !" do
+    Debugger::QuitCommand.any_instance.expects(:exit!)
+    enter 'quit!'
+    debug_file 'quit'
+    check_output_doesnt_include "Really quit? (y/n)", interface.confirm_queue
+  end
+
+  it "must quit immediatly if used with 'unconditionally'" do
+    Debugger::QuitCommand.any_instance.expects(:exit!)
+    enter 'quit unconditionally'
+    debug_file 'quit'
+    check_output_doesnt_include "Really quit? (y/n)", interface.confirm_queue
+  end
+
+  it "must finalize interface before quitting" do
+    Debugger::QuitCommand.any_instance.stubs(:exit!)
+    interface.expects(:finalize)
+    enter 'quit!'
+    debug_file 'quit'
+  end
+
+  it "must quit if used 'exit' alias" do
+    Debugger::QuitCommand.any_instance.expects(:exit!)
+    enter 'exit!'
+    debug_file 'quit'
+  end
+
+  describe "Post Mortem" do
+    it "must work in post-mortem mode" do
+      Debugger::QuitCommand.any_instance.expects(:exit!)
+      enter 'cont', 'exit!'
+      debug_file 'post_mortem'
+    end
+  end
+
+end

data/test/reload_test.rb

@@ -0,0 +1,43 @@
+require_relative 'test_helper'
+
+describe "Reload Command" do
+  include TestDsl
+  temporary_change_hash_value(Debugger::Command.settings, :reload_source_on_change, false)
+
+  it "must notify that automatic reloading is off" do
+    enter 'reload'
+    debug_file 'reload'
+    check_output_includes "Source code is reloaded. Automatic reloading is off."
+  end
+
+  it "must notify that automatic reloading is on" do
+    enter 'set autoreload', 'reload'
+    debug_file 'reload'
+    check_output_includes "Source code is reloaded. Automatic reloading is on."
+  end
+
+  describe "reloading" do
+    after { change_line_in_file(fullpath('reload'), 4, '4') }
+    it "must reload the code" do
+      enter 'break 3', 'cont', 'l 4-4', -> do
+        change_line_in_file(fullpath('reload'), 4, '100')
+        'reload'
+      end, 'l 4-4'
+      debug_file 'reload'
+      check_output_includes "4  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