tailor 1.4.0 → 1.4.1

data/spec/unit/tailor/lexer/token_spec.rb

@@ -3,7 +3,7 @@ require 'tailor/lexer/token'
 
 describe Tailor::Lexer::Token do
   before do
-    Tailor::Logger.stub(:log)
+    allow(Tailor::Logger).to receive(:log)
   end
 
   describe '#modifier_keyword?' do
@@ -14,24 +14,24 @@ describe Tailor::Lexer::Token do
 
     context 'the current line has a keyword that is also a modifier' do
       context 'the keyword is acting as a modifier' do
-        let!(:full_line_of_text) { %q{puts "hi" if true == true} }
+        let!(:full_line_of_text) { %(puts "hi" if true == true) }
 
         it 'returns true' do
-          subject.modifier_keyword?.should be_true
+          expect(subject.modifier_keyword?).to eq true
         end
       end
 
       context 'they keyword is NOT acting as a modifier' do
-        let!(:full_line_of_text) { %q{if true == true; puts "hi"; end} }
+        let!(:full_line_of_text) { %(if true == true; puts "hi"; end) }
 
         it 'returns false' do
-          subject.modifier_keyword?.should be_false
+          expect(subject.modifier_keyword?).to eq false
         end
       end
     end
 
     context 'the current line does not have a keyword' do
-      let!(:full_line_of_text) { %q{puts true} }
+      let!(:full_line_of_text) { %(puts true) }
 
       subject do
         options = { full_line_of_text: full_line_of_text }
@@ -39,7 +39,7 @@ describe Tailor::Lexer::Token do
       end
 
       it 'returns false' do
-        subject.modifier_keyword?.should be_false
+        expect(subject.modifier_keyword?).to eq false
       end
     end
   end

data/spec/unit/tailor/lexer_spec.rb

@@ -9,13 +9,13 @@ describe Tailor::Lexer do
   subject do
     r = Tailor::Lexer.new(file_text)
     r.instance_variable_set(:@buf, [])
-    r.stub(:log)
+    allow(r).to receive(:log)
 
     r
   end
 
   before do
-    Tailor::Lexer.any_instance.stub(:ensure_trailing_newline).
+    allow_any_instance_of(Tailor::Lexer).to receive(:ensure_trailing_newline).
       and_return(file_text)
   end
 
@@ -29,8 +29,8 @@ describe Tailor::Lexer do
 
       it 'opens and reads the file by the name passed in' do
         file = double 'File'
-        file.should_receive(:read).and_return file_text
-        File.should_receive(:open).with('test', 'r').and_return file
+        expect(file).to receive(:read).and_return file_text
+        expect(File).to receive(:open).with('test', 'r').and_return file
         Tailor::Lexer.new(file_name)
       end
     end
@@ -39,7 +39,7 @@ describe Tailor::Lexer do
       let(:text) { 'some text' }
 
       it 'does not try to open a file' do
-        File.should_not_receive(:open)
+        expect(File).to_not receive(:open)
         Tailor::Lexer.new(text)
       end
     end
@@ -48,7 +48,7 @@ describe Tailor::Lexer do
   describe '#on_sp' do
     context 'token is a backslash then newline' do
       it 'calls #notify_ignored_nl_observers' do
-        subject.should_receive(:notify_ignored_nl_observers)
+        expect(subject).to receive(:notify_ignored_nl_observers)
         subject.on_sp("\\\n")
       end
     end
@@ -57,43 +57,43 @@ describe Tailor::Lexer do
   describe '#current_line_of_text' do
     before do
       subject.instance_variable_set(:@file_text, file_text)
-      subject.stub(:lineno).and_return 1
+      allow(subject).to receive(:lineno).and_return 1
     end
 
-    context "@file_text is 1 line with 0 \\ns" do
+    context '@file_text is 1 line with 0 \ns' do
       let(:file_text) { "puts 'code'" }
 
       it 'returns the line' do
-        subject.current_line_of_text.should == file_text
+        expect(subject.current_line_of_text).to eq file_text
       end
     end
 
-    context "@file_text is 1 empty line with 0 \\ns" do
+    context '@file_text is 1 empty line with 0 \ns' do
       let(:file_text) { '' }
 
       it 'returns the an empty string' do
