ruby-lint 0.0.1a

data/spec/rlint/parser/methods.rb

@@ -0,0 +1,249 @@
+require File.expand_path('../../../helper', __FILE__)
+
+describe 'Rlint::Parser' do
+  it 'Parse a method definition' do
+    token = Rlint::Parser.new('def number; return 10; end').parse[0]
+
+    token.class.should      == Rlint::Token::MethodDefinitionToken
+    token.line.should       == 1
+    token.column.should     == 4
+    token.name.should       == 'number'
+    token.visibility.should == :public
+    token.type.should       == :method_definition
+
+    token.parameters.class.should == Rlint::Token::ParametersToken
+
+    token.value.class.should  == Array
+    token.value.length.should == 1
+
+    token.value[0].class.should == Rlint::Token::StatementToken
+    token.value[0].type.should  == :return
+
+    token.value[0].value.class.should  == Array
+    token.value[0].value.length.should == 1
+
+    value = token.value[0].value[0]
+
+    value.class.should == Rlint::Token::Token
+    value.type.should  == :integer
+    value.value.should == '10'
+  end
+
+  it 'Parse a method definition with parameters' do
+    code = <<-CODE
+def number(a, b = 10, *args, more, &block)
+  return num
+end
+    CODE
+
+    token = Rlint::Parser.new(code).parse[0]
+
+    token.class.should  == Rlint::Token::MethodDefinitionToken
+    token.name.should   == 'number'
+    token.line.should   == 1
+    token.column.should == 4
+
+    params = token.parameters
+
+    params.class.should       == Rlint::Token::ParametersToken
+    params.value.class.should == Array
+
+    # Required parameters
+    params.value.length.should    == 1
+    params.value[0].class.should  == Rlint::Token::VariableToken
+    params.value[0].name.should   == 'a'
+
+    # Optional parameters
+    params.optional.class.should  == Array
+    params.optional.length.should == 1
+
+    optional = params.optional[0]
+
+    optional.class.should  == Rlint::Token::VariableToken
+    optional.name.should   == 'b'
+
+    optional.value.class.should == Rlint::Token::Token
+    optional.value.type.should  == :integer
+    optional.value.value.should == '10'
+
+    # Rest parameters
+    params.rest.name.should  == 'args'
+    params.rest.class.should == Rlint::Token::VariableToken
+
+    # More parameters
+    params.more.class.should   == Array
+    params.more.length.should  == 1
+    params.more[0].name.should == 'more'
+
+    # Block parameters
+    params.block.class.should == Rlint::Token::VariableToken
+    params.block.name.should  == 'block'
+  end
+
+  it 'Parse a class method definition' do
+    code = <<-CODE
+def self.example(number = 10)
+  return number
+end
+    CODE
+
+    token = Rlint::Parser.new(code).parse[0]
+
+    token.class.should == Rlint::Token::MethodDefinitionToken
+    token.name.should  == 'example'
+
+    token.receiver.class.should == Rlint::Token::VariableToken
+    token.receiver.name.should  == 'self'
+    token.receiver.type.should  == :keyword
+
+    token.operator.class.should == Rlint::Token::Token
+    token.operator.value.should == '.'
+    token.operator.type.should  == :period
+
+    token.parameters.class.should           == Rlint::Token::ParametersToken
+    token.parameters.optional.class.should  == Array
+    token.parameters.optional.length.should == 1
+  end
+
+  it 'Parse a private method definition' do
+    code = <<-CODE
+private
+
+def example
+  return 10
+end
+    CODE
+
+    token = Rlint::Parser.new(code).parse[1]
+
+    token.class.should      == Rlint::Token::MethodDefinitionToken
+    token.name.should       == 'example'
+    token.visibility.should == :private
+
+    token.value.class.should  == Array
+    token.value.length.should == 1
+  end
+
+  it 'Parse a method call' do
+    token = Rlint::Parser.new('puts').parse[0]
+
+    token.class.should == Rlint::Token::MethodToken
+    token.name.should  == 'puts'
+  end
+
+  it 'Parse a method call with parenthesis' do
+    token = Rlint::Parser.new('puts()').parse[0]
+
+    token.class.should == Rlint::Token::MethodToken
+    token.name.should  == 'puts'
+  end
+
+  it 'Parse a method call with parenthesis and parameters' do
+    token = Rlint::Parser.new('puts(10)').parse[0]
+
+    token.class.should == Rlint::Token::MethodToken
+    token.name.should  == 'puts'
+
+    token.parameters.class.should  == Array
+    token.parameters.length.should == 1
+
+    param = token.parameters[0]
+
+    param.class.should == Rlint::Token::Token
+    param.type.should  == :integer
+    param.value.should == '10'
+  end
+
+  it 'Parse a method call with parameters but without parenthesis' do
+    token = Rlint::Parser.new('puts 10').parse[0]
+
+    token.class.should == Rlint::Token::MethodToken
+    token.name.should  == 'puts'
+
+    token.parameters.class.should  == Array
+    token.parameters.length.should == 1
+
+    param = token.parameters[0]
+
+    param.class.should == Rlint::Token::Token
+    param.type.should  == :integer
+    param.value.should == '10'
+  end
+
+  it 'Parse a method called on an object with parenthesis' do
+    token = Rlint::Parser.new('Struct.new(:name)').parse[0]
+
+    token.class.should == Rlint::Token::MethodToken
+    token.name.should  == 'new'
+
+    token.receiver.class.should == Rlint::Token::VariableToken
+    token.receiver.type.should  == :constant
+    token.receiver.name.should  == 'Struct'
+
+    token.parameters.class.should  == Array
+    token.parameters.length.should == 1
+
+    token.parameters[0].class.should == Rlint::Token::Token
+    token.parameters[0].type.should  == :symbol
+    token.parameters[0].value.should == 'name'
+  end
+
+  it 'Parse a method called on an object without parenthesis' do
+    token = Rlint::Parser.new('Struct.new :name').parse[0]
+
+    token.class.should == Rlint::Token::MethodToken
+    token.name.should  == 'new'
+
+    token.receiver.class.should == Rlint::Token::VariableToken
+    token.receiver.type.should  == :constant
+    token.receiver.name.should  == 'Struct'
+
+    token.parameters.class.should  == Array
+    token.parameters.length.should == 1
+
+    token.parameters[0].class.should == Rlint::Token::Token
+    token.parameters[0].type.should  == :symbol
+    token.parameters[0].value.should == 'name'
+  end
+
+  it 'Parse a method called on an object with a block passed' do
+    token = Rlint::Parser.new('Foo.bar { |example| example }').parse[0]
+
+    token.class.should == Rlint::Token::MethodToken
+    token.name.should  == 'bar'
+
+    token.receiver.class.should == Rlint::Token::VariableToken
+    token.receiver.type.should  == :constant
+    token.receiver.name.should  == 'Foo'
+
+    token.block.class.should            == Rlint::Token::BlockToken
+    token.block.parameters.class.should == Rlint::Token::ParametersToken
+
+    token.block.parameters.value.class.should  == Array
+    token.block.parameters.value.length.should == 1
+  end
+
+  it 'Parse a method call with a bare Hash as a parameter' do
+    token = Rlint::Parser.new('language(:name => "Ruby")').parse[0]
+
+    token.class.should == Rlint::Token::MethodToken
+
+    token.parameters.class.should  == Array
+    token.parameters.length.should == 1
+
+    token.parameters[0].class.should == Rlint::Token::Token
+    token.parameters[0].type.should  == :hash
+  end
+
+  it 'Parse a method call with a bare JSON style Hash as a parameter' do
+    token = Rlint::Parser.new('language(name: "Ruby")').parse[0]
+
+    token.class.should == Rlint::Token::MethodToken
+
+    token.parameters.class.should  == Array
+    token.parameters.length.should == 1
+
+    token.parameters[0].class.should == Rlint::Token::Token
+    token.parameters[0].type.should  == :hash
+  end
+end

