tailor 0.1.5 → 1.0.0.alpha

data/spec/tailor/rulers/indentation_spaces_ruler_spec.rb

@@ -0,0 +1,128 @@
+require_relative '../../spec_helper'
+require 'tailor/rulers/indentation_spaces_ruler'
+require 'ripper'
+
+describe Tailor::Rulers::IndentationSpacesRuler do
+  let!(:spaces) { 5 }
+
+  subject do
+    Tailor::Rulers::IndentationSpacesRuler.new(spaces)
+  end
+
+  describe "#initialize" do
+    it "sets @proper to an Hash with :this_line and :next_line keys = 0" do
+      proper_indentation = subject.instance_variable_get(:@proper)
+      proper_indentation.should be_a Hash
+      proper_indentation[:this_line].should be_zero
+      proper_indentation[:next_line].should be_zero
+    end
+  end
+
+
+  describe "#comment_update" do
+    pending
+    context "token does not contain a trailing newline" do
+      
+    end
+    
+    context "token contains a trailing newline" do
+      context "lexed_line is spaces then a comment" do
+        
+      end
+      
+      context "lexed_line is no spaces and a comment" do
+        
+      end
+      
+      context "lexed_line ends with an operator" do
+        
+      end
+      
+      context "lexed_line ends with a comma" do
+        
+      end
+    end
+  end
+
+  describe "#embexpr_beg_update" do
+    it "sets @embexpr_beg to true" do
+      subject.instance_variable_set(:@embexpr_beg, false)
+      subject.embexpr_beg_update
+      subject.instance_variable_get(:@embexpr_beg).should be_true
+    end
+  end
+
+
+  describe "#embexpr_end_update" do
+    it "sets @embexpr_beg to false" do
+      subject.instance_variable_set(:@embexpr_beg, true)
+      subject.embexpr_end_update
+      subject.instance_variable_get(:@embexpr_beg).should be_false
+    end
+  end
+
+  describe "#ignored_nl_update" do
+    pending
+  end
+
+  describe "#kw_update" do
+    pending
+  end
+
+  describe "#lbrace_update" do
+    pending
+  end
+
+  describe "#lbracket_update" do
+    pending
+  end
+
+  describe "#lparen_update" do
+    pending
+  end
+
+  describe "#nl_update" do
+    pending
+  end
+
+  describe "#period_update" do
+    pending
+  end
+
+  describe "#rbrace_update" do
+    pending
+  end
+
+  describe "#rbracket_update" do
+    pending
+  end
+
+  describe "#rparen_update" do
+    pending
+  end
+
+  describe "#tstring_beg_update" do
+    it "calls #stop" do
+      subject.should_receive(:stop)
+      subject.tstring_beg_update 1
+    end
+    
+    it "adds the lineno to @tstring_nesting" do
+      subject.tstring_beg_update 1
+      subject.instance_variable_get(:@tstring_nesting).should == [1]
+    end
+  end
+
+  describe "#tstring_end_update" do
+    it "calls #start" do
+      subject.should_receive(:start)
+      subject.tstring_end_update
+    end
+
+    it "removes the lineno to @tstring_nesting" do
+      subject.instance_variable_set(:@tstring_nesting, [1])
+      subject.tstring_end_update
+      subject.instance_variable_get(:@tstring_nesting).should be_empty
+    end
+  end
+end

data/spec/tailor/rulers/spaces_after_comma_spec.rb

@@ -0,0 +1,31 @@
+require_relative '../../spec_helper'
+require 'tailor/rulers/spaces_after_comma_ruler'
+
+describe Tailor::Rulers::SpacesAfterCommaRuler do
+  subject { Tailor::Rulers::SpacesAfterCommaRuler.new({}) }
+  
+  describe "#comma_update" do
+    it "adds the column number to @comma_columns" do
+      subject.comma_update(",", 2, 1)
+      subject.instance_variable_get(:@comma_columns).should == [1]
+    end
+  end
+  
+  describe "#check_spaces_after_comma" do
+    context "no event after comma" do
+      let(:lexed_line) do
+        l = double "LexedLine"
+        l.stub(:event_at)
+        l.stub(:index)
+        
+        l
+      end
+      
+      it "doesn't detect any problems" do
+        Tailor::Problem.should_not_receive(:new)
+        expect { subject.check_spaces_after_comma(lexed_line, 1) }.
+          to_not raise_error
+      end
+    end
+  end
+end

data/spec/tailor/rulers/spaces_after_lbrace_ruler_spec.rb

