tailor 0.1.5 → 1.0.0.alpha

data/spec/tailor/lexer/token_spec.rb

@@ -0,0 +1,46 @@
+require_relative '../../spec_helper'
+require 'tailor/lexer/token'
+
+describe Tailor::Lexer::Token do
+  before do
+    Tailor::Logger.stub(:log)
+  end
+
+  describe "#modifier_keyword?" do
+    subject do
+      options = { full_line_of_text: full_line_of_text }
+      Tailor::Lexer::Token.new("if", options)
+    end
+
+    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} }
+
+        it "returns true" do
+          subject.modifier_keyword?.should be_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} }
+
+        it "returns false" do
+          subject.modifier_keyword?.should be_false
+        end
+      end
+    end
+
+    context "the current line doesn't have a keyword" do
+      let!(:full_line_of_text) { %q{puts true} }
+
+      subject do
+        options = { full_line_of_text: full_line_of_text }
+        Tailor::Lexer::Token.new("puts", options)
+      end
+
+      it "returns false" do
+        subject.modifier_keyword?.should be_false
+      end
+    end
+  end
+end

data/spec/tailor/lexer_spec.rb

@@ -0,0 +1,181 @@
+require 'fakefs/spec_helpers'
+require_relative '../spec_helper'
+require 'tailor/lexer'
+
+describe Tailor::Lexer do
+  let!(:file_text) { "" }
+  let(:style) { {} }
+  let(:indentation_ruler) { double "IndentationSpacesRuler" }
+
+  subject do
+    r = Tailor::Lexer.new(file_text)
+    r.instance_variable_set(:@buf, [])
+    r.stub(:log)
+
+    r
+  end
+
+  before do
+    Tailor::Lexer.any_instance.stub(:ensure_trailing_newline).and_return(file_text)
+  end
+
+  describe "#initialize" do
+    context "name of file is passed in" do
+      let(:file_name) { "test" }
+
+      before do
+        File.open(file_name, 'w') { |f| f.write "some text" }
+      end
+
+      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
+        Tailor::Lexer.new(file_name)
+      end
+    end
+
+    context "text to lex is passed in" do
+      let(:text) { "some text" }
+
+      it "doesn't try to open a file" do
+        File.should_not_receive(:open)
+        Tailor::Lexer.new(text)
+      end
+    end
+  end
+
+  describe "#on_ignored_nl" do
+    it "calls #current_line_lex" do
+      pending
+      subject.stub(:only_spaces?).and_return true
+      subject.should_receive(:current_line_lex)
+      subject.on_ignored_nl("\n")
+    end
+
+    context "#only_spaces? is true" do
+      pending
+      before { subject.stub(:only_spaces?).and_return true }
+
+      it "does not call #update_actual_indentation" do
+        pending
+      end
+    end
+  end
+
+  describe "#on_sp" do
+    context "@config says to disallow hard tabs" do
+      before do
+        config = { horizontal_spacing: { allow_hard_tabs: false } }
+        subject.instance_variable_set(:@config, config)
+      end
+
+      context "token contains a hard tab" do
+        it "adds a new problem to @problems" do
+          pending "This behavior moved to indent_sp_ruler--move there."
+
+          subject.instance_variable_set(:@problems, [])
+
+          expect { subject.on_sp("\t") }.
+            to change{subject.instance_variable_get(:@problems).size}.
+            from(0).to 1
+        end
+      end
+
+      context "token does not contain a hard tab" do
+        it "does not add a new problem to @problems" do
+          pending "This behavior moved to indent_sp_ruler--move there."
+
+          subject.instance_variable_set(:@problems, [])
+
+          expect { subject.on_sp("\x20") }.
+            to_not change{subject.instance_variable_get(:@problems).size}.
+            from(0).to 1
+        end
+      end
+    end
+
+    context "@config says to allow hard tabs" do
+      before do
+        config = { horizontal_spacing: { allow_hard_tabs: true } }
+        subject.instance_variable_set(:@config, config)
+      end
+
+      it "does not check the token" do
+        token = double "token"
+        token.stub(:size)
+        token.should_not_receive(:=~)
+        subject.on_sp(token)
+      end
+    end
+  end
+
+  describe "#current_line_of_text" do
+    before do
+      subject.instance_variable_set(:@file_text, file_text)
+      subject.stub(:lineno).and_return 1
+    end
+
+    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
+      end
+    end
+
+    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
+      end
+    end
+
+    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 == ""
+      end
+    end
+  end
+
+  describe "#count_trailing_newlines" do
+    context "text contains 0 trailing \\n" do
+      let(:text) { "text" }
+      specify { subject.count_trailing_newlines(text).should be_zero }
+    end
+
+    context "text contains 1 trailing \\n" do
+      let(:text) { "text\n" }
+      specify { subject.count_trailing_newlines(text).should == 1 }
+    end
+  end
+
+  describe "#ensure_trailing_newline" do
+    before do
+      Tailor::Lexer.any_instance.unstub(:ensure_trailing_newline)
+    end
+    
+    context "text contains a trailing newline already" do
+      let!(:text) { "text\n" }
+      
+      before do
+        subject.stub(:count_trailing_newlines).and_return 1
+      end
+
+      it "doesn't alter the text" do
+        subject.ensure_trailing_newline(text).should == text
+      end
+    end
+
+    context "text does not contain a trailing newline" do
+      let!(:text) { "text" }
+
+      it "adds a newline at the end" do
+        subject.ensure_trailing_newline(text).should == text + "\n"
+      end
+    end
+  end
+end