-        subject.current_line_of_text.should == file_text
+        expect(subject.current_line_of_text).to eq file_text
       end
     end
 
-    context "@file_text is 1 empty line with 1 \\n" do
+    context '@file_text is 1 empty line with 1 \n' do
       let(:file_text) { "\n" }
 
       it 'returns an empty string' do
-        subject.current_line_of_text.should == ''
+        expect(subject.current_line_of_text).to eq ''
       end
     end
   end
 
   describe '#count_trailing_newlines' do
-    context "text contains 0 trailing \\n" do
+    context 'text contains 0 trailing \n' do
       let(:text) { 'text' }
-      specify { subject.count_trailing_newlines(text).should be_zero }
+      specify { expect(subject.count_trailing_newlines(text)).to be_zero }
     end
 
-    context "text contains 1 trailing \\n" do
+    context 'text contains 1 trailing \n' do
       let(:text) { "text\n" }
-      specify { subject.count_trailing_newlines(text).should == 1 }
+      specify { expect(subject.count_trailing_newlines(text)).to eq 1 }
     end
   end
 
@@ -106,11 +106,11 @@ describe Tailor::Lexer do
       let!(:text) { "text\n" }
 
       before do
-        subject.stub(:count_trailing_newlines).and_return 1
+        allow(subject).to receive(:count_trailing_newlines).and_return 1
       end
 
       it 'does not alter the text' do
-        subject.ensure_trailing_newline(text).should == text
+        expect(subject.ensure_trailing_newline(text)).to eq text
       end
     end
 
@@ -118,15 +118,15 @@ describe Tailor::Lexer do
       let!(:text) { 'text' }
 
       it 'adds a newline at the end' do
-        subject.ensure_trailing_newline(text).should == text + "\n"
+        expect(subject.ensure_trailing_newline(text)).to eq(text + "\n")
       end
     end
   end
 
   describe '#sub_line_ending_backslashes' do
     let!(:text) do
-      %Q{command \\
-  'something'}
+      %(command \\
+  'something')
     end
 
     before do
@@ -136,8 +136,8 @@ describe Tailor::Lexer do
     end
 
     it 'replaces all line-ending backslashes with a comment' do
-      subject.sub_publicly(text).should == %Q{command # TAILOR REMOVED BACKSLASH
-  'something'}
+      expect(subject.sub_publicly(text)).to eq %(command # TAILOR REMOVED BACKSLASH
+  'something')
     end
   end
 end

data/spec/unit/tailor/problem_spec.rb

@@ -3,7 +3,7 @@ require 'tailor/problem'
 
 describe Tailor::Problem do
   before do
-    Tailor::Problem.any_instance.stub(:log)
+    allow_any_instance_of(Tailor::Problem).to receive(:log)
   end
 
   let(:lineno) { 10 }
@@ -11,32 +11,32 @@ describe Tailor::Problem do
 
   describe '#set_values' do
     before do
-      Tailor::Problem.any_instance.stub(:message)
+      allow_any_instance_of(Tailor::Problem).to receive(:message)
     end
 
     it 'sets self[:type] to the type param' do
-      Tailor::Problem.new(:test, lineno, column, '', :b).
-        should include(type: :test)
+      expect(Tailor::Problem.new(:test, lineno, column, '', :b)).
+        to include(type: :test)
     end
 
     it 'sets self[:line] to the lineno param' do
-      Tailor::Problem.new(:test, lineno, column, '', :c).
-        should include(line: lineno)
+      expect(Tailor::Problem.new(:test, lineno, column, '', :c)).
+        to include(line: lineno)
     end
 
     it 'sets self[:column] to the column param' do
-      Tailor::Problem.new(:test, lineno, column, '', :d).
-        should include(column: column)
+      expect(Tailor::Problem.new(:test, lineno, column, '', :d)).
+        to include(column: column)
     end
 
     it 'sets self[:message] to the message param' do
-      Tailor::Problem.new(:test, lineno, column, 'test', :d).
-        should include(message: 'test')
+      expect(Tailor::Problem.new(:test, lineno, column, 'test', :d)).
+        to include(message: 'test')
     end
 
     it 'sets self[:level] to the level param' do
