byebug 1.0.3 → 1.1.0

data/test/show_test.rb

@@ -1,39 +1,41 @@
 require_relative 'test_helper'
 
-describe "Show Command" do
+describe 'Show Command' do
   include TestDsl
 
-  describe "annotate" do
-    it "must show annotate setting" do
+  describe 'annotate' do
+    it 'must show annotate setting' do
       enter 'show annotate'
       debug_file 'show'
       Byebug.annotate.must_equal 0
-      check_output_includes "Annotation level is 0"
+      check_output_includes 'Annotation level is 0'
     end
   end
 
-  describe "args" do
+  describe 'args' do
     before do
       Byebug::Command.settings[:argv] = %w{foo bar}
     end
 
-    it "must show args" do
+    it 'must show args' do
       enter 'show args'
       debug_file 'show'
-      check_output_includes 'Argument list to give program being debugged when it is started is "foo bar".'
+      check_output_includes 'Argument list to give program being debugged ' \
+                            'when it is started is "foo bar".'
     end
 
-    it "must not show the first arg if BYEBUG_SCRIPT is defined" do
-      temporary_set_const(Byebug, "BYEBUG_SCRIPT", "bla") do
+    it 'must not show the first arg if BYEBUG_SCRIPT is defined' do
+      temporary_set_const(Byebug, 'BYEBUG_SCRIPT', 'bla') do
         enter 'show args'
         debug_file 'show'
-        check_output_includes 'Argument list to give program being debugged when it is started is "bar".'
+        check_output_includes 'Argument list to give program being debugged ' \
+                              'when it is started is "bar".'
       end
     end
   end
 
 
-  it "must show autolist" do
+  it 'must show autolist' do
     temporary_change_hash_value(Byebug::Command.settings, :autolist, 1) do
       enter 'show autolist'
       debug_file 'show'
@@ -41,7 +43,7 @@ describe "Show Command" do
     end
   end
 
-  it "must show autoeval" do
+  it 'must show autoeval' do
     temporary_change_hash_value(Byebug::Command.settings, :autoeval, true) do
       enter 'show autoeval'
       debug_file 'show'
@@ -49,7 +51,7 @@ describe "Show Command" do
     end
   end
 
-  it "must show autoreload" do
+  it 'must show autoreload' do
     temporary_change_hash_value(Byebug::Command.settings, :reload_source_on_change, true) do
       enter 'show autoreload'
       debug_file 'show'
@@ -57,7 +59,7 @@ describe "Show Command" do
     end
   end
 
-  it "must show autoirb" do
+  it 'must show autoirb' do
     Byebug::IRBCommand.any_instance.stubs(:execute)
     temporary_change_hash_value(Byebug::Command.settings, :autoirb, 1) do
       enter 'show autoirb'
@@ -66,7 +68,7 @@ describe "Show Command" do
     end
   end
 
-  it "must show basename" do
+  it 'must show basename' do
     temporary_change_hash_value(Byebug::Command.settings, :basename, true) do
       enter 'show basename'
       debug_file 'show'
@@ -74,7 +76,7 @@ describe "Show Command" do
     end
   end
 
-  it "must show callstyle" do
+  it 'must show callstyle' do
     temporary_change_hash_value(Byebug::Command.settings, :callstyle, :short) do
       enter 'show callstyle'
       debug_file 'show'
@@ -82,7 +84,7 @@ describe "Show Command" do
     end
   end
 
-  it "must show forcestep" do
+  it 'must show forcestep' do
     temporary_change_hash_value(Byebug::Command.settings, :force_stepping, true) do
       enter 'show forcestep'
       debug_file 'show'
@@ -90,40 +92,39 @@ describe "Show Command" do
     end
   end
 
-  it "must show fullpath" do
+  it 'must show fullpath' do
     temporary_change_hash_value(Byebug::Command.settings, :full_path, true) do
       enter 'show fullpath'
       debug_file 'show'