data/spec/tailor/options_spec.rb

@@ -0,0 +1,6 @@
+require_relative '../spec_helper'
+require 'tailor/cli/options'
+
+describe Tailor::CLI::Options do
+
+end

data/spec/tailor/problem_spec.rb

@@ -0,0 +1,74 @@
+require_relative '../spec_helper'
+require 'tailor/problem'
+
+describe Tailor::Problem do
+  before do
+    Tailor::Problem.any_instance.stub(:log)
+  end
+  
+  let(:lineno) { 10 }
+  let(:column) { 11 }
+
+  describe "#set_values" do
+    before do
+      Tailor::Problem.any_instance.stub(:message)
+    end
+
+    it "sets self[:type] to the type param" do
+      Tailor::Problem.new(:test, lineno, column).should include(type: :test)
+    end
+
+    it "sets self[:line] to the lineno param" do
+      Tailor::Problem.new(:test, lineno, column).should include(line: lineno)
+    end
+
+    it "sets self[:column] to 'column' from the binding" do
+      Tailor::Problem.new(:test, lineno, column).should include(column: column)
+    end
+
+    it "sets self[:message] to what's returned from #message for @type" do
+      Tailor::Problem.any_instance.should_receive(:message).with(:test).
+        and_return("test message")
+
+      problem = Tailor::Problem.new(:test, lineno, column)
+      problem.should include(message: "test message")
+    end
+  end
+
+  describe "#message" do
+    before do
+      Tailor::Problem.any_instance.stub(:set_values)
+    end
+
+    context "type is :indentation" do
+      it "builds a successful message" do
+        options = { actual_indentation: 10, should_be_at: 97 }
+        problem = Tailor::Problem.new(:test, lineno, column, options)
+        problem.message(:indentation).should match /10.*97/
+      end
+    end
+
+    context "type is :trailing_newlines" do
+      it "builds a successful message" do
+        options = { actual_trailing_newlines: 123, should_have: 777 }
+        problem = Tailor::Problem.new(:test, lineno, column, options)
+        problem.message(:trailing_newlines).should match /123.*777/
+      end
+    end
+
+    context "type is :hard_tab" do
+      it "builds a successful message" do
+        problem = Tailor::Problem.new(:test, lineno, column)
+        problem.message(:hard_tab).should match /Hard tab found./
+      end
+    end
+
+    context "type is :line_length" do
+      it "builds a successful message" do
+        options = { actual_length: 88, should_be_at: 77 }
+        problem = Tailor::Problem.new(:test, lineno, column, options)
+        problem.message(:line_length).should match /88.*77/
+      end
+    end
+  end
+end

data/spec/tailor/reporter_spec.rb