-      Tailor::Problem.new(:test, lineno, column, 'test', :d).
-        should include(level: :d)
+      expect(Tailor::Problem.new(:test, lineno, column, 'test', :d)).
+        to include(level: :d)
     end
   end
 end

data/spec/unit/tailor/reporter_spec.rb

@@ -8,7 +8,7 @@ describe Tailor::Reporter do
 
       it 'creates a new Formatter object of the type passed in' do
         reporter = Tailor::Reporter.new(formats)
-        reporter.formatters.first.should be_a Tailor::Formatters::Text
+        expect(reporter.formatters.first).to be_a Tailor::Formatters::Text
       end
     end
   end
@@ -26,7 +26,7 @@ describe Tailor::Reporter do
 
     it 'calls #file_report on each @formatters' do
       label = :some_label
-      formatter.should_receive(:file_report).with(file_problems, label)
+      expect(formatter).to receive(:file_report).with(file_problems, label)
 
       subject.file_report(file_problems, label)
     end
@@ -45,8 +45,8 @@ describe Tailor::Reporter do
 
     context 'without output file' do
       it 'calls #file_report on each @formatters' do
-        formatter.should_receive(:summary_report).with(all_problems)
-        File.should_not_receive(:open)
+        expect(formatter).to receive(:summary_report).with(all_problems)
+        expect(File).to_not receive(:open)
 
         subject.summary_report(all_problems)
       end
@@ -55,18 +55,17 @@ describe Tailor::Reporter do
     context 'with output file' do
       let(:output_file) { 'output.whatever' }
       before do
-        formatter.should_receive(:respond_to?).with(:accepts_output_file).
+        expect(formatter).to receive(:respond_to?).with(:accepts_output_file).
           and_return(true)
-        formatter.should_receive(:accepts_output_file).and_return(true)
+        expect(formatter).to receive(:accepts_output_file).and_return(true)
       end
 
       it 'calls #summary_report on each @formatters' do
-        formatter.should_receive(:summary_report).with(all_problems)
-        File.should_receive(:open).with(output_file, 'w')
+        expect(formatter).to receive(:summary_report).with(all_problems)
+        expect(File).to receive(:open).with(output_file, 'w')
 
         subject.summary_report(all_problems, output_file: output_file)
       end
     end
-
   end
 end

data/spec/unit/tailor/ruler_spec.rb

@@ -2,54 +2,54 @@ require 'spec_helper'
 require 'tailor/ruler'
 
 describe Tailor::Ruler do
-  before { Tailor::Logger.stub(:log) }
+  before { allow(Tailor::Logger).to receive(:log) }
 
   describe '#add_child_ruler' do
     it 'adds new rulers to @child_rulers' do
       ruler = double 'Ruler'
       subject.add_child_ruler(ruler)
-      subject.instance_variable_get(:@child_rulers).first.should == ruler
+      expect(subject.instance_variable_get(:@child_rulers).first).to eq ruler
     end
   end
 
   describe '#problems' do
     context 'no child_rulers' do
       context '@problems is empty' do
-        specify { subject.problems.should be_empty }
+        specify { expect(subject.problems).to be_empty }
       end
 
       context '@problems.size is 1' do
         before do
           problem = double 'Problem'
-          problem.should_receive(:[]).with :line
+          expect(problem).to receive(:[]).with :line
           subject.instance_variable_set(:@problems, [problem])
         end
 
-        specify { subject.problems.size.should == 1 }
+        specify { expect(subject.problems.size).to eq 1 }
       end
     end
 
     context 'child_rulers have problems' do
       before do
         problem = double 'Problem'
-        problem.should_receive(:[]).with :line
+        expect(problem).to receive(:[]).with :line
         child_ruler = double 'Ruler'
-        child_ruler.stub(:problems).and_return([problem])
+        allow(child_ruler).to receive(:problems).and_return([problem])
         subject.instance_variable_set(:@child_rulers, [child_ruler])
       end
 
       context '@problems is empty' do
-        specify { subject.problems.size.should == 1 }
+        specify { expect(subject.problems.size).to eq 1 }
       end
 
       context '@problems.size is 1' do
         before do
           problem = double 'Problem'
