byebug 3.1.2 → 3.2.0

data/test/interrupt_test.rb

@@ -1,5 +1,5 @@
-module InterruptTest
-  class Example
+module Byebug
+  class InterruptExample
     def self.a(num)
       num += 2
       b(num)
@@ -16,49 +16,30 @@ module InterruptTest
     end
   end
 
-  class InterruptTestCase < TestDsl::TestCase
-    before do
+  class InterruptTestCase < TestCase
+    def setup
       @example = -> do
         byebug
-        ex = Example.a(7)
+        ex = InterruptExample.a(7)
         2.times do
           ex += 1
         end
-        Example.b(ex)
+        InterruptExample.b(ex)
       end
-    end
-
-    describe 'Interrupt Command' do
-      describe 'method call behaviour' do
-        it 'must interrupt on the next line' do
-          enter 'interrupt', 'continue'
-          debug_proc(@example) do
-            state.line.must_equal 4
-            state.file.must_equal __FILE__
-          end
-        end
-
-        describe 'when forcestep is set' do
-          temporary_change_hash Byebug::Setting, :forcestep, true
-
-          it 'must interrupt on the next line' do
-            enter 'interrupt', 'continue'
-            debug_proc(@example) do
-              state.line.must_equal 4
-              state.file.must_equal __FILE__
-            end
-          end
 
-          describe 'block behaviour' do
-            before { enter 'break 24', 'cont' }
+      super
+    end
 
-            it 'must step into blocks' do
-              enter 'interrupt', 'continue'
-              debug_proc(@example) { state.line.must_equal 25 }
-            end
-          end
-        end
+    def test_interrupt_stops_at_the_next_statement
+      enter 'interrupt', 'continue'
+      debug_proc(@example) do
+        assert_equal [__FILE__, 4], [state.file, state.line]
       end
     end
+
+    def test_interrupt_steps_into_blocks
+      enter 'break 24', 'cont', 'interrupt', 'cont'
+      debug_proc(@example) { assert_equal 25, state.line }
+    end
   end
 end

data/test/irb_test.rb

@@ -0,0 +1,47 @@
+module Byebug
+  class IrbTestCase < TestCase
+    def setup
+      @example = -> do
+        byebug
+        a = 2
+        a = 3
+        a = 4
+        a = 5
+        a = 6
+      end
+
+      super
+
+      interface.stubs(:kind_of?).with(LocalInterface).returns(true)
+      IRB::Irb.stubs(:new).returns(irb)
+    end
+
+    def irb
+      @irb ||= stub(context: -> {})
+    end
+
+    def test_irb_supports_next_command
+      irb.stubs(:eval_input).throws(:IRB_EXIT, :next)
+      enter 'irb'
+      debug_proc(@example) { assert_equal 7, state.line }
+    end
+
+    def test_irb_supports_step_command
+      irb.stubs(:eval_input).throws(:IRB_EXIT, :step)
+      enter 'irb'
+      debug_proc(@example) { assert_equal 7, state.line }
+    end
+
+    def test_irb_supports_cont_command
+      irb.stubs(:eval_input).throws(:IRB_EXIT, :cont)
+      enter 'break 8', 'irb'
+      debug_proc(@example) { assert_equal 8, state.line }
+    end
+
+    def test_autoirb_calls_irb_automatically_after_every_stop
+      irb.expects(:eval_input)
+      enter 'set autoirb', 'break 8', 'cont'
+      debug_proc(@example)
+    end
+  end
+end

data/test/kill_test.rb

@@ -1,50 +1,50 @@
-module KillTest
-  class Example
+module Byebug
+  class KillExample
     def self.kill_me
       'dieeee'
     end
   end
 
-  class KillTestCase < TestDsl::TestCase
-    before do
+  class KillTestCase < TestCase
+    def setup
       @example = -> do
         byebug
-        Example.kill_me
+        KillExample.kill_me
       end
+
+      super
     end
 
-    it 'must send signal to some pid' do
+    def test_kill_sends_signal_to_some_pid
       Process.expects(:kill).with('USR1', Process.pid)
       enter 'kill USR1'
       debug_proc(@example)
     end
 