@@ -0,0 +1,145 @@
+require_relative '../../spec_helper'
+require 'tailor/rulers/spaces_after_lbrace_ruler'
+
+describe Tailor::Rulers::SpacesAfterLbraceRuler do
+  subject { Tailor::Rulers::SpacesAfterLbraceRuler.new('')}
+  
+  describe "#comment_update" do
+    context "token has a trailing newline" do
+      it "calls #ignored_nl_update" do
+        subject.should_receive(:ignored_nl_update)
+        subject.comment_update("\n", '', '', 1, 1)
+      end
+    end
+    
+    context "token does not have a trailing newline" do
+      it "does not call #ignored_nl_update" do
+        subject.should_not_receive(:ignored_nl_update)
+        subject.comment_update("# comment", '', '', 1, 1)
+      end
+    end
+  end
+  
+  describe "#ignored_nl_update" do
+    it "calls #check_spaces_after_lbrace" do
+      subject.should_receive(:check_spaces_after_lbrace)
+      subject.ignored_nl_update('', 1, 1)
+    end
+  end
+  
+  describe "#lbrace_update" do
+    it "adds column to @lbrace_columns" do
+      subject.lbrace_update('', 1, 1)
+      subject.instance_variable_get(:@lbrace_columns).should == [1]
+    end
+  end
+  
+  describe "#nl_update" do
+    it "calls #ignored_nl_update" do
+      subject.should_receive(:ignored_nl_update)
+      subject.nl_update('', 1, 1)
+    end
+  end
+  
+  describe "#count_spaces" do
+    context "lexed_line.event_index returns nil" do
+      let(:lexed_line) do
+        l = double "LexedLine"
+        l.stub(:event_index).and_return nil
+        
+        l
+      end
+      
+      it "breaks from the loop and returns nil" do
+        lexed_line.should_not_receive(:at)
+        subject.count_spaces(lexed_line, 1).should be_nil
+      end
+    end
+    
+    context "lexed_line.at returns nil" do
+      let(:lexed_line) do
+        l = double "LexedLine"
+        l.stub(:event_index).and_return 1
+        l.stub(:at).and_return nil
+
+        l
+      end
+
+      it "returns 0" do
+        subject.count_spaces(lexed_line, 1).should be_zero
+      end
+    end
+
+    context "next_event is a :on_nl" do
+      let!(:next_event) do
+        [[1,1], :on_nl, "\n"]
+      end
+
+      let(:lexed_line) do
+        l = double "LexedLine"
+        l.stub(:event_index).and_return 1
+        l.should_receive(:at).with(2).and_return next_event
+
+        l
+      end
+
+      it "returns 0" do
+        subject.count_spaces(lexed_line, 1).should be_zero
+      end
+    end
+
+    context "next_event is a :on_ignored_nl" do
+      let!(:next_event) do
+        [[1,1], :on_ignored_nl, "\n"]
+      end
+
+      let(:lexed_line) do
+        l = double "LexedLine"
+        l.stub(:event_index).and_return 1
+        l.should_receive(:at).with(2).and_return next_event
+
+        l
+      end
+
+      it "breaks from the loop and returns nil" do
+        subject.count_spaces(lexed_line, 1)
+      end
+    end
+
+    context "next_event is a non-space event" do
+      let!(:next_event) do
+        [[1,1], :on_kw, "def"]
+      end
+
+      let(:lexed_line) do
+        l = double "LexedLine"
+        l.stub(:event_index).and_return 1
+        l.stub(:at).and_return next_event
+
+        l
+      end
+
+      it "returns 0" do
+        subject.count_spaces(lexed_line, 1).should be_zero
+      end
+    end
+
+    context "next_event is :on_sp" do
+      let!(:next_event) do
+        [[1,1], :on_sp, "  "]
+      end
+
+      let(:lexed_line) do
+        l = double "LexedLine"
+        l.stub(:event_index).and_return 1
+        l.stub(:at).and_return next_event
+
+        l
+      end
+
+      it "returns 2" do
+        subject.count_spaces(lexed_line, 1).should == 2
+      end
+    end
+  end
+end

data/spec/tailor/rulers/spaces_before_lbrace_ruler_spec.rb