-          problem.should_receive(:[]).with :line
+          expect(problem).to receive(:[]).with :line
           subject.instance_variable_set(:@problems, [problem])
         end
 
-        specify { subject.problems.size.should == 2 }
+        specify { expect(subject.problems.size).to eq 2 }
       end
     end
   end

data/spec/unit/tailor/rulers/indentation_spaces_ruler/indentation_manager_spec.rb

@@ -2,13 +2,12 @@ require 'ripper'
 require 'spec_helper'
 require 'tailor/rulers/indentation_spaces_ruler/indentation_manager'
 
-
 describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
   let!(:spaces) { 5 }
   let!(:lexed_line) { double 'LexedLine' }
 
   before do
-    Tailor::Logger.stub(:log)
+    allow(Tailor::Logger).to receive(:log)
     subject.instance_variable_set(:@spaces, spaces)
   end
 
@@ -18,8 +17,8 @@ describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
 
   describe '#should_be_at' do
     it 'returns @proper[:this_line]' do
-      subject.instance_variable_set(:@proper, { this_line: 321 })
-      subject.should_be_at.should == 321
+      subject.instance_variable_set(:@proper, this_line: 321)
+      expect(subject.should_be_at).to eq 321
     end
   end
 
@@ -27,66 +26,61 @@ describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
     let!(:spaces) { 27 }
 
     context '#started? is true' do
-      before { subject.stub(:started?).and_return true }
+      before { allow(subject).to receive(:started?).and_return true }
 
       context '@proper[:this_line] gets decremented < 0' do
         it 'sets @proper[:this_line] to 0' do
-          subject.instance_variable_set(:@proper, {
-            this_line: 0, next_line: 0
-          })
+          subject.instance_variable_set(:@proper, this_line: 0, next_line: 0)
 
           subject.decrease_this_line
           proper_indentation = subject.instance_variable_get(:@proper)
-          proper_indentation[:this_line].should == 0
+          expect(proper_indentation[:this_line]).to be_zero
         end
       end
 
       context '@proper[:this_line] NOT decremented < 0' do
         it 'decrements @proper[:this_line] by @spaces' do
-          subject.instance_variable_set(:@proper, {
-            this_line: 28, next_line: 28
-          })
+          subject.instance_variable_set(:@proper, this_line: 28, next_line: 28)
           subject.decrease_this_line
 
           proper_indentation = subject.instance_variable_get(:@proper)
-          proper_indentation[:this_line].should == 1
+          expect(proper_indentation[:this_line]).to eq 1
         end
       end
     end
 
     context '#started? is false' do
-      before { subject.stub(:started?).and_return false }
+      before { allow(subject).to receive(:started?).and_return false }
 
       it 'does not decrement @proper[:this_line]' do
-        subject.instance_variable_set(:@proper, {
-          this_line: 28, next_line: 28
-        })
+        subject.instance_variable_set(:@proper, this_line: 28, next_line: 28)
         subject.decrease_this_line
 
         proper_indentation = subject.instance_variable_get(:@proper)
-        proper_indentation[:this_line].should == 28
+        expect(proper_indentation[:this_line]).to eq 28
       end
     end
   end
 
   describe '#transition_lines' do
     context '#started? is true' do
-      before { subject.stub(:started?).and_return true }
+      before { allow(subject).to receive(:started?).and_return true }
 
       it 'sets @proper[:this_line] to @proper[:next_line]' do
-        subject.instance_variable_set(:@proper, { next_line: 33 })
+        subject.instance_variable_set(:@proper, next_line: 33)
 
-        expect { subject.transition_lines }.to change{subject.should_be_at}.
+        expect { subject.transition_lines }.to change { subject.should_be_at }.
           from(subject.should_be_at).to(33)
       end
     end
 
     context '#started? is false' do
-      before { subject.stub(:started?).and_return false }
+      before { allow(subject).to receive(:started?).and_return false }
 
       it 'sets @proper[:this_line] to @proper[:next_line]' do