-    it 'must close interface when sending KILL signal explicitly' do
+    def test_kill_closes_interface_when_sending_KILL_signal_explicitly
       Process.stubs(:kill).with('KILL', Process.pid)
       interface.expects(:close)
       enter 'kill KILL'
       debug_proc(@example)
     end
 
-    it 'must ask confirmation when sending KILL implicitly' do
+    def test_kill_asks_confirmation_when_sending_kill_implicitly
       Process.expects(:kill).with('KILL', Process.pid)
       enter 'kill', 'y'
       debug_proc(@example)
       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_proc(@example)
-      end
+    def test_kill_does_not_send_an_unknown_signal
+      Process.expects(:kill).with('BLA', Process.pid).never
+      enter 'kill BLA'
+      debug_proc(@example)
+    end
 
-      it 'must show an error' do
-        enter 'kill BLA'
-        debug_proc(@example)
-        check_error_includes 'signal name BLA is not a signal I know about'
-      end
+    def test_kill_shows_an_error_when_the_signal_in_unknown
+      enter 'kill BLA'
+      debug_proc(@example)
+      check_error_includes 'signal name BLA is not a signal I know about'
     end
   end
 end

data/test/list_test.rb

@@ -1,6 +1,6 @@
-module ListTest
-  class ListTestCase < TestDsl::TestCase
-    before do
+module Byebug
+  class ListTestCase < TestCase
+    def setup
       @example = -> do
         byebug
         a = 6
@@ -25,6 +25,8 @@ module ListTest
         a = 25
         a = '%26'
       end
+
+      super
     end
 
     def lines_between(min, max, mark_current = true)
@@ -38,167 +40,158 @@ module ListTest
       output
     end
 