-      check_output_includes "Displaying frame's full file names is on."
+      check_output_includes 'Displaying frame\'s full file names is on.'
     end
   end
 
-  it "must show linetrace" do
+  it 'must show linetrace' do
     enter 'trace on', 'show linetrace', 'trace off'
     debug_file 'show'
-    check_output_includes "line tracing is on."
+    check_output_includes 'line tracing is on.'
   end
 
-  describe "linetrace+" do
-    it "must show a message when linetrace+ is on" do
+  describe 'linetrace+' do
+    it 'must show a message when linetrace+ is on' do
       temporary_change_hash_value(Byebug::Command.settings, :tracing_plus, true) do
         enter 'show linetrace+'
         debug_file 'show'
-        check_output_includes "line tracing style is different consecutive lines."
+        check_output_includes 'line tracing style is different consecutive lines.'
       end
     end
 
-    it "must show a message when linetrace+ is off" do
+    it 'must show a message when linetrace+ is off' do
       temporary_change_hash_value(Byebug::Command.settings, :tracing_plus, false) do
         enter 'show linetrace+'
         debug_file 'show'
-        check_output_includes "line tracing style is every line."
+        check_output_includes 'line tracing style is every line.'
       end
     end
   end
 
-
-  it "must show listsize" do
+  it 'must show listsize' do
     temporary_change_hash_value(Byebug::Command.settings, :listsize, 10) do
       enter 'show listsize'
       debug_file 'show'
@@ -131,29 +132,29 @@ describe "Show Command" do
     end
   end
 
-  it "must show port" do
-    temporary_set_const(Byebug, "PORT", 12345) do
+  it 'must show port' do
+    temporary_set_const(Byebug, 'PORT', 12345) do
       enter 'show port'
       debug_file 'show'
       check_output_includes 'server port is 12345.'
     end
   end
 
-  it "must show trace" do
+  it 'must show trace' do
     temporary_change_hash_value(Byebug::Command.settings, :stack_trace_on_error, true) do
       enter 'show trace'
       debug_file 'show'
-      check_output_includes "Displaying stack trace is on."
+      check_output_includes 'Displaying stack trace is on.'
     end
   end
 
-  it "must show version" do
+  it 'must show version' do
     enter 'show version'
     debug_file 'show'
     check_output_includes "byebug #{Byebug::VERSION}"
   end
 
-  it "must show forcestep" do
+  it 'must show forcestep' do
     temporary_change_hash_value(Byebug::Command.settings, :width, 35) do
       enter 'show width'
       debug_file 'show'
@@ -161,52 +162,54 @@ describe "Show Command" do
     end
   end
 
-  it "must show a message about unknown command" do
+  it 'must show a message about unknown command' do
     enter 'show bla'
     debug_file 'show'
     check_output_includes 'Unknown show command bla'
   end
 
 
-  describe "history" do
-    describe "without arguments" do
+  describe 'history' do
+    describe 'without arguments' do
       before do
-        interface.histfile = "hist_file.txt"
+        interface.histfile = 'hist_file.txt'
         interface.history_save = true
         interface.history_length = 25
         enter 'show history'
         debug_file 'show'
       end
 
-      it "must show history file" do
-        check_output_includes /filename: The filename in which to record the command history is "hist_file\.txt"/
+      it 'must show history file' do
+        check_output_includes \
+          /The filename in which to record the command history is "hist_file\.txt"/
       end
 
-      it "must show history save setting" do
+      it 'must show history save setting' do
         check_output_includes /save: Saving of history save is on\./
       end
 
-      it "must show history length" do
+      it 'must show history length' do
         check_output_includes /size: Byebug history size is 25/
       end
     end
 
-    describe "with 'filename' argument" do
-      it "must show history filename" do
-        interface.histfile = "hist_file.txt"
+    describe 'with "filename" argument' do
+      it 'must show history filename' do
+        interface.histfile = 'hist_file.txt'
         enter 'show history filename'
         debug_file 'show'
