ruby-lint 0.0.3 → 0.0.4

data/spec/ruby-lint/definitions_builder/methods.rb

@@ -1,147 +0,0 @@
-require File.expand_path('../../../helper', __FILE__)
-
-describe 'Building method definitions' do
-  describe 'scoping method definitions' do
-    should 'process a global method' do
-      defs    = build_definitions('def example; end')
-      example = defs.lookup(:instance_method, 'example')
-
-      example.is_a?(ruby_method).should == true
-
-      example.type.should        == :method_definition
-      example.method_type.should == :instance_method
-      example.name.should        == 'example'
-    end
-
-    should 'process a nested method' do
-      code = <<-CODE
-def first
-  def second
-  end
-end
-      CODE
-
-      defs  = build_definitions(code)
-      first = defs.lookup(:instance_method, 'first')
-
-      first.is_a?(ruby_method).should == true
-
-      first.lookup(:instance_method, 'second') \
-        .is_a?(ruby_method) \
-        .should == true
-
-      defs.lookup(:instance_method, 'second').nil?.should == true
-    end
-
-    should 'process a global and nested method' do
-      code = <<-CODE
-def first
-  def second
-  end
-end
-
-def third
-end
-      CODE
-
-      defs  = build_definitions(code)
-      first = defs.lookup(:instance_method, 'first')
-
-      first.lookup(:instance_method, 'second') \
-        .is_a?(ruby_method) \
-        .should == true
-
-      first.lookup(:instance_method, 'second') \
-        .lookup(:instance_method, 'third') \
-        .is_a?(ruby_method) \
-        .should == true
-
-      defs.lookup(:instance_method, 'third') \
-        .is_a?(ruby_method) \
-        .should == true
-    end
-  end
-
-  should 'process a definition with a variable assignment in the body' do
-    code = <<-CODE
-def example
-  number = 10
-end
-    CODE
-
-    defs = build_definitions(code)
-
-    defs.lookup(:instance_method, 'example') \
-      .lookup(:local_variable, 'number') \
-      .is_a?(ruby_object) \
-      .should == true
-
-    defs.lookup(:local_variable, 'number').nil?.should == true
-  end
-
-  should 'process a definition with a receiver' do
-    code = <<-CODE
-def String.example
-  number = 10
-end
-    CODE
-
-    defs = build_definitions(code)
-
-    defs.lookup(:constant, 'String') \
-      .lookup(:method, 'example') \
-      .is_a?(ruby_method) \
-      .should == true
-
-    defs.lookup(:method, 'example').nil?.should          == true
-    defs.lookup(:instance_method, 'example').nil?.should == true
-  end
-
-  describe 'creating variables for method parameters' do
-    should 'create local variables' do
-      code = <<-CODE
-def example(number)
-  return number
-end
-      CODE
-
-      defs = build_definitions(code)
-
-      defs.lookup(:instance_method, 'example') \
-        .lookup(:local_variable, 'number') \
-        .is_a?(ruby_object) \
-        .should == true
-    end
-
-    should 'allow the assignment using parameters' do
-      code = <<-CODE
-def example(number)
-  other_number = number
-end
-      CODE
-
-      defs = build_definitions(code)
-
-      defs.lookup(:instance_method, 'example') \
-        .lookup(:local_variable, 'other_number') \
-        .is_a?(ruby_object) \
-        .should == true
-    end
-  end
-
-  describe 'exporting variables' do
-    should 'export variables to the outer scope' do
-      code = <<-CODE
-def example
-  @number = 10
-end
-      CODE
-
-      defs = build_definitions(code)
-
-      defs.lookup(:instance_variable, '@number') \
-        .is_a?(ruby_object) \
-        .should == true
-    end
-  end
-end

data/spec/ruby-lint/definitions_builder/reference_amount.rb

@@ -1,31 +0,0 @@
-require File.expand_path('../../../helper', __FILE__)
-
-describe 'Count the reference amount of variables' do
-  should 'count the amount of references for a variable' do
-    code = <<-CODE
-number = 10
-number
-number
-    CODE
-
-    defs = build_definitions(code)
-
-    defs.lookup(:local_variable, 'number').reference_amount.should == 2
-  end
-
-  should 'count the amount of references for a constant path' do
-    code = <<-CODE
-module A; end
-A::B = 10
-A::B
-A::B
-    CODE
-
-    defs = build_definitions(code)
-
-    defs.lookup(:constant, 'A') \
-      .lookup(:constant, 'B') \
-      .reference_amount \
-      .should == 2
-  end
-end

data/spec/ruby-lint/definitions_builder/unused.rb

