byebug 1.0.3 → 1.1.0

data/test/post_mortem_test.rb

@@ -4,26 +4,23 @@ describe 'Post Mortem' do
   include TestDsl
 
   it 'must enter into post mortem mode' do
-    skip('No post morten mode for now')
     enter 'cont'
     debug_file('post_mortem') { Byebug.post_mortem?.must_equal true }
   end
 
   it 'must stop at the correct line' do
-    skip('No post morten mode for now')
     enter 'cont'
     debug_file('post_mortem') { state.line.must_equal 8 }
   end
 
   it 'must exit from post mortem mode after stepping command' do
-    skip('No post morten mode for now')
     enter 'cont', 'break 12', 'cont'
     debug_file('post_mortem') { Byebug.post_mortem?.must_equal false }
   end
 
   it 'must save the raised exception' do
-    skip('No post morten mode for now')
     enter 'cont'
-    debug_file('post_mortem') { Byebug.last_exception.must_be_kind_of RuntimeError }
+    debug_file('post_mortem') {
+      Byebug.last_exception.must_be_kind_of RuntimeError }
   end
 end

data/test/quit_test.rb

@@ -1,55 +1,54 @@
 require_relative 'test_helper'
 
-describe "Quit Command" do
+describe 'Quit Command' do
   include TestDsl
 
-  it "must quit if user confirmed" do
+  it 'must quit if user confirmed' do
     Byebug::QuitCommand.any_instance.expects(:exit!)
     enter 'quit', 'y'
     debug_file 'quit'
-    check_output_includes "Really quit? (y/n)", interface.confirm_queue
+    check_output_includes 'Really quit? (y/n)', interface.confirm_queue
   end
 
-  it "must not quit if user didn't confirm" do
+  it 'must not quit if user didn\'t confirm' do
     Byebug::QuitCommand.any_instance.expects(:exit!).never
     enter 'quit', 'n'
     debug_file 'quit'
-    check_output_includes "Really quit? (y/n)", interface.confirm_queue
+    check_output_includes 'Really quit? (y/n)', interface.confirm_queue
   end
 
-  it "must quit immediatly if used with !" do
+  it 'must quit immediatly if used with !' do
     Byebug::QuitCommand.any_instance.expects(:exit!)
     enter 'quit!'
     debug_file 'quit'
-    check_output_doesnt_include "Really quit? (y/n)", interface.confirm_queue
+    check_output_doesnt_include 'Really quit? (y/n)', interface.confirm_queue
   end
 
-  it "must quit immediatly if used with 'unconditionally'" do
+  it 'must quit immediatly if used with "unconditionally"' do
     Byebug::QuitCommand.any_instance.expects(:exit!)
     enter 'quit unconditionally'
     debug_file 'quit'
-    check_output_doesnt_include "Really quit? (y/n)", interface.confirm_queue
+    check_output_doesnt_include 'Really quit? (y/n)', interface.confirm_queue
   end
 
-  it "must finalize interface before quitting" do
+  it 'must finalize interface before quitting' do
     Byebug::QuitCommand.any_instance.stubs(:exit!)
     interface.expects(:finalize)
     enter 'quit!'
     debug_file 'quit'
   end
 
-  it "must quit if used 'exit' alias" do
+  it 'must quit if used "exit" alias' do
     Byebug::QuitCommand.any_instance.expects(:exit!)
     enter 'exit!'
     debug_file 'quit'
   end
 
-  describe "Post Mortem" do
-    it "must work in post-mortem mode" do
-      skip("No post morten mode for now")
-      #Byebug::QuitCommand.any_instance.expects(:exit!)
-      #enter 'cont', 'exit!'
-      #debug_file 'post_mortem'
+  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
 

data/test/reload_test.rb

@@ -34,9 +34,9 @@ describe 'Reload Command' do
   end
 
   describe 'Post Mortem' do
-     after { change_line_in_file(fullpath('post_mortem'), 7, '        z = 4') }
+    after { change_line_in_file(fullpath('post_mortem'), 7, '        z = 4') }
+
     it 'must work in post-mortem mode' do
-      skip('No post morten mode for now')
       enter 'cont', -> do
         change_line_in_file(fullpath('post_mortem'), 7, 'z = 100')
         'reload'

data/test/restart_test.rb

@@ -143,7 +143,6 @@ describe 'Restart Command' do
 
   describe 'Post Mortem' do
     it 'must work in post-mortem mode' do
-      skip('No post morten mode for now')
       must_restart
       enter 'cont', 'restart'
       debug_file 'post_mortem'

data/test/save_test.rb

@@ -1,9 +1,9 @@
 require_relative 'test_helper'
 
-describe "Save Command" do
+describe 'Save Command' do
   include TestDsl
 