@@ -0,0 +1,53 @@
+require_relative '../spec_helper'
+require 'tailor/reporter'
+
+describe Tailor::Reporter do
+  describe "#initialize" do
+    context "text formatter" do
+      let(:formats) { ['text'] }
+
+      it "creates a new Formatter object of the type passed in" do
+        reporter = Tailor::Reporter.new(formats)
+        reporter.formatters.first.should be_a Tailor::Formatter::Text
+      end
+    end
+  end
+  
+  describe "#file_report" do
+    let(:file_problems) { double "file problems" }
+    let(:formatter) { double "Tailor::Formatter::SomeFormatter" }
+    
+    subject do
+      t = Tailor::Reporter.new
+      t.instance_variable_set(:@formatters, [formatter])
+      
+      t
+    end 
+    
+    it "calls #file_report on each @formatters" do
+      label = :some_label
+      formatter.should_receive(:file_report).with(file_problems, label)
+
+      subject.file_report(file_problems, label)
+    end
+  end
+
+  describe "#summary_report" do
+    let(:all_problems) { double "all problems" }
+    let(:formatter) { double "Tailor::Formatter::SomeFormatter" }
+
+    subject do
+      t = Tailor::Reporter.new
+      t.instance_variable_set(:@formatters, [formatter])
+
+      t
+    end
+
+    it "calls #file_report on each @formatters" do
+      label = :some_label
+      formatter.should_receive(:summary_report).with(all_problems)
+
+      subject.summary_report(all_problems)
+    end
+  end
+end

data/spec/tailor/ruler_spec.rb