@@ -1,15 +0,0 @@
-require File.expand_path('../../../helper', __FILE__)
-
-describe 'Marking variables as used/unused' do
-  should 'mark variables as unused by default' do
-    defs = build_definitions('number = 10')
-
-    defs.lookup(:local_variable, 'number').used?.should == false
-  end
-
-  should 'mark variables as used' do
-    defs = build_definitions('number = 10; number')
-
-    defs.lookup(:local_variable, 'number').used?.should == true
-  end
-end

data/spec/ruby-lint/node.rb

@@ -1,38 +0,0 @@
-require File.expand_path('../../helper', __FILE__)
-
-describe RubyLint::Node do
-  should 'return the value for a scalar' do
-    s(:string, 'hello').value.should == 'hello'
-  end
-
-  should 'return the value for an Array' do
-    numbers = s(:array, s(:integer, '10'), s(:integer, '20')).value
-
-    numbers.is_a?(Array).should == true
-    numbers.length.should       == 2
-
-    numbers[0].value.should == '10'
-    numbers[1].value.should == '20'
-  end
-
-  should 'return the value for a Hash' do
-    pairs = s(:hash, s(:key_value, 'number', s(:integer, '10'))).value
-
-    pairs.is_a?(Array).should == true
-
-    pairs[0].children[1].value.should == '10'
-  end
-
-  should 'return the arguments of a method' do
-    method = s(:method, 'puts', s(:arguments, s(:argument, s(:string, 'foo'))))
-
-    method.gather_arguments(:argument).length.should == 1
-
-    method.gather_arguments(:argument)[0].type.should  == :string
-    method.gather_arguments(:argument)[0].value.should == 'foo'
-  end
-
-  should 'should try to guess the Ruby class of a node' do
-    s(:string, 'foo').ruby_class.should == 'String'
-  end
-end

data/spec/ruby-lint/parser/assignments.rb