-        check_output_includes 'The filename in which to record the command history is "hist_file.txt"'
+        check_output_includes 'The filename in which to record the command ' \
+                              'history is "hist_file.txt"'
       end
 
-      it "must show history save setting" do
+      it 'must show history save setting' do
         interface.history_save = true
         enter 'show history save'
         debug_file 'show'
         check_output_includes 'Saving of history save is on.'
       end
 
-      it "must show history length" do
+      it 'must show history length' do
         interface.history_length = 30
         enter 'show history size'
         debug_file 'show'
@@ -215,19 +218,18 @@ describe "Show Command" do
     end
   end
 
-
-  describe "commands" do
+  describe 'commands' do
     before { interface.readline_support = true }
 
-    it "must not show records from readline if there is no readline support" do
+    it 'must not show records from readline if there is no readline support' do
       interface.readline_support = false
       enter 'show commands'
       debug_file 'show'
-      check_output_includes "No readline support"
+      check_output_includes 'No readline support'
     end
 
-    it "must show records from readline history" do
-      temporary_set_const(Readline, "HISTORY", %w{aaa bbb ccc ddd eee fff}) do
+    it 'must show records from readline history' do
+      temporary_set_const(Readline, 'HISTORY', %w{aaa bbb ccc ddd eee fff}) do
         enter 'show commands'
         debug_file 'show'
         check_output_includes /1  aaa/
@@ -235,8 +237,8 @@ describe "Show Command" do
       end
     end
 
-    it "must show last 10 records from readline history" do
-      temporary_set_const(Readline, "HISTORY", %w{aaa bbb ccc ddd eee fff ggg hhh iii jjj kkk lll mmm nnn}) do
+    it 'must show last 10 records from readline history' do
+      temporary_set_const(Readline, 'HISTORY', %w{aaa bbb ccc ddd eee fff ggg hhh iii jjj kkk lll mmm nnn}) do
         enter 'show commands'
         debug_file 'show'
         check_output_doesnt_include /3  ccc/
@@ -245,9 +247,9 @@ describe "Show Command" do
       end
     end
 
-    describe "with specified positions" do
-      it "must show records within boundaries" do
-        temporary_set_const(Readline, "HISTORY", %w{aaa bbb ccc ddd eee fff ggg hhh iii jjj kkk lll mmm nnn}) do
+    describe 'with specified positions' do
+      it 'must show records within boundaries' do
+        temporary_set_const(Readline, 'HISTORY', %w{aaa bbb ccc ddd eee fff ggg hhh iii jjj kkk lll mmm nnn}) do
           # Really don't know why it substracts 4, and shows starting from position 6
           enter 'show commands 10'
           debug_file 'show'
@@ -257,8 +259,8 @@ describe "Show Command" do
         end
       end
 
-      it "must adjust first line if it is < 0" do
-        temporary_set_const(Readline, "HISTORY", %w{aaa bbb ccc ddd eee fff ggg hhh iii jjj kkk lll mmm nnn}) do
+      it 'must adjust first line if it is < 0' do
+        temporary_set_const(Readline, 'HISTORY', %w{aaa bbb ccc ddd eee fff ggg hhh iii jjj kkk lll mmm nnn}) do
           enter 'show commands 3'
           debug_file 'show'
           check_output_includes /1  bbb/
@@ -269,12 +271,11 @@ describe "Show Command" do
     end
   end
 
-  describe "Post Mortem" do
-    it "must work in post-mortem mode" do
-      skip("No post morten mode for now")
-      enter 'cont', "show autolist"
+  describe 'Post Mortem' do
+    it 'must work in post-mortem mode' do
+      enter 'cont', 'show autolist'
       debug_file 'post_mortem'
-      check_output_includes "autolist is on."
+      check_output_includes 'autolist is on.'
     end
   end
 

data/test/source_test.rb