data/spec/rlint/parser/modules.rb

@@ -0,0 +1,49 @@
+require File.expand_path('../../../helper', __FILE__)
+
+describe 'Rlint::Parser' do
+  it 'Parse a module' do
+    code = <<-CODE
+module Foo
+  def foo
+    return 'foo'
+  end
+end
+    CODE
+
+    token = Rlint::Parser.new(code).parse[0]
+
+    token.class.should  == Rlint::Token::Token
+    token.type.should   == :module
+    token.name.should   == ['Foo']
+    token.line.should   == 1
+    token.column.should == 7
+
+    token.value.class.should  == Array
+    token.value.length.should == 1
+
+    token.value[0].class.should == Rlint::Token::MethodDefinitionToken
+    token.value[0].name.should  == 'foo'
+  end
+
+  it 'Parse a module with multiple name segments' do
+    token = Rlint::Parser.new('module A::B; end').parse[0]
+
+    token.class.should == Rlint::Token::Token
+    token.type.should  == :module
+    token.name.should  == ['A', 'B']
+
+    token.line.should   == 1
+    token.column.should == 7
+  end
+
+  it 'Parse a top level module' do
+    token = Rlint::Parser.new('module ::A::B; end').parse[0]
+
+    token.class.should == Rlint::Token::Token
+    token.type.should  == :module
+    token.name.should  == ['A', 'B']
+
+    token.line.should   == 1
+    token.column.should == 9
+  end
+end