@@ -0,0 +1,63 @@
+require_relative '../../spec_helper'
+require 'tailor/rulers/spaces_before_lbrace_ruler'
+
+describe Tailor::Rulers::SpacesBeforeLbraceRuler do
+  subject { Tailor::Rulers::SpacesBeforeLbraceRuler.new 1 }
+  before { Tailor::Logger.stub(:log) }
+
+  describe "#count_spaces" do
+    context "lexed_line.event_index is 0" do
+      let(:lexed_line) do
+        l = double "LexedLine"
+        l.stub(:event_index).and_return 0
+        l.stub(:at).and_return nil
+
+        l
+      end
+
+      specify { subject.count_spaces(lexed_line, 1).should be_zero }
+
+      it "sets @do_measurement to false" do
+        expect { subject.count_spaces(lexed_line, 1) }.
+          to change{subject.instance_variable_get(:@do_measurement)}.from(true).
+          to(false)
+      end
+    end
+
+    context "no space before lbrace" do
+      let(:lexed_line) do
+        l = double "LexedLine"
+        l.stub(:event_index).and_return 1
+        l.stub(:at).and_return [[10, 0], :on_const, "HI"]
+
+        l
+      end
+
+      specify { subject.count_spaces(lexed_line, 1).should be_zero }
+    end
+
+    context "1 space before lbrace" do
+      let(:lexed_line) do
+        l = double "LexedLine"
+        l.stub(:event_index).and_return 1
+        l.stub(:at).and_return [[10, 0], :on_sp, " "]
+
+        l
+      end
+
+      specify { subject.count_spaces(lexed_line, 1).should == 1 }
+    end
+
+    context "> 1 space before lbrace" do
+      let(:lexed_line) do
+        l = double "LexedLine"
+        l.stub(:event_index).and_return 1
+        l.stub(:at).and_return [[10, 1], :on_sp, "  "]
+
+        l
+      end
+
+      specify { subject.count_spaces(lexed_line, 3).should == 2 }
+    end
+  end
+end

data/spec/tailor/rulers/spaces_before_rbrace_ruler_spec.rb

@@ -0,0 +1,63 @@
+require_relative '../../spec_helper'
+require 'tailor/rulers/spaces_before_rbrace_ruler'
+
+describe Tailor::Rulers::SpacesBeforeRbraceRuler do
+  subject { Tailor::Rulers::SpacesBeforeRbraceRuler.new 1 }
+  before { Tailor::Logger.stub(:log) }
+
+  describe "#count_spaces" do
+    context "lexed_line.event_index is 0" do
+      let(:lexed_line) do
+        l = double "LexedLine"
+        l.stub(:event_index).and_return 0
+        l.stub(:at).and_return nil
+
+        l
+      end
+
+      specify { subject.count_spaces(lexed_line, 1).should be_zero }
+
+      it "sets @do_measurement to false" do
+        expect { subject.count_spaces(lexed_line, 1) }.
+          to change{subject.instance_variable_get(:@do_measurement)}.from(true).
+          to(false)
+      end
+    end
+
+    context "no space before rbrace" do
+      let(:lexed_line) do
+        l = double "LexedLine"
+        l.stub(:event_index).and_return 1
+        l.stub(:at).and_return [[10, 0], :on_const, "HI"]
+
+        l
+      end
+
+      specify { subject.count_spaces(lexed_line, 1).should be_zero }
+    end
+
+    context "1 space before rbrace" do
+      let(:lexed_line) do
+        l = double "LexedLine"
+        l.stub(:event_index).and_return 1
+        l.stub(:at).and_return [[10, 0], :on_sp, " "]
+
+        l
+      end
+
+      specify { subject.count_spaces(lexed_line, 1).should == 1 }
+    end
+
+    context "> 1 space before rbrace" do
+      let(:lexed_line) do
+        l = double "LexedLine"
+        l.stub(:event_index).and_return 1
+        l.stub(:at).and_return [[10, 0], :on_sp, "  "]
+
+        l
+      end
+
+      specify { subject.count_spaces(lexed_line, 1).should == 2 }
+    end
+  end
+end

data/spec/tailor/rulers_spec.rb

@@ -0,0 +1,9 @@
+require_relative '../spec_helper'
+require 'tailor/rulers'
+
+describe Tailor::Rulers do
+  it "requires all of its children" do
+    # if it does one, it'll have done them all.
+    subject.const_get('AllowCamelCaseMethodsRuler').should be_true
+  end
+end

data/spec/tailor/version_spec.rb

@@ -0,0 +1,6 @@
+require_relative '../spec_helper'
+require 'tailor/version'
+
+describe Tailor::VERSION do
+  it { should == "1.0.0.alpha" }
+end