-        subject.instance_variable_set(:@proper, { next_line: 33 })
-        expect { subject.transition_lines }.to_not change{subject.should_be_at}
+        subject.instance_variable_set(:@proper, next_line: 33)
+        expect { subject.transition_lines }.
+          to_not change { subject.should_be_at }
       end
     end
   end
@@ -95,7 +89,7 @@ describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
     it 'sets @do_measurement to true' do
       subject.instance_variable_set(:@do_measurement, false)
       subject.start
-      subject.instance_variable_get(:@do_measurement).should be_true
+      expect(subject.instance_variable_get(:@do_measurement)).to eq true
     end
   end
 
@@ -103,38 +97,39 @@ describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
     it 'sets @do_measurement to false' do
       subject.instance_variable_set(:@do_measurement, true)
       subject.stop
-      subject.instance_variable_get(:@do_measurement).should be_false
+      expect(subject.instance_variable_get(:@do_measurement)).to eq false
     end
   end
 
   describe '#started?' do
     context '@do_measurement is true' do
       before { subject.instance_variable_set(:@do_measurement, true) }
-      specify { subject.started?.should be_true }
+      specify { expect(subject).to be_started }
     end
 
     context '@do_measurement is false' do
       before { subject.instance_variable_set(:@do_measurement, false) }
-      specify { subject.started?.should be_false }
+      specify { expect(subject).to_not be_started }
     end
   end
 
   describe '#update_actual_indentation' do
     context 'lexed_line_output.end_of_multi_line_string? is true' do
       before do
-        lexed_line.stub(:end_of_multi_line_string?).and_return true
+        allow(lexed_line).to receive(:end_of_multi_line_string?).and_return true
       end
 
       it 'returns without updating @actual_indentation' do
-        lexed_line.should_not_receive(:first_non_space_element)
+        expect(lexed_line).to_not receive(:first_non_space_element)
         subject.update_actual_indentation(lexed_line)
       end
     end
 
     context 'lexed_line_output.end_of_multi_line_string? is false' do
       before do
-        lexed_line.stub(:end_of_multi_line_string?).and_return false
-        lexed_line.stub(:first_non_space_element).
+        allow(lexed_line).to receive(:end_of_multi_line_string?).
+          and_return false
+        allow(lexed_line).to receive(:first_non_space_element).
           and_return first_non_space_element
       end
 
@@ -145,7 +140,7 @@ describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
 
         it 'returns the column value of that element' do
           subject.update_actual_indentation(lexed_line)
-          subject.instance_variable_get(:@actual_indentation).should == 5
+          expect(subject.instance_variable_get(:@actual_indentation)).to eq 5
         end
       end
 
@@ -156,7 +151,7 @@ describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
 
         it 'returns the column value of that element' do
           subject.update_actual_indentation(lexed_line)
-          subject.instance_variable_get(:@actual_indentation).should == 0
+          expect(subject.instance_variable_get(:@actual_indentation)).to be_zero
         end
       end
     end
@@ -165,86 +160,91 @@ describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
   describe '#line_ends_with_single_token_indenter?' do
     context 'lexed_line does not end with an op, comma, period, label, or kw' do
       before do
-        lexed_line.stub(ends_with_op?: false)
-        lexed_line.stub(ends_with_comma?: false)
-        lexed_line.stub(ends_with_period?: false)
-        lexed_line.stub(ends_with_label?: false)
-        lexed_line.stub(ends_with_modifier_kw?: false)
+        allow(lexed_line).to receive(:ends_with_op?).and_return false
+        allow(lexed_line).to receive(:ends_with_comma?).and_return false
+        allow(lexed_line).to receive(:ends_with_period?).and_return false
+        allow(lexed_line).to receive(:ends_with_label?).and_return false
+        allow(lexed_line).to receive(:ends_with_modifier_kw?).and_return false
       end
 
       specify do
-        subject.line_ends_with_single_token_indenter?(lexed_line).
-          should be_false
+        expect(subject.line_ends_with_single_token_indenter?(lexed_line)).
+          to eq false
       end
     end
 
     context 'lexed_line ends with an op' do
       before do