-  describe "successful saving" do
+  describe 'successful saving' do
     let(:file_name) { 'save_output.txt' }
     let(:file_contents) { File.read(file_name) }
     before do
@@ -16,77 +16,76 @@ describe "Save Command" do
       FileUtils.rm(file_name)
     end
 
-    it "must save usual breakpoints" do
+    it 'must save usual breakpoints' do
       file_contents.must_include "break #{fullpath('save')}:2"
     end
 
-    it "must save conditinal breakpoints" do
+    it 'must save conditinal breakpoints' do
       file_contents.must_include "break #{fullpath('save')}:3 if true"
     end
 
-    it "must save catchpoints" do
-      file_contents.must_include "catch NoMethodError"
+    it 'must save catchpoints' do
+      file_contents.must_include 'catch NoMethodError'
     end
 
     # Not sure why it is suppressed, but this is like it is now.
-    it "must not save displays" do
-      file_contents.wont_include "display 2 + 3"
+    it 'must not save displays' do
+      file_contents.wont_include 'display 2 + 3'
     end
 
-    describe "saving settings" do
-      it "must save autoeval" do
-        file_contents.must_include "set autoeval on"
+    describe 'saving settings' do
+      it 'must save autoeval' do
+        file_contents.must_include 'set autoeval on'
       end
 
-      it "must save basename" do
-        file_contents.must_include "set basename off"
+      it 'must save basename' do
+        file_contents.must_include 'set basename off'
       end
 
-      it "must save byebugtesting" do
-        file_contents.must_include "set byebugtesting on"
+      it 'must save byebugtesting' do
+        file_contents.must_include 'set byebugtesting on'
       end
 
-      it "must save autolist" do
-        file_contents.must_include "set autolist on"
+      it 'must save autolist' do
+        file_contents.must_include 'set autolist on'
       end
 
-      it "must save autoirb" do
-        file_contents.must_include "set autoirb off"
+      it 'must save autoirb' do
+        file_contents.must_include 'set autoirb off'
       end
     end
 
-    it "must show a message about successful saving" do
+    it 'must show a message about successful saving' do
       check_output_includes "Saved to '#{file_name}'"
     end
 
   end
 
-  describe "without filename" do
+  describe 'without filename' do
     let(:file_contents) { File.read(interface.restart_file) }
     after { FileUtils.rm(interface.restart_file) }
 
-    it "must fabricate a filename if not provided" do
-      enter "save"
+    it 'must fabricate a filename if not provided' do
+      enter 'save'
       debug_file 'save'
-      file_contents.must_include "set autoirb"
+      file_contents.must_include 'set autoirb'
     end
 
-    it "must show a message where the file is saved" do
-      enter "save"
+    it 'must show a message where the file is saved' do
+      enter 'save'
       debug_file 'save'
       check_output_includes "Saved to '#{interface.restart_file}'"
     end
   end
 
-  describe "Post Mortem" do
+  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
-      skip("No post morten mode for now")
+    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"
+      file_contents.must_include 'set autoirb off'
     end
   end
 

data/test/set_test.rb

@@ -1,174 +1,177 @@
 require_relative 'test_helper'
 
-describe "Set Command" do
+describe 'Set Command' do
   include TestDsl
 
-  describe "setting to on" do
+  describe 'setting to on' do
     Byebug::Command.settings[:autolist] = 0
 
-    it "must set a setting to on" do
+    it 'must set a setting to on' do
       enter 'set autolist on'
       debug_file 'set'
       Byebug::Command.settings[:autolist].must_equal 1
     end
 
-    it "must set a setting to on by 1" do
+    it 'must set a setting to on by 1' do
       enter 'set autolist 1'
       debug_file 'set'
       Byebug::Command.settings[:autolist].must_equal 1
     end
 
-    it "must set a setting to on by default" do
+    it 'must set a setting to on by default' do
       enter 'set autolist'
       debug_file 'set'
       Byebug::Command.settings[:autolist].must_equal 1
     end
 
-    it "must set a setting using shortcut" do
+    it 'must set a setting using shortcut' do
       enter 'set autol'
       debug_file 'set'
       Byebug::Command.settings[:autolist].must_equal 1
     end
   end
 
-  describe "setting to off" do
+  describe 'setting to off' do
     Byebug::Command.settings[:autolist] = 1
 
-    it "must set a setting to off" do
+    it 'must set a setting to off' do
       enter 'set autolist off'
       debug_file 'set'
       Byebug::Command.settings[:autolist].must_equal 0
     end
 
-    it "must set a setting to off by 0" do
+    it 'must set a setting to off by 0' do
       enter 'set autolist 0'
       debug_file 'set'
       Byebug::Command.settings[:autolist].must_equal 0
     end
 
-    it "must set a setting to off by 'no' suffix" do
+    it 'must set a setting to off by "no" prefix' do
       enter 'set noautolist'
       debug_file 'set'
       Byebug::Command.settings[:autolist].must_equal 0
     end
   end
 