@@ -1,48 +1,45 @@
 require_relative 'test_helper'
 
-describe "Source Command" do
+describe 'Source Command' do
   include TestDsl
 
-  describe "Source Command (Setup)" do
-    let(:filename) { 'source_example.txt' }
-    before do
-      File.open(filename, 'w') do |f|
-        f.puts 'break 2'
-        f.puts 'break 3 if true'
-      end
-    end
-    after do
-      FileUtils.rm(filename)
-    end
+  let(:filename) { 'source_example.txt' }
 
-    it "must run commands from file" do
-      enter "source #{filename}"
-      debug_file 'source' do
-        Byebug.breakpoints[0].pos.must_equal 2
-        Byebug.breakpoints[1].pos.must_equal 3
-        Byebug.breakpoints[1].expr.must_equal "true"
-      end
+  def after_setup
+    File.open(filename, 'w') do |f|
+      f.puts 'break 2'
+      f.puts 'break 3 if true'
     end
+  end
 
-    it "must be able to use shortcut" do
-      enter "so #{filename}"
-      debug_file('source') { Byebug.breakpoints[0].pos.must_equal 2 }
-    end
+  def before_teardown
+    FileUtils.rm(filename)
+  end
 
-    it "must show an error if file is not found" do
-      enter "source blabla"
-      debug_file 'source'
-      check_output_includes /Command file '.*blabla' is not found/, interface.error_queue
+  it 'must run commands from file' do
+    enter "source #{filename}"
+    debug_file 'source' do
+      Byebug.breakpoints[0].pos.must_equal 2
+      Byebug.breakpoints[1].pos.must_equal 3
+      Byebug.breakpoints[1].expr.must_equal 'true'
     end
+  end
 
-    describe "Post Mortem" do
-      it "must work in post-mortem mode" do
-        skip("No post morten mode for now")
-        enter 'cont', "so #{filename}"
-        debug_file('post_mortem') { Byebug.breakpoints[0].pos.must_equal 3 }
-      end
-    end
+  it 'must be able to use shortcut' do
+    enter "so #{filename}"
+    debug_file('source') { Byebug.breakpoints[0].pos.must_equal 2 }
+  end
 
+  it 'must show an error if file is not found' do
+    enter 'source blabla'
+    debug_file 'source'
+    check_output_includes /Command file '.*blabla' is not found/, interface.error_queue
   end
 
+  describe 'Post Mortem' do
+    it 'must work in post-mortem mode' do
+      enter 'cont', "so #{filename}"
+      debug_file('post_mortem') { Byebug.breakpoints[0].pos.must_equal 3 }
+    end
+  end
 end

data/test/stepping_test.rb

@@ -1,119 +1,117 @@
 require_relative 'test_helper'
 
-describe "Stepping Commands" do
+describe 'Stepping Commands' do
   include TestDsl
 
-  describe "Next Command" do
+  describe 'Next Command' do
 
-    describe "Usual mode" do
+    describe 'Usual mode' do
       before do
         enter 'break 10', 'cont'
       end
 
-      it "must go to the next line if forced by a setting" do
+      it 'must go to the next line if forced by a setting' do
         Byebug::Command.settings[:force_stepping] = true
         enter 'next'
         debug_file('stepping') { state.line.must_equal 11 }
       end
 
-      it "must go to the next line if forced by a setting (by shortcut)" do
+      it 'must go to the next line if forced by a setting (by shortcut)' do
         Byebug::Command.settings[:force_stepping] = true
         enter 'n'
         debug_file('stepping') { state.line.must_equal 11 }
       end
 
-      it "must leave on the same line if forced by a setting" do
-        Byebug::Command.settings[:force_stepping] = false
+      it 'must leave on the same line by default' do
         enter 'next'
         debug_file('stepping') { state.line.must_equal 10 }
       end
 
-      it "must go the specified number of lines forward by default" do
+      it 'must go the specified number of lines forward by default' do
         Byebug::Command.settings[:force_stepping] = true
         enter 'next 2'
         debug_file('stepping') { state.line.must_equal 21 }
       end
 