-    describe 'listsize' do
-      it 'must show lines according to :listsize setting' do
-        debug_proc(@example)
-        check_output_includes "[1, 10] in #{__FILE__}"
-      end
-
-      it 'must not set it if the param is not an integer' do
-        enter 'set listsize 4.0', 'list'
-        debug_proc(@example)
-        check_output_includes "[1, 10] in #{__FILE__}"
-      end
-
-      describe 'when it goes before beginning of file' do
-        temporary_change_hash Byebug::Setting, :listsize, 12
-
-        it 'must move range up' do
-          enter 'list'
-          debug_proc(@example)
-          check_output_includes "[1, 12] in #{__FILE__}"
-        end
-      end
-
-      describe 'when it goes after the end of file' do
-        it 'must move range down' do
-          skip "Can't test this with the current setup"
-        end
-      end
-
-      describe 'very large' do
-        temporary_change_hash Byebug::Setting, :listsize, 1000
-
-        it 'must list whole file if number of lines is smaller than listsize' do
-          n_lines = %x{wc -l #{__FILE__}}.split.first.to_i
-          debug_proc(@example)
-          check_output_includes "[1, #{n_lines}] in #{__FILE__}"
-        end
-     end
+    def test_lists_source_code_lines
+      Setting[:listsize] = 10
+      enter 'list'
+      debug_proc(@example)
+      check_output_includes "[1, 10] in #{__FILE__}"
     end
 
-    describe 'without arguments' do
-      it 'must show surrounding lines with the first call' do
-        enter 'break 8', 'cont'
-        debug_proc(@example)
-        check_output_includes("[3, 12] in #{__FILE__}", *lines_between(3, 12))
-      end
-
-      it 'must list forward after second call' do
-        enter 'break 8', 'cont', 'list'
-        debug_proc(@example)
-        check_output_includes("[13, 22] in #{__FILE__}",
-                              *lines_between(13, 22, false))
-      end
+    def test_listsize_is_not_set_if_parameter_is_not_an_integer
+      enter 'set listsize 15.0', 'list'
+      debug_proc(@example)
+      check_output_doesnt_include "[1, 15] in #{__FILE__}"
     end
 
-    describe 'list backwards' do
-      temporary_change_hash Byebug::Setting, :autolist, false
-
-      it 'must show surrounding lines with the first call' do
-        enter 'break 18', 'cont', 'list -'
-        debug_proc(@example)
-        check_output_includes("[13, 22] in #{__FILE__}", *lines_between(13, 22))
-      end
+    def test_moves_range_up_when_it_goes_before_beginning_of_file
+      Setting[:listsize] = 12
+      enter 'list'
+      debug_proc(@example)
+      check_output_includes "[1, 12] in #{__FILE__}"
+    end
 
-      it 'must list backward after second call' do
-        enter 'break 18', 'cont', 'list -', 'list -'
-        debug_proc(@example)
-        check_output_includes("[3, 12] in #{__FILE__}",
-                              *lines_between(3, 12, false))
-      end
+    def test_moves_range_down_when_it_goes_after_the_end_of_file
+      skip "Can't test this with the current setup"
     end
 
-    describe 'list surrounding' do
-      temporary_change_hash Byebug::Setting, :autolist, false
+    def test_lists_the_whole_file_if_number_of_lines_is_smaller_than_listsize
+      Setting[:listsize] = 1000
+      n_lines = %x{wc -l #{__FILE__}}.split.first.to_i
+      enter 'list'
+      debug_proc(@example)
+      check_output_includes "[1, #{n_lines}] in #{__FILE__}"
+    end
 
-      it 'must show the surrounding lines with =' do
-        enter 'break 8', 'cont', 'list ='
-        debug_proc(@example)
-        check_output_includes("[3, 12] in #{__FILE__}", *lines_between(3, 12))
-      end
+    def test_lists_surrounding_lines_after_the_first_call_to_list
+      enter 'break 8', 'cont', 'list'
+      debug_proc(@example)
+      check_output_includes("[3, 12] in #{__FILE__}", *lines_between(3, 12))
     end
 
-    describe 'specific range' do
-      it 'must show with mm-nn' do
-        enter 'list 7-9'
-        debug_proc(@example)
-        check_output_includes("[7, 9] in #{__FILE__}",
-                              *lines_between(7, 9, false))
-      end
+    def test_lists_forwards_after_the_second_call_to_list
+      enter 'break 8', 'cont', 'list', 'list'
+      debug_proc(@example)
+      check_output_includes("[13, 22] in #{__FILE__}",
+                            *lines_between(13, 22, false))
+    end
 
-      it 'must show with mm,nn' do
-        enter 'list 7,9'
-        debug_proc(@example)
-        check_output_includes("[7, 9] in #{__FILE__}",
-                              *lines_between(7, 9, false))
-      end
+    def test_lists_surrounding_lines_after_the_first_call_to_list_minus
+      enter 'break 18', 'cont', 'list -'
+      debug_proc(@example)
+      check_output_includes("[13, 22] in #{__FILE__}", *lines_between(13, 22))
+    end
 
-      it 'must show nothing if there is no such lines' do
-        enter 'list 500,505'
-        debug_proc(@example)
-        check_error_includes 'Invalid line range'
-        check_output_doesnt_include "[500, 505] in #{__FILE__}"
-        check_output_doesnt_include(/^500  \S/)
-      end
+    def test_lists_backwards_after_the_second_call_to_list_minus
+      enter 'break 18', 'cont', 'list -', 'list -'
+      debug_proc(@example)
+      check_output_includes("[3, 12] in #{__FILE__}",
+                            *lines_between(3, 12, false))
+    end
 
-      it 'must show nothing if range is incorrect' do
-        enter 'list 5,4'
-        debug_proc(@example)
-        check_output_includes "[5, 4] in #{__FILE__}"
-      end
+    def test_lists_surrounding_lines_when_list_equals_is_called
+      enter 'break 8', 'cont', 'list ='
+      debug_proc(@example)
+      check_output_includes("[3, 12] in #{__FILE__}", *lines_between(3, 12))
     end
 
-    describe 'arround specific line' do
-      it 'must show surroundings with mm-' do
-        enter 'list 17-'
-        debug_proc(@example)
-        check_output_includes("[12, 21] in #{__FILE__}",
-                              *lines_between(12, 21, false))
-      end
+    def test_lists_specific_range_when_requested_in_hyphen_format
+      enter 'list 7-9'
+      debug_proc(@example)
+      check_output_includes("[7, 9] in #{__FILE__}",
+                            *lines_between(7, 9, false))
+    end
 
-      it 'must show surroundings with mm,' do
-        enter 'list 17,'
-        debug_proc(@example)
-        check_output_includes("[12, 21] in #{__FILE__}",
-                              *lines_between(12, 21, false))
-      end
+    def test_lists_specific_range_when_requested_in_comma_format
+      enter 'list 7,9'
+      debug_proc(@example)
+      check_output_includes("[7, 9] in #{__FILE__}",
+                            *lines_between(7, 9, false))
     end
 
-    describe 'reload source' do
-      after  { change_line_in_file(__FILE__, 7, '        a = 7')   }
+    def test_lists_nothing_if_unexistent_range_is_specified
+      enter 'list 500,505'
+      debug_proc(@example)
+      check_error_includes 'Invalid line range'
+      check_output_doesnt_include "[500, 505] in #{__FILE__}"
+      check_output_doesnt_include(/^500  \S/)
+    end
 
-      describe 'when autoreload is false' do
-        temporary_change_hash Byebug::Setting, :autoreload, false
+    def test_lists_nothing_if_invalid_range_is_specified
+      enter 'list 5,4'
+      debug_proc(@example)
+      check_output_includes "[5, 4] in #{__FILE__}"
+    end
 
-        it 'must not reload listing with file changes' do
-          enter -> { change_line_in_file __FILE__, 7, '        a = 100' ;
-                     'list 7-7' }
-          debug_proc(@example)
-          check_output_includes(/7:\s+a = 7/)
-        end
-      end
+    def test_list_proper_lines_when_range_around_specific_line_with_hyphen
+      enter 'list 17-'
+      debug_proc(@example)
+      check_output_includes("[12, 21] in #{__FILE__}",
+                            *lines_between(12, 21, false))
+    end
 
-      describe 'when autoreload is true' do
-        it 'must reload listing with file changes' do
-          enter -> { change_line_in_file __FILE__, 7, '        a = 100' ;
-                     'list 7-7' }
-          debug_proc(@example)
-          check_output_includes(/7:\s+a = 100/)
-        end
-      end
+    def test_list_proper_lines_when_range_around_specific_line_with_comma
+      enter 'list 17,'
+      debug_proc(@example)
+      check_output_includes("[12, 21] in #{__FILE__}",
+                            *lines_between(12, 21, false))
     end
 
-    it 'must show an error when there is no such file' do
+    def test_shows_an_error_when_the_file_to_list_does_not_exist
       enter -> { state.file = 'blabla'; 'list 7-7' }
       debug_proc(@example)
       check_error_includes 'No sourcefile available for blabla'
     end
 
-    it 'must correctly print lines containing % sign' do
+    def test_correctly_print_lines_containing_the_percentage_symbol
       enter 'list 26'
       debug_proc(@example)
       check_output_includes "26:         a = '%26'"
     end
   end
+
+  class ListTestCaseAutoreload < ListTestCase
+    def setup
+      super
+      Setting[:autoreload] = true
+      enter 'list' # force first reading of file
+    end
+
+    def test_lists_file_changes_with_autoreload_enabled
+      enter -> do
+        change_line_in_file(__FILE__, 7, '        a = 100')
+        'list 7-7'
+      end
+      debug_proc(@example)
+      check_output_includes(/7:\s+a = 100/)
+      change_line_in_file(__FILE__, 7, '        a = 7')
+    end
+  end
+
+  class ListTestCaseNoAutoreload < ListTestCase
+    def setup
+      super
+      Setting[:autoreload] = false
+      enter 'list' # force first reading of file
+    end
+
+    def test_does_not_list_file_changes_with_autoreload_disabled
+      enter -> do
+        change_line_in_file(__FILE__, 7, '        a = 100')
+        'list 7-7'
+      end
+      debug_proc(@example)
+      check_output_doesnt_include(/7:\s+a = 100/)
+      change_line_in_file(__FILE__, 7, '        a = 7')
+    end
+  end
 end