@@ -1,225 +0,0 @@
-require File.expand_path('../../../helper', __FILE__)
-
-describe 'Parsing variable assignments' do
-  should 'parse a local variable' do
-    parse('number = 10').should == s(
-      :assign,
-      s(:local_variable, 'number'),
-      s(:integer, '10')
-    )
-  end
-
-  should 'parse an instance variable' do
-    parse('@number = 10').should == s(
-      :assign,
-      s(:instance_variable, '@number'),
-      s(:integer, '10')
-    )
-  end
-
-  should 'parse a class variable' do
-    parse('@@number = 10').should == s(
-      :assign,
-      s(:class_variable, '@@number'),
-      s(:integer, '10')
-    )
-  end
-
-  should 'parse a global variable' do
-    parse('$number = 10').should == s(
-      :assign,
-      s(:global_variable, '$number'),
-      s(:integer, '10')
-    )
-  end
-
-  should 'parse a constant' do
-    parse('NUMBER = 10').should == s(
-      :assign,
-      s(:constant, 'NUMBER'),
-      s(:integer, '10')
-    )
-  end
-
-  should 'parse a variable if it does not already have a value' do
-    parse('number ||= 10').should == s(
-      :op_assign,
-      s(:local_variable, 'number'),
-      s(:integer, '10')
-    )
-  end
-
-  should 'parse assigning two values to two variables' do
-    parse('first, second = 10, 20').should == s(
-      :mass_assign,
-      s(:assign, s(:local_variable, 'first'), s(:integer, '10')),
-      s(:assign, s(:local_variable, 'second'), s(:integer, '20'))
-    )
-  end
-
-  should 'parse assigning one value to two variables' do
-    parse('first, second = 10').should == s(
-      :mass_assign,
-      s(:assign, s(:local_variable, 'first'), s(:integer, '10')),
-      s(:assign, s(:local_variable, 'second'), s(:keyword, 'nil'))
-    )
-  end
-
-  should 'parse assigning an array with two values to two variables' do
-    parse('first, second = [10, 20]').should == s(
-      :mass_assign,
-      s(:assign, s(:local_variable, 'first'), s(:integer, '10')),
-      s(:assign, s(:local_variable, 'second'), s(:integer, '20'))
-    )
-  end
-
-  should 'parse assigning an array with one value to two variables' do
-    parse('first, second = [10]').should == s(
-      :mass_assign,
-      s(:assign, s(:local_variable, 'first'), s(:integer, '10')),
-      s(:assign, s(:local_variable, 'second'), s(:keyword, 'nil'))
-    )
-  end
-
-  should 'parse assigning a value to a variable and expander token' do
-    parse('first, * = 10').should == s(
-      :mass_assign,
-      s(:assign, s(:local_variable, 'first'), s(:integer, '10'))
-    )
-  end
-
-  should 'parse assigning two values to a variable and expander token' do
-    parse('first, * = 10, 20').should == s(
-      :mass_assign,
-      s(:assign, s(:local_variable, 'first'), s(:integer, '10'))
-    )
-  end
-
-  should 'parse assigning three values to two variables and an expander token' do
-    parse('first, *, second = 10, 20, 30').should == s(
-      :mass_assign,
-      s(:assign, s(:local_variable, 'first'), s(:integer, '10')),
-      s(:assign, s(:local_variable, 'second'), s(:integer, '30'))
-    )
-  end
-
-  should 'parse assigning a single value to an expander variable' do
-    parse('*numbers = 10').should == s(
-      :mass_assign,
-      s(:assign, s(:local_variable, 'numbers'), s(:array, s(:integer, '10')))
-    )
-  end
-
-  should 'parse assigning two values to an expander variable' do
-    parse('*numbers = 10, 20').should == s(
-      :mass_assign,
-      s(
-        :assign,
-        s(:local_variable, 'numbers'),
-        s(:array, s(:integer, '10'), s(:integer, '20'))
-      )
-    )
-  end
-
-  should 'parse assigning a value to a local and expander variable' do
-    parse('number, *numbers = 10').should == s(
-      :mass_assign,
-      s(:assign, s(:local_variable, 'number'), s(:integer, '10')),
-      s(:assign, s(:local_variable, 'numbers'), s(:array))
-    )
-  end
-
-  should 'parse assigning one value to an expander and two local variables' do
-    parse('*numbers, number, numberx = 10').should == s(
-      :mass_assign,
-      s(:assign, s(:local_variable, 'numbers'), s(:array)),
-      s(:assign, s(:local_variable, 'number'), s(:integer, '10')),
-      s(:assign, s(:local_variable, 'numberx'), s(:keyword, 'nil'))
-    )
-
-    parse('number, *numbers, numberx = 10').should == s(
-      :mass_assign,
-      s(:assign, s(:local_variable, 'number'), s(:integer, '10')),
-      s(:assign, s(:local_variable, 'numbers'), s(:array)),
-      s(:assign, s(:local_variable, 'numberx'), s(:keyword, 'nil'))
-    )
-
-    parse('number, numberx, *numbers = 10').should == s(
-      :mass_assign,
-      s(:assign, s(:local_variable, 'number'), s(:integer, '10')),
-      s(:assign, s(:local_variable, 'numberx'), s(:keyword, 'nil')),
-      s(:assign, s(:local_variable, 'numbers'), s(:array))
-    )
-  end
-
-  should 'parse assigning two values to an expander and local variable' do
-    parse('*numbers, number = 10, 20').should == s(
-      :mass_assign,
-      s(:assign, s(:local_variable, 'numbers'), s(:array, s(:integer, '10'))),
-      s(:assign, s(:local_variable, 'number'), s(:integer, '20'))
-    )
-  end
-
-  should 'parse an uneven number of (expander) variables and values' do
-    parse('*numbers, number = 10, 20, 30').should == s(
-      :mass_assign,
-      s(
-        :assign,
-        s(:local_variable, 'numbers'),
-        s(:array, s(:integer, '10'), s(:integer, '20'))
-      ),
-      s(:assign, s(:local_variable, 'number'), s(:integer, '30'))
-    )
-
-    parse('*numbers, number = 10, 20, 30, 40').should == s(
-      :mass_assign,
-      s(
-        :assign,
-        s(:local_variable, 'numbers'),
-        s(:array, s(:integer, '10'), s(:integer, '20'), s(:integer, '30'))
-      ),
-      s(:assign, s(:local_variable, 'number'), s(:integer, '40'))
-    )
-
-    parse('number, *numbers = 10, 20, 30, 40').should == s(
-      :mass_assign,
-      s(:assign, s(:local_variable, 'number'), s(:integer, '10')),
-      s(
-        :assign,
-        s(:local_variable, 'numbers'),
-        s(:array, s(:integer, '20'), s(:integer, '30'), s(:integer, '40'))
-      )
-    )
-
-    parse('number, *numbers, numberx = 10, 20, 30, 40').should == s(
-      :mass_assign,
-      s(:assign, s(:local_variable, 'number'), s(:integer, '10')),
-      s(
-        :assign,
-        s(:local_variable, 'numbers'),
-        s(:array, s(:integer, '20'), s(:integer, '30'))
-      ),
-      s(:assign, s(:local_variable, 'numberx'), s(:integer, '40'))
-    )
-  end
-
-  should 'parse the assignment of a attribute in a constant path' do
-    parse('Foo::Bar.example = 10').should == s(
-      :assign,
-      s(
-        :field,
-        s(:constant_path, s(:constant, 'Foo'), s(:constant, 'Bar')),
-        s(:identifier, 'example')
-      ),
-      s(:integer, '10')
-    )
-  end
-
-  should 'parse the assignment of a constant path' do
-    parse('Foo::Bar = 10').should == s(
-      :assign,
-      s(:constant_path, s(:constant, 'Foo'), s(:constant, 'Bar')),
-      s(:integer, '10')
-    )
-  end
-end