byebug 0.0.1 → 1.0.0

data/test/irb_test.rb

@@ -2,80 +2,85 @@ require_relative 'test_helper'
 
 describe "Irb Command" do
   include TestDsl
-  before do
-    interface.stubs(:kind_of?).with(Byebug::LocalInterface).returns(true)
-    IRB::Irb.stubs(:new).returns(irb)
-    Signal.trap("SIGINT", "IGNORE")
-  end
-  after do
-    Signal.trap("SIGINT", "DEFAULT")
-  end
-  let(:irb) { stub(context: ->{}) }
 
-  it "must support next command" do
-    irb.stubs(:eval_input).throws(:IRB_EXIT, :next)
-    enter 'irb'
-    debug_file('irb') { state.line.must_equal 3 }
-  end
-
-  it "must support step command" do
-    irb.stubs(:eval_input).throws(:IRB_EXIT, :step)
-    enter 'irb'
-    debug_file('irb') { state.line.must_equal 3 }
-  end
+  describe "Irb Command Setup" do
+    before do
+      interface.stubs(:kind_of?).with(Byebug::LocalInterface).returns(true)
+      IRB::Irb.stubs(:new).returns(irb)
+      Signal.trap("SIGINT", "IGNORE")
+    end
+    after do
+      Signal.trap("SIGINT", "DEFAULT")
+    end
+    let(:irb) { stub(context: ->{}) }
 
-  it "must support cont command" do
-    irb.stubs(:eval_input).throws(:IRB_EXIT, :cont)
-    enter 'break 4', 'irb'
-    debug_file('irb') { state.line.must_equal 4 }
-  end
+    it "must support next command" do
+      irb.stubs(:eval_input).throws(:IRB_EXIT, :next)
+      enter 'irb'
+      debug_file('irb') { state.line.must_equal 3 }
+    end
 
-  describe "autoirb" do
-    temporary_change_hash_value(Byebug::Command.settings, :autoirb, 0)
+    it "must support step command" do
+      irb.stubs(:eval_input).throws(:IRB_EXIT, :step)
+      enter 'irb'
+      debug_file('irb') { state.line.must_equal 3 }
+    end
 
-    it "must call irb automatically after breakpoint" do
-      irb.expects(:eval_input)
-      enter 'set autoirb', 'break 4', 'cont'
-      debug_file 'irb'
+    it "must support cont command" do
+      irb.stubs(:eval_input).throws(:IRB_EXIT, :cont)
+      enter 'break 4', 'irb'
+      debug_file('irb') { state.line.must_equal 4 }
     end
-  end
 
-  # TODO: Can't reliably test the signal, from time to time Signal.trap, which is defined in IRBCommand, misses
-  # the SIGINT signal, which makes the test suite exit. Not sure how to fix that...
-  it "must translate SIGINT into 'cont' command" # do
-  #  irb.stubs(:eval_input).calls { Process.kill("SIGINT", Process.pid) }
-  #  enter 'break 4', 'irb'
-  #  debug_file('irb') { state.line.must_equal 4 }
-  #end
+    describe "autoirb" do
+      temporary_change_hash_value(Byebug::Command.settings, :autoirb, 0)
 
-  describe "setting context to $rdebug_state" do
-    before { $rdebug_state = nil }
-    temporary_change_hash_value(Byebug::Command.settings, :byebugtesting, false)
+      it "must call irb automatically after breakpoint" do
+        irb.expects(:eval_input)
+        enter 'set autoirb', 'break 4', 'cont'
+        debug_file 'irb'
+      end
+    end
 
-    it "must set $rdebug_state if irb is in the debug mode" do
-      rdebug_state = nil
-      irb.stubs(:eval_input).calls { rdebug_state = $rdebug_state }
-      enter 'irb -d'
-      debug_file('irb')
-      rdebug_state.must_be_kind_of Byebug::CommandProcessor::State
+    # TODO: Can't reliably test the signal, from time to time Signal.trap, which
+    # is defined in IRBCommand, misses the SIGINT signal, which makes the test
+    # suite exit. Not sure how to fix that...
+    it "must translate SIGINT into 'cont' command" do
+      irb.stubs(:eval_input).calls { Process.kill("SIGINT", Process.pid) }
+      enter 'break 4', 'irb'
+      debug_file('irb') { state.line.must_equal 4 }
     end
 
-    it "must not set $rdebug_state if irb is not in the debug mode" do
-      rdebug_state = nil
-      irb.stubs(:eval_input).calls { rdebug_state = $rdebug_state }
-      enter 'irb'
-      debug_file('irb')
-      rdebug_state.must_be_nil
+    describe "setting context to $byebug_state" do
+      before { $byebug_state = nil }
+      temporary_change_hash_value(Byebug::Command.settings, :byebugtesting, false)
+
+      it "must set $byebug_state if irb is in the debug mode" do
+        byebug_state = nil
+        irb.stubs(:eval_input).calls { byebug_state = $byebug_state }
+        enter 'irb -d'
+        debug_file('irb')
+        byebug_state.must_be_kind_of Byebug::CommandProcessor::State
+      end
+
+      it "must not set $byebug_state if irb is not in the debug mode" do
+        byebug_state = nil
+        irb.stubs(:eval_input).calls { byebug_state = $byebug_state }
+        enter 'irb'
+        debug_file('irb')
+        byebug_state.must_be_nil
+      end
     end