@@ -0,0 +1,56 @@
+require_relative '../spec_helper'
+require 'tailor/ruler'
+
+describe Tailor::Ruler do
+  before { subject.stub(: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
+    end
+  end
+
+  describe "#problems" do
+    context "no child_rulers" do
+      context "@problems is empty" do
+        specify { subject.problems.should be_empty }
+      end
+
+      context "@problems.size is 1" do
+        before do
+          problem = double "Problem"
+          problem.should_receive(:[]).with :line
+          subject.instance_variable_set(:@problems, [problem])
+        end
+
+        specify { subject.problems.size.should == 1 }
+      end
+    end
+
+    context "child_rulers have problems" do
+      before do
+        problem = double "Problem"
+        problem.should_receive(:[]).with :line
+        child_ruler = double "Ruler"
+        child_ruler.stub(:problems).and_return([problem])
+        subject.instance_variable_set(:@child_rulers, [child_ruler])
+      end
+
+      context "@problems is empty" do
+        specify { subject.problems.size.should == 1 }
+      end
+
+      context "@problems.size is 1" do
+        before do
+          problem = double "Problem"
+          problem.should_receive(:[]).with :line
+          subject.instance_variable_set(:@problems, [problem])
+        end
+
+        specify { subject.problems.size.should == 2 }
+      end
+    end
+  end
+end

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

@@ -0,0 +1,454 @@
+require 'ripper'
+require_relative '../../../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)
+    subject.instance_variable_set(:@spaces, spaces)
+  end
+
+  subject do
+    Tailor::Rulers::IndentationSpacesRuler::IndentationManager.new spaces
+  end
+
+  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
+    end
+  end
+
+  describe "#next_should_be_at" do
+    it "returns @proper[:next_line]" do
+      subject.instance_variable_set(:@proper, { next_line: 123 })
+      subject.next_should_be_at.should == 123
+    end
+  end
+
+  describe "#decrease_this_line" do
+    let!(:spaces) { 27 }
+
+    context "#started? is true" do
+      before { subject.stub(: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.decrease_this_line
+          proper_indentation = subject.instance_variable_get(:@proper)
+          proper_indentation[:this_line].should == 0
+        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.decrease_this_line
+
+          proper_indentation = subject.instance_variable_get(:@proper)
+          proper_indentation[:this_line].should == 1
+        end
+      end
+    end
+
+    context "#started? is false" do
+      before { subject.stub(:started?).and_return false }
+
+      it "does not decrement @proper[:this_line]" do
+        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
+      end
+    end
+  end
+
+  describe "#increase_next_line" do
+    context "#started? is true" do
+      before { subject.stub(:started?).and_return true }
+
+      it "increases @proper[:next_line] by @spaces" do
+        expect { subject.increase_next_line }.to change{subject.next_should_be_at}.
+          by(spaces)
+      end
+    end
+
+    context "#started? is false" do
+      before { subject.stub(:started?).and_return false }
+
+      it "does not increases @proper[:next_line]" do
+        expect { subject.increase_next_line }.to_not change{subject.next_should_be_at}.
+          by(spaces)
+      end
+    end
+  end
+
+  describe "#decrease_next_line" do
+    let!(:spaces) { 27 }
+
+    context "#started? is true" do
+      before { subject.stub(:started?).and_return true }
+
+      it "decrements @proper[:next_line] by @spaces" do
+        expect { subject.decrease_next_line }.to change{subject.next_should_be_at}.
+          by(-spaces)
+      end
+    end
+
+    context "#started? is false" do
+      before { subject.stub(:started?).and_return false }
+
+      it "decrements @proper[:next_line] by @spaces" do
+        expect { subject.decrease_next_line }.to_not change{subject.next_should_be_at}.
+          by(-spaces)
+      end
+    end
+  end
+
+  describe "#set_up_line_transition" do
+    context "@amount_to_change_next > 0" do
+      before { subject.instance_variable_set(:@amount_to_change_next, 1) }
+
+      it "should call #increase_next_line" do
+        subject.should_receive(:increase_next_line)
+        subject.set_up_line_transition
+      end
+    end
+
+    context "@amount_to_change_next < 0" do
+      before { subject.instance_variable_set(:@amount_to_change_next, -1) }
+
+      it "should call #increase_next_line" do
+        subject.should_receive(:decrease_next_line)
+        subject.set_up_line_transition
+      end
+    end
+
+    context "@amount_to_change_this < 0" do
+      before { subject.instance_variable_set(:@amount_to_change_this, -1) }
+
+      it "should call #decrease_this_line" do
+        subject.should_receive(:decrease_this_line)
+        subject.set_up_line_transition
+      end
+    end
+  end
+
+  describe "#transition_lines" do
+    context "#started? is true" do
+      before { subject.stub(:started?).and_return true }
+
+      it "sets @proper[:this_line] to @proper[:next_line]" do
+        subject.instance_variable_set(:@proper, { next_line: 33 })
+
+        expect { subject.transition_lines }.to change{subject.should_be_at}.
+          from(subject.should_be_at).to(subject.next_should_be_at)
+      end
+    end
+
+    context "#started? is true" do
+      before { subject.stub(: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}.
+          from(subject.should_be_at).to(subject.next_should_be_at)
+      end
+    end
+  end
+
+  describe "#start" 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
+    end
+  end
+
+  describe "#stop" 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
+    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 }
+    end
+
+    context "@do_measurement is false" do
+      before { subject.instance_variable_set(:@do_measurement, false) }
+      specify { subject.started?.should be_false }
+    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
+      end
+
+      it "returns without updating @actual_indentation" do
+        lexed_line.should_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).
+          and_return first_non_space_element
+      end
+
+      context "when indented" do
+        let(:first_non_space_element) do
+          [[1, 5], :on_comma, ',']
+        end
+
+        it "returns the column value of that element" do
+          subject.update_actual_indentation(lexed_line)
+          subject.instance_variable_get(:@actual_indentation).should == 5
+        end
+      end
+
+      context "when not indented" do
+        let(:first_non_space_element) do
+          [[1, 0], :on_kw, 'class']
+        end
+
+        it "returns the column value of that element" do
+          subject.update_actual_indentation(lexed_line)
+          subject.instance_variable_get(:@actual_indentation).should == 0
+        end
+      end
+    end
+  end
+
+  describe "#line_ends_with_single_token_indenter?" do
+    context "lexed_line doesn't end with an op, comma, or period" do
+      before do
+        lexed_line.stub(ends_with_op?: false)
+        lexed_line.stub(ends_with_comma?: false)
+        lexed_line.stub(ends_with_period?: false)
+      end
+
+      specify { subject.line_ends_with_single_token_indenter?(lexed_line).should be_false }
+    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)
+      end
+
+      specify { subject.line_ends_with_single_token_indenter?(lexed_line).should be_true }
+    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)
+      end
+
+      specify { subject.line_ends_with_single_token_indenter?(lexed_line).should be_true }
+    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)
+      end
+
+      specify { subject.line_ends_with_single_token_indenter?(lexed_line).should be_true }
+    end
+  end
+
+  describe "#line_ends_with_same_as_last" do
+    context "@single_tokens is empty" do
+      before do
+        subject.instance_variable_set(:@single_tokens, [])
+      end
+
+      it "returns false" do
+        subject.line_ends_with_same_as_last([]).should be_false
+      end
+    end
+
+    context "@single_tokens.last[:token] != token_event.last" do
+      let(:last_single_token) { [[1, 2], :on_comma, ','] }
+
+      before do
+        subject.instance_variable_set(:@single_tokens, [{ event: :on_op }])
+      end
+
+      it "returns false" do
+        subject.line_ends_with_same_as_last(last_single_token).should be_false
+      end
+    end
+
+    context "@single_tokens.last[:token] == token_event.last" do
+      let(:last_single_token) { [[1, 2], :on_comma, ','] }
+
+      before do
+        subject.instance_variable_set(:@single_tokens, [{ event: :on_comma }])
+      end
+
+      it "returns false" do
+        subject.line_ends_with_same_as_last(last_single_token).should be_true
+      end
+    end
+  end
+
+  describe "#comma_is_part_of_enclosed_statement?" do
+    context "lexed_line does not end with a comma" do
+      before { lexed_line.stub(ends_with_comma?: false) }
+
+      it "returns false" do
+        subject.comma_is_part_of_enclosed_statement?(lexed_line, 1).
+          should be_false
+      end
+    end
+
+    context "lexed_line ends with a comma" do
+      before { lexed_line.stub(ends_with_comma?: true) }
+
+      context "continuing_enclosed_statement? is true" do
+        before { subject.stub(continuing_enclosed_statement?: true) }
+
+        it "returns true" do
+          subject.comma_is_part_of_enclosed_statement?(lexed_line, 1).
+            should be_true
+        end
+      end
+
+      context "continuing_enclosed_statement? is true" do
+        before { subject.stub(continuing_enclosed_statement?: false) }
+
+        it "returns false" do
+          subject.comma_is_part_of_enclosed_statement?(lexed_line, 1).
+            should be_false
+        end
+      end
+    end
+  end
+
+  describe "#keyword_and_single_token_line?" do
+    context "no @double_tokens on this line" do
+      before do
+        d_tokens = [{ lineno: 12345 }]
+        subject.instance_variable_set(:@double_tokens, d_tokens)
+      end
+
+      it "returns false" do
+        subject.keyword_and_single_token_line?(1).should be_false
+      end
+    end
+
+    context "@double_tokens exist on this line" do
+      context "no kw tokens on this line" do
+        before do
+          d_tokens = [{ token: '{', lineno: 1 }]
+          subject.instance_variable_set(:@double_tokens, d_tokens)
+        end
+
+        it "returns false" do
+          subject.keyword_and_single_token_line?(1).should be_false
+        end
+      end
+
+      context "kw tokens on this line" do
+        before do
+          d_tokens = [{ token: 'class', lineno: 1 }]
+          subject.instance_variable_set(:@double_tokens, d_tokens)
+        end
+
+        context "no @single_tokens on this line" do
+          it "returns false" do
+            subject.keyword_and_single_token_line?(1).should be_false
+          end
+        end
+
+        context "@single_tokens exist on this line" do
+          before do
+            s_tokens = [{ token: '[', lineno: 1 }]
+            subject.instance_variable_set(:@single_tokens, s_tokens)
+          end
+
+          it "returns true" do
+            subject.keyword_and_single_token_line?(1).should be_true
+          end
+        end
+      end
+    end
+  end
+
+  describe "#multi_line_braces?" do
+    pending
+  end
+
+  describe "#multi_line_brackets?" do
+    pending
+  end
+
+  describe "#multi_line_parens?" do
+    context "an unclosed ( exists on the previous line" do
+      context "an unclosed ( does not exist on the current line" do
+        before do
+          d_tokens = [{ token: '(', lineno: 1 }]
+          subject.instance_variable_set(:@double_tokens, d_tokens)
+        end
+
+        it "returns true" do
+          subject.multi_line_parens?(2).should be_true
+        end
+      end
+
+      context "an unclosed ( exists on the current line" do
+        before do
+          d_tokens = [{ token: '(', lineno: 1 }, { token: '(', lineno: 2 }]
+          subject.instance_variable_set(:@double_tokens, d_tokens)
+        end
+
+        it "returns true" do
+          subject.multi_line_parens?(2).should be_false
+        end
+      end
+    end
+
+    context "an unclosed ( does not exist on the previous line" do
+      before do
+        d_tokens = [{ token: '(', lineno: 1 }]
+        subject.instance_variable_set(:@double_tokens, d_tokens)
+      end
+
+      it "returns true" do
+        subject.multi_line_parens?(1).should be_false
+      end
+    end
+  end
+
+  describe "#in_tstring?" do
+    pending
+  end
+end