data/spec/rlint/parser/objects.rb

@@ -0,0 +1,39 @@
+require File.expand_path('../../../helper', __FILE__)
+
+describe 'Rlint::Parser' do
+  it 'Parse the reference of an object attribute' do
+    code = <<-CODE
+obj = Struct.new(:name).new('Ruby')
+obj.name
+    CODE
+
+    token = Rlint::Parser.new(code).parse[1]
+
+    token.class.should == Rlint::Token::MethodToken
+    token.name.should  == 'name'
+
+    token.receiver.class.should == Rlint::Token::VariableToken
+    token.receiver.name.should  == 'obj'
+    token.receiver.type.should  == :local_variable
+  end
+
+  it 'Parse the assignment of a value to an object attribute' do
+    code = <<-CODE
+obj = Struct.new(:name).new('Ruby')
+obj.name = 'Ruby'
+    CODE
+
+    token = Rlint::Parser.new(code).parse[1]
+
+    token.class.should == Rlint::Token::AssignmentToken
+    token.name.should  == 'name'
+
+    token.value.class.should == Rlint::Token::Token
+    token.value.type.should  == :string
+    token.value.value.should == 'Ruby'
+
+    token.receiver.class.should == Rlint::Token::VariableToken
+    token.receiver.name.should  == 'obj'
+    token.receiver.type.should  == :local_variable
+  end
+end

data/spec/rlint/parser/operators.rb

@@ -0,0 +1,75 @@
+require File.expand_path('../../../helper', __FILE__)
+
+describe 'Rlint::Parser' do
+  it 'Parse the use of various operators' do
+    ['*', '-', '+', '/', '%', '^'].each do |operator|
+      token = Rlint::Parser.new("10 #{operator} 2").parse[0]
+
+      token.class.should  == Rlint::Token::Token
+      token.type.should   == :binary
+
+      token.value.class.should  == Array
+      token.value.length.should == 3
+
+      left, op, right = token.value
+
+      left.class.should == Rlint::Token::Token
+      left.type.should  == :integer
+      left.value.should == '10'
+
+      op.should == operator.to_sym
+
+      right.class.should == Rlint::Token::Token
+      right.type.should  == :integer
+      right.value.should == '2'
+    end
+  end
+
+  it 'Parse the use of multiple operators' do
+    token = Rlint::Parser.new('10 and 20 and 30').parse[0]
+
+    token.class.should  == Rlint::Token::Token
+    token.type.should   == :binary
+
+    token.value.class.should  == Array
+    token.value.length.should == 3
+
+    first = token.value[0]
+    last  = token.value[2]
+
+    first.class.should        == Rlint::Token::Token
+    first.type.should         == :binary
+    first.value.length.should == 3
+
+    first.value[0].class.should == Rlint::Token::Token
+    first.value[0].type.should  == :integer
+    first.value[0].value.should == '10'
+
+    first.value[1].should == :and
+
+    first.value[2].class.should == Rlint::Token::Token
+    first.value[2].type.should  == :integer
+    first.value[2].value.should == '20'
+
+    token.value[1].should == :and
+
+    last.class.should == Rlint::Token::Token
+    last.type.should  == :integer
+    last.value.should == '30'
+  end
+
+  it 'Parse unary operators' do
+    token = Rlint::Parser.new('!foobar').parse[0]
+
+    token.class.should == Rlint::Token::Token
+    token.type.should  == :unary
+
+    token.value.class.should  == Array
+    token.value.length.should == 2
+
+    token.value[0].should == :!
+
+    token.value[1].class.should == Rlint::Token::MethodToken
+    token.value[1].name.should  == 'foobar'
+  end
+end

