byebug 1.0.3 → 1.1.0

data/test/kill_test.rb

@@ -1,48 +1,48 @@
 require_relative 'test_helper'
 
-describe "Kill Command" do
+describe 'Kill Command' do
   include TestDsl
 
-  it "must send signal to some pid" do
-    Process.expects(:kill).with("USR1", Process.pid)
+  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)
+  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)
+  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
+    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
+  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
+    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
+      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
-      skip("No post morten mode for now")
-      #Process.expects(:kill).with("USR1", Process.pid)
-      #enter 'cont', 'kill USR1'
-      #debug_file "post_mortem"
+  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

@@ -3,196 +3,180 @@ require_relative 'test_helper'
 describe 'List Command' do
   include TestDsl
 
-  describe 'List Command (Setup)' do
-
-    before { LineCache.clear_file_cache }
-
-    describe 'listsize' do
-      before do
-        Byebug::Command.settings[:listsize] = 3
-        Byebug::Command.settings[:autolist] = 0
-      end
-
-      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
-
-      it 'must move range up when it goes before begining of file' do
-        enter 'set listsize 10', 'break 3', 'cont', 'list'
-        debug_file 'list'
-        check_output_includes "[1, 10] in #{fullpath('list')}"
-      end
-
-      it 'must move range down when it goes after end of file' do
-        enter 'set listsize 10', 'break 10', 'cont', 'list'
-        debug_file 'list'
-        check_output_includes "[3, 12] in #{fullpath('list')}"
-      end
-
-      it 'must list whole file if number of lines is smaller than listsize' do
-        enter 'set listsize 13', 'list'
-        debug_file 'list'
-        check_output_includes "[1, 12] in #{fullpath('list')}"
-      end
+  def after_setup
+    LineCache.clear_file_cache
+    Byebug::Command.settings[:listsize] = 3
+    Byebug::Command.settings[:autolist] = 0
+  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
 
-    describe 'without arguments' do
-      before { Byebug::Command.settings[:listsize] = 3 }
-
-      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
+    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
 
-    describe 'list backward' do
-      before { Byebug::Command.settings[:listsize] = 3 }
+    it 'must move range up when it goes before begining of file' do
+      enter 'set listsize 10', 'break 3', 'cont', 'list'
+      debug_file 'list'
+      check_output_includes "[1, 10] in #{fullpath('list')}"
+    end
 
-      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 move range down when it goes after end of file' do
+      enter 'set listsize 10', 'break 10', 'cont', 'list'
+      debug_file 'list'
+      check_output_includes "[3, 12] in #{fullpath('list')}"
+    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'
-      end
+    it 'must list whole file if number of lines is smaller than listsize' do
+      enter 'set listsize 13', 'list'
+      debug_file 'list'
+      check_output_includes "[1, 12] in #{fullpath('list')}"
     end
 
+  end
 
-    describe 'list surrounding' do
-      before { Byebug::Command.settings[:listsize] = 3 }
+  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 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
+    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 'autolist' do
-      before { Byebug::Command.settings[:listsize] = 3 }
+  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 show the surronding lines after stop 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
+    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
 
-    describe 'specified lines' do
-      before { Byebug::Command.settings[:listsize] = 3 }
 
-      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 'list surrounding' do
+    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
 
-      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
+    it 'must show the surronding lines after stop 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_error_includes 'Invalid line range'
-        check_output_doesnt_include "[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
+    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
-      after do
-        change_line_in_file(fullpath('list'), 4, '4')
-        Byebug::Command.settings[:reload_source_on_change] = true
-      end
+    it 'must show nothing if there is no such lines' do
+      enter 'list 44,44'
+      debug_file 'list'
+      check_error_includes 'Invalid line range'
+      check_output_doesnt_include "[44, 44] in #{fullpath('list')}"
+      check_output_doesnt_include /^44  \S/
+    end
 
-      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 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 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
+  describe 'reload source' do
+    after do
+      change_line_in_file(fullpath('list'), 4, '4')
+      Byebug::Command.settings[:reload_source_on_change] = true
     end
 
-    it 'must show an error when there is no such file' do
-      enter ->{state.file = 'blabla'; '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 'No sourcefile available for blabla',
-                            interface.error_queue
+      check_output_includes '4  4'
     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')}"
+    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

@@ -1,71 +1,70 @@
 require_relative 'test_helper'
 
-describe "Method Command" do
+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.
-
   def after_setup
     Byebug::Command.settings[:autolist] = 0
   end
 
-  describe "show instance method of a class" do
-    it "must show using full command name" do
-      enter 'break 15', 'cont', 'm MethodEx'
+  describe 'show instance method of a class' do
+    it 'must show using full command name' do
+      enter 'break 15', 'cont', 'method 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'
+    it 'must show using shortcut' do
+      enter 'break 15', 'cont', 'm MethodEx'
       debug_file 'method'
       check_output_includes /bla/
     end
 
-    it "must show an error if specified object is not a class or module" do
+    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"
+      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
+  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
+    it 'must show using shortcut' do
       enter 'break 15', 'cont', 'm i a'
       debug_file 'method'
       check_output_includes /bla/
     end
   end
 
+  describe 'show signature of a method' do
+    it 'must work' do
+      skip('TODO, can\'t install ruby-internal gem')
+    end
+  end
 
-  describe "show instance variables of an object" do
-    it "must show using full name command" do
+  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
+    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
-      skip("No post morten mode for now")
+  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/