-  describe "messages" do
+  describe 'messages' do
     Byebug::Command.settings[:autolist] = 0
 
-    it "must show a message after setting" do
+    it 'must show a message after setting' do
       enter 'set autolist on'
       debug_file 'set'
-      check_output_includes "autolist is on."
+      check_output_includes 'autolist is on.'
     end
   end
 
-  describe "byebugtesting" do
-    it "must set $byebug_state if byebugsetting is on" do
+  describe 'byebugtesting' do
+    it 'must set $byebug_state if byebugsetting is on' do
       enter 'set byebugtesting', 'break 3', 'cont'
       debug_file('set') {
         state.must_be_kind_of Byebug::CommandProcessor::State }
     end
 
-    it "must set basename on too" do
+    it 'must set basename on too' do
       temporary_change_hash_value(Byebug::Command.settings, :basename, false) do
         enter 'set byebugtesting', 'show basename'
         debug_file('set')
-        check_output_includes "basename is on."
+        check_output_includes 'basename is on.'
       end
     end
 
-    it "must not set $byebug_state if byebugsetting is off" do
+    it 'must not set $byebug_state if byebugsetting is off' do
       enter 'set nobyebugtesting', 'break 3', 'cont'
       debug_file('set') { $byebug_state.must_be_nil }
     end
   end
 
-  describe "history" do
-    describe "save" do
-      it "must set history save to on" do
+  describe 'history' do
+    describe 'save' do
+      it 'must set history save to on' do
         enter 'set history save on'
         debug_file 'set'
         interface.history_save.must_equal true
       end
 
-      it "must show a message" do
+      it 'must show a message' do
         enter 'set history save on'
         debug_file 'set'
-        check_output_includes "Saving of history save is on."
+        check_output_includes 'Saving of history save is on.'
       end
 
-      it "must set history save to off" do
+      it 'must set history save to off' do
         enter 'set history save off'
         debug_file 'set'
         interface.history_save.must_equal false
       end
     end
 
-    describe "size" do
-      it "must set history size" do
+    describe 'size' do
+      it 'must set history size' do
         enter 'set history size 250'
         debug_file 'set'
         interface.history_length.must_equal 250
       end
 
-      it "must show a message" do
+      it 'must show a message' do
         enter 'set history size 250'
         debug_file 'set'
-        check_output_includes "Byebug history size is 250"
+        check_output_includes 'Byebug history size is 250'
       end
     end
 
-    describe "filename" do
-      it "must set history filename" do
+    describe 'filename' do
+      it 'must set history filename' do
         enter 'set history filename .byebug-hist'
         debug_file 'set'
-        interface.histfile.must_equal File.join(ENV["HOME"]||ENV["HOMEPATH"]||".", '.byebug-hist')
+        interface.histfile.must_equal \
+          File.join(ENV['HOME']||ENV['HOMEPATH']||'.', '.byebug-hist')
       end
 
-      it "must show a message" do
+      it 'must show a message' do
         enter 'set history filename .byebug-hist'
         debug_file 'set'
-        check_output_includes "The filename in which to record the command history is \"#{File.join(ENV["HOME"]||ENV["HOMEPATH"]||".", ".byebug-hist")}\""
+        check_output_includes \
+          'The filename in which to record the command history is ' \
+          "\"#{File.join(ENV['HOME']||ENV['HOMEPATH']||'.', '.byebug-hist')}\""
       end
     end
 
-    it "must show an error message if used wrong subcommand" do
+    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'."
+      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
+    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."
+      check_output_includes 'Need two parameters for "set history"; got 1.'
     end
   end
 
-  describe "width" do
+  describe 'width' do
     Byebug::Command.settings[:width] = 20
 
-    it "must set ENV['COLUMNS'] by the 'set width' command" do
-      old_columns = ENV["COLUMNS"]
+    it 'must set ENV[\'COLUMNS\'] by the "set width" command' do
+      old_columns = ENV['COLUMNS']
       begin
         enter 'set width 10'
         debug_file 'set'
-        ENV["COLUMNS"].must_equal '10'
+        ENV['COLUMNS'].must_equal '10'
       ensure
-        ENV["COLUMNS"] = old_columns
+        ENV['COLUMNS'] = old_columns
       end
     end
   end
 
-  describe "Post Mortem" do
+  describe 'Post Mortem' do
     Byebug::Command.settings[:autolist] = 0
 
-    it "must work in post-mortem mode" do
-      skip("No post morten mode for now")
-      enter 'cont', "set autolist on"
+    it 'must work in post-mortem mode' do
+      enter 'cont', 'set autolist on'
       debug_file 'post_mortem'
-      check_output_includes "autolist is on."
+      check_output_includes 'autolist is on.'
     end
   end