-      it "must go to the next line if forced by 'plus' sign" do
+      it 'must go to the next line if forced by "plus" sign' do
         enter 'next+'
         debug_file('stepping') { state.line.must_equal 11 }
       end
 
-      it "must leave on the same line if forced by 'minus' sign" do
+      it 'must leave on the same line if forced by "minus" sign' do
         enter 'next-'
         debug_file('stepping') { state.line.must_equal 10 }
       end
 
-      it "must ignore the setting if 'minus' is specified" do
+      it 'must ignore the setting if "minus" is specified' do
         Byebug::Command.settings[:force_stepping] = true
         enter 'next-'
         debug_file('stepping') { state.line.must_equal 10 }
       end
     end
 
-    describe "Post Mortem" do
-      before { Byebug::Command.settings[:autoeval] = 0 }
+    describe 'Post Mortem' do
+      before { Byebug::Command.settings[:autoeval] = false }
 
-      it "must not work in post-mortem mode" do
-        skip("No post morten mode for now")
-        enter 'cont', "next"
-        debug_file('post_mortem')
-        check_output_includes 'Unknown command: "next".  Try "help".', interface.error_queue
+      it 'must not work in post-mortem mode' do
+        enter 'cont', 'next'
+        debug_file 'post_mortem'
+        check_output_includes \
+          'Unknown command: "next".  Try "help".', interface.error_queue
       end
     end
   end
 
-  describe "Step Command" do
+  describe 'Step Command' do
 
-    describe "Usual mode" do
+    describe 'Usual mode' do
 
       before do
         enter 'break 10', 'cont'
       end
 
-      it "must go to the step line if forced by a setting" do
+      it 'must go to the step line if forced by a setting' do
         Byebug::Command.settings[:force_stepping] = true
         enter 'step'
         debug_file('stepping') { state.line.must_equal 11 }
       end
 
-      it "must go to the next line by shortcut" do
+      it 'must go to the next line if forced by a setting (by shortcut)' do
         Byebug::Command.settings[:force_stepping] = true
         enter 's'
         debug_file('stepping') { state.line.must_equal 11 }
       end
 
-      it "must leave on the same line if forced by a setting" do
-        Byebug::Command.settings[:force_stepping] = false
+      it 'must leave on the same line if forced by a setting' do
         enter 'step'
         debug_file('stepping') { state.line.must_equal 10 }
       end
 
-      it "must go the specified number of lines forward by default" do
+      it 'must go the specified number of lines forward by default' do
         Byebug::Command.settings[:force_stepping] = true
         enter 'step 2'
         debug_file('stepping') { state.line.must_equal 15 }
       end
 
-      it "must go to the step line if forced to do that by 'plus' sign" do
+      it 'must go to the step line if forced to do that by "plus" sign' do
         enter 'step+'
         debug_file('stepping') { state.line.must_equal 11 }
       end
 
-      it "must leave on the same line if forced to do that by 'minus' sign" do
+      it 'must leave on the same line if forced to do that by "minus" sign' do
         enter 'step-'
         debug_file('stepping') { state.line.must_equal 10 }
       end
     end
 
-    describe "Post Mortem" do
-      before { Byebug::Command.settings[:autoeval] = 0 }
+    describe 'Post Mortem' do
+      before { Byebug::Command.settings[:autoeval] = false }
 
-      it "must not work in post-mortem mode" do
-        skip("No post morten mode for now")
-        enter 'cont', "step"
-        debug_file('post_mortem')
-        check_output_includes 'Unknown command: "step".  Try "help".', interface.error_queue
+      it 'must not work in post-mortem mode' do
+        enter 'cont', 'step'
+        debug_file 'post_mortem'
+        check_output_includes \
+          'Unknown command: "step".  Try "help".', interface.error_queue
       end
     end
   end