data/spec/rlint/parser/procs.rb

@@ -0,0 +1,113 @@
+require File.expand_path('../../../helper', __FILE__)
+
+describe 'Rlint::Parser' do
+  it 'Parse a Proc' do
+    token = Rlint::Parser.new('proc { |example| example }').parse[0]
+
+    token.class.should == Rlint::Token::MethodToken
+    token.name.should  == 'proc'
+
+    token.block.class.should            == Rlint::Token::BlockToken
+    token.block.parameters.class.should == Rlint::Token::ParametersToken
+    token.block.type.should             == :block
+
+    params = token.block.parameters
+
+    params.value.class.should  == Array
+    params.value.length.should == 1
+
+    params.value[0].class.should == Rlint::Token::VariableToken
+    params.value[0].type.should  == :local_variable
+    params.value[0].name.should  == 'example'
+
+    token.block.value.class.should  == Array
+    token.block.value.length.should == 1
+  end
+
+  it 'Parse a proc using do/end instead of curly braces' do
+    token = Rlint::Parser.new('proc do |example|; example; end').parse[0]
+
+    token.class.should == Rlint::Token::MethodToken
+    token.name.should  == 'proc'
+
+    token.block.class.should            == Rlint::Token::BlockToken
+    token.block.parameters.class.should == Rlint::Token::ParametersToken
+    token.block.type.should             == :block
+
+    params = token.block.parameters
+
+    params.value.class.should  == Array
+    params.value.length.should == 1
+
+    params.value[0].class.should == Rlint::Token::VariableToken
+    params.value[0].type.should  == :local_variable
+    params.value[0].name.should  == 'example'
+
+    token.block.value.class.should  == Array
+    token.block.value.length.should == 1
+  end
+
+  it 'Parse a Proc using Proc.new' do
+    token = Rlint::Parser.new('Proc.new { |example| example }').parse[0]
+
+    token.class.should == Rlint::Token::MethodToken
+    token.name.should  == 'new'
+
+    token.receiver.class.should == Rlint::Token::VariableToken
+    token.receiver.name.should  == 'Proc'
+    token.receiver.type.should  == :constant
+
+    token.block.class.should            == Rlint::Token::BlockToken
+    token.block.parameters.class.should == Rlint::Token::ParametersToken
+    token.block.type.should             == :block
+
+    token.block.parameters.value.class.should  == Array
+    token.block.parameters.value.length.should == 1
+
+    param = token.block.parameters.value[0]
+
+    param.class.should == Rlint::Token::VariableToken
+    param.name.should  == 'example'
+    param.type.should  == :local_variable
+  end
+
+  it 'Parse a Lambda' do
+    token = Rlint::Parser.new('lambda { |example| example }').parse[0]
+
+    token.class.should == Rlint::Token::MethodToken
+    token.name.should  == 'lambda'
+
+    token.block.class.should == Rlint::Token::BlockToken
+    token.block.type.should  == :block
+
+    token.block.parameters.class.should        == Rlint::Token::ParametersToken
+    token.block.parameters.value.class.should  == Array
+    token.block.parameters.value.length.should == 1
+
+    param = token.block.parameters.value[0]
+
+    param.class.should == Rlint::Token::VariableToken
+    param.name.should  == 'example'
+    param.type.should  == :local_variable
+  end
+
+  it 'Parse a Lambda using the dash rocket syntax' do
+    token = Rlint::Parser.new('-> example { example }').parse[0]
+
+    token.class.should == Rlint::Token::BlockToken
+    token.type.should  == :lambda
+
+    token.parameters.class.should        == Rlint::Token::ParametersToken
+    token.parameters.value.class.should  == Array
+    token.parameters.value.length.should == 1
+
+    param = token.parameters.value[0]
+
+    param.class.should == Rlint::Token::VariableToken
+    param.name.should  == 'example'
+    param.type.should  == :local_variable
+
+    token.value.class.should  == Array
+    token.value.length.should == 1
+  end
+end