-  end
 
-  describe "Post Mortem" do
-    it "must work in post-mortem mode" do
-      skip("No post morten mode for now")
-      #irb.stubs(:eval_input).throws(:IRB_EXIT, :cont)
-      #enter 'cont', 'break 12', 'irb'
-      #debug_file("post_mortem") { state.line.must_equal 12 }
+    describe "Post Mortem" do
+      it "must work in post-mortem mode" do
+        skip("No post morten mode for now")
+        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
 
 end

data/test/jump_test.rb

@@ -5,11 +5,13 @@ describe "Jump Command" do
 
   describe "successful" do
     it "must jump with absolute line number" do
+      skip("No jumping for now")
       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
+      skip("No jumping for now")
       enter 'break 6', 'cont', "jump 8 #{fullpath('jump')}", 'next'
       enter 'var local'
       debug_file('jump')
@@ -17,11 +19,13 @@ describe "Jump Command" do
     end
 
     it "must jump with relative line number (-)" do
+      skip("No jumping for now")
       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
+      skip("No jumping for now")
       enter 'break 8', 'cont', "jump +2 #{fullpath('jump')}"
       debug_file('jump') { state.line.must_equal 10 }
     end
@@ -29,42 +33,47 @@ describe "Jump Command" do
 
   describe "errors" do
     it "must show an error if line number is invalid" do
+      skip("No jumping for now")
       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
+      skip("No jumping for now")
       enter 'jump'
       debug_file('jump')
-      check_output_includes '"jump" must be followed by a line number', interface.error_queue
+      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
+        skip("No jumping for now")
         enter "jump 13 #{fullpath('jump')}"
         debug_file('jump') { state.line.must_equal 3 }
       end
 
       it "must show an error" do
+        skip("No jumping for now")
         enter "jump 13 #{fullpath('jump')}"
         debug_file('jump')
-        check_output_includes "Couldn't find active code at #{fullpath('jump')}:13", interface.error_queue
+        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.
+    # TODO: This test fails with "Segmentation fault". Probably need to fix it
+    # somehow or forbid this command in post mortem mode. Seems like
+    # state.context.frame_file and state.context.frame_line cause that.
     it "must work in post-mortem mode" do
       skip("No post morten mode for now")
-      #enter 'cont', 'jump 12'
-      #debug_file 'post_mortem'
-      #pi
+      enter 'cont', 'jump 12'
+      debug_file 'post_mortem'
     end
   end
 
-
 end

data/test/list_test.rb

@@ -2,144 +2,151 @@ require_relative 'test_helper'
 
 describe "List Command" do
   include TestDsl
-  temporary_change_hash_value(Byebug::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')}"
+  describe "List Command Setup" do
+    before do
+      LineCache.clear_file_cache
+      Byebug::Command.settings[:listsize] = 3
     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
+    after  { LineCache.clear_file_cache }
 
-  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
+    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 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"
+      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
-  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
+    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 backward after second call" do
-      enter 'break 5', 'cont', 'list -', 'list -'
-      debug_file 'list'
-      check_output_includes "[1, 3] in #{fullpath('list')}", "1  byebug", "2  2", "3  3"
+      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
-  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 "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
 
-  describe "autolist" do
-    temporary_change_hash_value(Byebug::Command.settings, :autolist, 0)
+      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  byebug", "2  2", "3  3"
+      end
+    end
 
-    it "must show the surronding lines even without 'list' command if autolist is enabled" do
-      enter 'set autolist', 'break 5', 'cont'
+    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
-  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
+    describe "autolist" do
+      temporary_change_hash_value(Byebug::Command.settings, :autolist, 0)
 
-    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"
+      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
 
-    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
+    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 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 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 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 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 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
+      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
 
-  describe "reload source" do
-    temporary_change_hash_value(Byebug::Command.settings, :reload_source_on_change, false)
+      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
 
-    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'
+      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
-      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'
+    describe "reload source" do
+      temporary_change_hash_value(Byebug::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
-      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
+    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
-      skip("No post morten mode for now")
-      #enter 'cont', 'list'
-      #debug_file 'post_mortem'
-      #check_output_includes "[7, 9] in #{fullpath('post_mortem')}"
+    describe "Post Mortem" do
+      it "must work in post-mortem mode" do
+        skip("No post morten mode for now")
+        enter 'cont', 'list'
+        debug_file 'post_mortem'
+        check_output_includes "[7, 9] in #{fullpath('post_mortem')}"
+      end
     end
+
   end
 end