-        lexed_line.stub(ends_with_op?: true)
-        lexed_line.stub(ends_with_comma?: false)
-        lexed_line.stub(ends_with_period?: false)
-        lexed_line.stub(ends_with_label?: false)
-        lexed_line.stub(ends_with_modifier_kw?: false)
+        allow(lexed_line).to receive(:ends_with_op?).and_return true
+        allow(lexed_line).to receive(:ends_with_comma?).and_return false
+        allow(lexed_line).to receive(:ends_with_period?).and_return false
+        allow(lexed_line).to receive(:ends_with_label?).and_return false
+        allow(lexed_line).to receive(:ends_with_modifier_kw?).and_return false
       end
 
       specify do
-        subject.line_ends_with_single_token_indenter?(lexed_line).should be_true
+        expect(subject.line_ends_with_single_token_indenter?(lexed_line)).
+          to eq true
       end
     end
 
     context 'lexed_line ends with a comma' do
       before do
-        lexed_line.stub(ends_with_op?: false)
-        lexed_line.stub(ends_with_comma?: true)
-        lexed_line.stub(ends_with_period?: false)
-        lexed_line.stub(ends_with_label?: false)
-        lexed_line.stub(ends_with_modifier_kw?: false)
+        allow(lexed_line).to receive(:ends_with_op?).and_return false
+        allow(lexed_line).to receive(:ends_with_comma?).and_return true
+        allow(lexed_line).to receive(:ends_with_period?).and_return false
+        allow(lexed_line).to receive(:ends_with_label?).and_return false
+        allow(lexed_line).to receive(:ends_with_modifier_kw?).and_return false
       end
 
       specify do
-        subject.line_ends_with_single_token_indenter?(lexed_line).should be_true
+        expect(subject.line_ends_with_single_token_indenter?(lexed_line)).
+          to eq true
       end
     end
 
     context 'lexed_line ends with a period' do
       before do
-        lexed_line.stub(ends_with_op?: false)
-        lexed_line.stub(ends_with_comma?: false)
-        lexed_line.stub(ends_with_period?: true)
-        lexed_line.stub(ends_with_label?: false)
-        lexed_line.stub(ends_with_modifier_kw?: false)
+        allow(lexed_line).to receive(:ends_with_op?).and_return false
+        allow(lexed_line).to receive(:ends_with_comma?).and_return false
+        allow(lexed_line).to receive(:ends_with_period?).and_return true
+        allow(lexed_line).to receive(:ends_with_label?).and_return false
+        allow(lexed_line).to receive(:ends_with_modifier_kw?).and_return false
       end
 
       specify do
-        subject.line_ends_with_single_token_indenter?(lexed_line).should be_true
+        expect(subject.line_ends_with_single_token_indenter?(lexed_line)).
+          to eq true
       end
     end
 
     context 'lexed_line ends with a label' do
       before do
-        lexed_line.stub(ends_with_op?: false)
-        lexed_line.stub(ends_with_comma?: false)
-        lexed_line.stub(ends_with_period?: false)
-        lexed_line.stub(ends_with_label?: true)
-        lexed_line.stub(ends_with_modifier_kw?: false)
+        allow(lexed_line).to receive(:ends_with_op?).and_return false
+        allow(lexed_line).to receive(:ends_with_comma?).and_return false
+        allow(lexed_line).to receive(:ends_with_period?).and_return false
+        allow(lexed_line).to receive(:ends_with_label?).and_return true
+        allow(lexed_line).to receive(:ends_with_modifier_kw?).and_return false
       end
 
       specify do
-        subject.line_ends_with_single_token_indenter?(lexed_line).should be_true
+        expect(subject.line_ends_with_single_token_indenter?(lexed_line)).
+          to eq true
       end
     end
 
     context 'lexed_line ends with a modified kw' do
       before do
-        lexed_line.stub(ends_with_op?: false)
-        lexed_line.stub(ends_with_comma?: false)
-        lexed_line.stub(ends_with_period?: false)
-        lexed_line.stub(ends_with_label?: false)
-        lexed_line.stub(ends_with_modifier_kw?: true)
+        allow(lexed_line).to receive(:ends_with_op?).and_return false
+        allow(lexed_line).to receive(:ends_with_comma?).and_return false
+        allow(lexed_line).to receive(:ends_with_period?).and_return false
+        allow(lexed_line).to receive(:ends_with_label?).and_return false
+        allow(lexed_line).to receive(:ends_with_modifier_kw?).and_return true
       end
 
       specify do
-        subject.line_ends_with_single_token_indenter?(lexed_line).should be_true
+        expect(subject.line_ends_with_single_token_indenter?(lexed_line)).
+          to eq true
       end
     end
   end
@@ -256,7 +256,7 @@ describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
       end
 
       it 'returns false' do
-        subject.line_ends_with_same_as_last([]).should be_false
+        expect(subject.line_ends_with_same_as_last([])).to eq false
       end
     end
 
@@ -269,7 +269,8 @@ describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
       end
 
       it 'returns false' do
-        subject.line_ends_with_same_as_last(last_single_token).should be_false
+        expect(subject.line_ends_with_same_as_last(last_single_token)).
+          to eq false
       end
     end
 
@@ -282,7 +283,8 @@ describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
       end
 
       it 'returns true' do
-        subject.line_ends_with_same_as_last(last_single_token).should be_true
+        expect(subject.line_ends_with_same_as_last(last_single_token)).
+          to eq true
       end
     end
   end
@@ -296,7 +298,7 @@ describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
         end
 
         it 'returns true' do
-          subject.multi_line_parens?(2).should be_true
+          expect(subject.multi_line_parens?(2)).to eq true
         end
       end
 
@@ -306,8 +308,8 @@ describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
           subject.instance_variable_set(:@indent_reasons, d_tokens)
         end
 
-        it 'returns true' do
-          subject.multi_line_parens?(2).should be_false
+        it 'returns false' do
+          expect(subject.multi_line_parens?(2)).to eq false
         end
       end
     end
@@ -318,8 +320,8 @@ describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
         subject.instance_variable_set(:@indent_reasons, d_tokens)
       end
 
-      it 'returns true' do
-        subject.multi_line_parens?(1).should be_false
+      it 'returns false' do
+        expect(subject.multi_line_parens?(1)).to eq false
       end
     end
   end
@@ -327,7 +329,7 @@ describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
   describe '#last_opening_event' do
     context '@indent_reasons is empty' do
       before { subject.instance_variable_set(:@indent_reasons, []) }
-      specify { subject.last_opening_event(nil).should be_nil }
+      specify { expect(subject.last_opening_event(nil)).to be_nil }
     end
 
     context '@indent_reasons contains the corresponding opening event' do
@@ -339,13 +341,15 @@ describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
 
       context 'the corresponding opening event is last' do
         it 'returns the matching opening event' do
-          subject.last_opening_event(:on_rbrace).should == indent_reasons.last
+          expect(subject.last_opening_event(:on_rbrace)).
+            to eq indent_reasons.last
         end
       end
 
       context 'the corresponding opening event is not last' do
         it 'returns the matching opening event' do
-          subject.last_opening_event(:on_rparen).should == indent_reasons.first
+          expect(subject.last_opening_event(:on_rparen)).
+            to eq indent_reasons.first
         end
       end
     end
@@ -365,7 +369,7 @@ describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
         i
       end
 
-      specify { subject.remove_continuation_keywords.should be_nil }
+      specify { expect(subject.remove_continuation_keywords).to be_nil }
     end
 
     context '@indent_reasons does not contain CONTINUATION_KEYWORDS' do
@@ -378,7 +382,7 @@ describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
       end
 
       it 'should not call #pop on @indent_reasons' do
-        indent_reasons.should_not_receive(:pop)
+        expect(indent_reasons).to_not receive(:pop)
         subject.remove_continuation_keywords
       end
     end
@@ -391,8 +395,8 @@ describe Tailor::Rulers::IndentationSpacesRuler::IndentationManager do
       it 'should call #pop on @indent_reasons one time' do
         subject.instance_variable_set(:@indent_reasons, indent_reasons)
         subject.remove_continuation_keywords
-        subject.instance_variable_get(:@indent_reasons).should ==
-          [{ token: 'if' }]
+        expect(subject.instance_variable_get(:@indent_reasons)).
+          to eq [{ token: 'if' }]
       end
     end
   end