ruby-lint 0.0.2 → 0.0.3

data/spec/ruby-lint/callback.rb

@@ -1,28 +0,0 @@
-require File.expand_path('../../helper', __FILE__)
-
-describe 'RubyLint::Callback' do
-  it 'Add an error to a report using RubyLint::Callback' do
-    report   = RubyLint::Report.new
-    callback = RubyLint::Callback.new(report)
-
-    report.levels.each do |level|
-      report.messages[level].nil?.should == true
-
-      callback.send(level, "test #{level}", 1, 1)
-
-      report.messages[level].class.should  == Array
-      report.messages[level].length.should == 1
-
-      report.messages[level][0][:message].should == "test #{level}"
-    end
-  end
-
-  it 'Ignore errors (when disabled) when adding one using RubyLint::Callback' do
-    report   = RubyLint::Report.new('(ruby-lint)', [:warning])
-    callback = RubyLint::Callback.new(report)
-
-    callback.send(:error, 'test error', 1, 1)
-
-    report.messages[:error].nil?.should == true
-  end
-end

data/spec/ruby-lint/constant_importer.rb

@@ -1,27 +0,0 @@
-require File.expand_path('../../helper', __FILE__)
-
-describe 'RubyLint::ConstantImporter' do
-  it 'Import methods from the Kernel constant' do
-    imported = RubyLint::ConstantImporter.import([:Kernel])
-
-    imported.key?('Kernel').should == true
-
-    found = imported['Kernel'].lookup(:method, 'puts')
-
-    found.class.should == RubyLint::Definition
-
-    found.token.class.should      == RubyLint::Token::MethodDefinitionToken
-    found.token.name.should       == 'puts'
-    found.token.visibility.should == :public
-
-    found.token.parameters.class.should == RubyLint::Token::ParametersToken
-
-    found.token.parameters.value.class.should  == Array
-    found.token.parameters.value.length.should == 0
-
-    found.token.parameters.rest.class.should == RubyLint::Token::VariableToken
-    found.token.parameters.rest.name.should  == ''
-    found.token.parameters.rest.type.should  == :local_variable
-    found.token.parameters.rest.event.should == :local_variable
-  end
-end

data/spec/ruby-lint/definition.rb

@@ -1,96 +0,0 @@
-require File.expand_path('../../helper', __FILE__)
-
-describe 'RubyLint::Definition' do
-  it 'Look up a symbol in a scope' do
-    scope = RubyLint::Definition.new
-
-    scope.lookup(:local_variable, 'number').should == nil
-
-    scope.add(:local_variable, 'number', 10)
-
-    scope.lookup(:local_variable, 'number').should == 10
-  end
-
-  it 'Look up a symbol in a parent scope' do
-    parent = RubyLint::Definition.new
-    child  = RubyLint::Definition.new(parent)
-
-    parent.add(:local_variable, 'number', 10)
-    parent.add(:global_variable, '$number', 20)
-
-    parent.lookup(:local_variable, 'number').should   == 10
-    parent.lookup(:global_variable, '$number').should == 20
-
-    child.lookup(:local_variable, 'number').should   == nil
-    child.lookup(:global_variable, '$number').should == 20
-  end
-
-  it 'Create a scope with default Ruby constants and methods' do
-    scope = RubyLint::Definition.new(nil, :lazy => true, :kernel => true)
-    found = scope.lookup(:constant, 'Kernel')
-
-    found.class.should == RubyLint::Definition
-
-    method = found.lookup(:method, 'warn')
-
-    method.class.should == RubyLint::Definition
-
-    method.token.class.should == RubyLint::Token::MethodDefinitionToken
-    method.token.name.should  == 'warn'
-
-    method.token.parameters.value.class.should  == Array
-    method.token.parameters.value.length.should == 1
-
-    param = method.token.parameters.value[0]
-
-    param.class.should == RubyLint::Token::VariableToken
-    param.name.should  == ''
-    param.type.should  == :local_variable
-  end
-
-  it 'Lazy import the Time class' do
-    scope = RubyLint::Definition.new
-
-    scope.lookup(:constant, 'Time').nil?.should == true
-
-    scope = RubyLint::Definition.new(nil, :lazy => true)
-
-    scope.lookup(:constant, 'Time').class.should == RubyLint::Definition
-  end
-
-  it 'Lazy import the Encoding class and a child constant' do
-    scope = RubyLint::Definition.new(nil, :lazy => true)
-    enc   = scope.lookup(:constant, 'Encoding')
-
-    enc.class.should == RubyLint::Definition
-
-    enc.lookup(:constant, 'BINARY').class.should == RubyLint::Definition
-  end
-
-  it 'Lazy import RubyLint::Definition' do
-    scope = RubyLint::Definition.new(nil, :lazy => true)
-
-    ruby_lint = scope.lookup(:constant, 'RubyLint')
-
-    ruby_lint.class.should == RubyLint::Definition
-
-    ruby_lint_scope = ruby_lint.lookup(:constant, 'Definition')
-
-    ruby_lint_scope.class.should == RubyLint::Definition
-
-    ruby_lint_scope.lookup(:constant, 'LOOKUP_PARENT').class.should == RubyLint::Definition
-  end
-
-  it 'Create a scope with multiple parent scopes' do
-    scope_1 = RubyLint::Definition.new
-    scope_2 = RubyLint::Definition.new
-
-    scope_1.add(:method, 'method_1', true)
-    scope_2.add(:method, 'method_2', true)
-
-    scope_3 = RubyLint::Definition.new([scope_1, scope_2])
-
-    scope_3.lookup(:method, 'method_1').should == true
-    scope_3.lookup(:method, 'method_2').should == true
-  end
-end

data/spec/ruby-lint/formatter/text.rb

@@ -1,21 +0,0 @@
-require File.expand_path('../../../helper', __FILE__)
-
-describe 'RubyLint::Formatter::Text' do
-  it 'Format a report in plain text' do
-    report_a  = RubyLint::Report.new('a.rb')
-    report_b  = RubyLint::Report.new('b.rb')
-    formatter = RubyLint::Formatter::Text.new
-
-    report_a.add(:info, 'test info', 2, 1)
-
-    report_b.add(:error, 'test error', 1, 1)
-    report_b.add(:info, 'test info b.rb', 3, 1)
-
-    output_a = 'a.rb: info: line 2, column 1: test info'
-    output_b = "b.rb: error: line 1, column 1: test error\n" \
-      'b.rb: info: line 3, column 1: test info b.rb'
-
-    formatter.format(report_a).should == output_a
-    formatter.format(report_b).should == output_b
-  end
-end

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

@@ -1,147 +0,0 @@
-require File.expand_path('../../../helper', __FILE__)
-
-describe 'RubyLint::Parser' do
-  it 'Parse an Array' do
-    token = RubyLint::Parser.new('[10, 20]').parse[0]
-
-    token.class.should == RubyLint::Token::Token
-    token.type.should  == :array
-
-    token.value.class.should  == Array
-    token.value.length.should == 2
-
-    token.line.should   == 1
-    token.column.should == 8
-    token.code.should   == '[10, 20]'
-
-    token.value[0].class.should == RubyLint::Token::Token
-    token.value[0].type.should  == :integer
-    token.value[0].value.should == '10'
-
-    token.value[1].class.should == RubyLint::Token::Token
-    token.value[1].type.should  == :integer
-    token.value[1].value.should == '20'
-  end
-
-  it 'Parse an Array using %w{}' do
-    token = RubyLint::Parser.new('%w{10 20}').parse[0]
-
-    token.class.should == RubyLint::Token::Token
-    token.type.should  == :array
-
-    token.value.class.should  == Array
-    token.value.length.should == 2
-
-    token.line.should   == 1
-    token.column.should == 9
-    token.code.should   == '%w{10 20}'
-
-    token.value[0].class.should == RubyLint::Token::Token
-    token.value[0].type.should  == :string
-    token.value[0].value.should == '10'
-
-    token.value[1].class.should == RubyLint::Token::Token
-    token.value[1].type.should  == :string
-    token.value[1].value.should == '20'
-  end
-
-  it 'Parse an Array using %W{}' do
-    token = RubyLint::Parser.new('%W{10 20}').parse[0]
-
-    token.class.should == RubyLint::Token::Token
-    token.type.should  == :array
-
-    token.value.class.should  == Array
-    token.value.length.should == 2
-
-    token.line.should   == 1
-    token.column.should == 9
-    token.code.should   == '%W{10 20}'
-
-    token.value[0].class.should == RubyLint::Token::Token
-    token.value[0].type.should  == :string
-    token.value[0].value.should == '10'
-
-    token.value[1].class.should == RubyLint::Token::Token
-    token.value[1].type.should  == :string
-    token.value[1].value.should == '20'
-  end
-
-  it 'Parse an Array index reference' do
-    token = RubyLint::Parser.new("[10][0]").parse[0]
-
-    token.class.should  == RubyLint::Token::Token
-    token.line.should   == 1
-    token.column.should == 4
-    token.code.should   == '[10][0]'
-
-    token.key.class.should  == Array
-    token.key.length.should == 1
-
-    token.key[0].class.should == RubyLint::Token::Token
-    token.key[0].type.should  == :integer
-    token.key[0].value.should == '0'
-  end
-
-  it 'Parse an Array index reference using a variable' do
-    token = RubyLint::Parser.new("numbers = [10]\nnumbers[0]").parse[1]
-
-    token.class.should  == RubyLint::Token::VariableToken
-    token.line.should   == 2
-    token.column.should == 0
-    token.name.should   == 'numbers'
-    token.code.should   == 'numbers[0]'
-
-    token.key.class.should  == Array
-    token.key.length.should == 1
-
-    token.key[0].class.should == RubyLint::Token::Token
-    token.key[0].type.should  == :integer
-    token.key[0].value.should == '0'
-  end
-
-  it 'Parse multiple Array index references' do
-    token = RubyLint::Parser.new("numbers = [10]\nnumbers[0,1]").parse[1]
-
-    token.class.should  == RubyLint::Token::VariableToken
-    token.line.should   == 2
-    token.column.should == 0
-    token.name.should   == 'numbers'
-    token.code.should   == 'numbers[0,1]'
-
-    token.key.class.should  == Array
-    token.key.length.should == 2
-
-    token.key[0].class.should == RubyLint::Token::Token
-    token.key[0].type.should  == :integer
-    token.key[0].value.should == '0'
-
-    token.key[1].class.should == RubyLint::Token::Token
-    token.key[1].type.should  == :integer
-    token.key[1].value.should == '1'
-  end
-
-  it 'Parse the assignment of a value to an array index' do
-    token = RubyLint::Parser.new("numbers = []\nnumbers[0] = 10").parse[1]
-
-    token.class.should == RubyLint::Token::AssignmentToken
-
-    token.line.should   == 2
-    token.column.should == 12
-    token.code.should   == 'numbers[0] = 10'
-
-    token.value.class.should == RubyLint::Token::Token
-    token.value.type.should  == :integer
-    token.value.value.should == '10'
-
-    token.receiver.class.should == RubyLint::Token::VariableToken
-    token.receiver.name.should  == 'numbers'
-
-    token.receiver.key.class.should  == Array
-    token.receiver.key.length.should == 1
-
-    token.receiver.key[0].class.should == RubyLint::Token::Token
-    token.receiver.key[0].type.should  == :integer
-    token.receiver.key[0].value.should == '0'
-  end
-end

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

@@ -1,183 +0,0 @@
-require File.expand_path('../../../helper', __FILE__)
-
-describe 'Rlint::Parser' do
-  it 'Parse the assignment of a value to a local and * variable' do
-    ast = RubyLint::Parser.new('a, * = 10').parse[0]
-
-    ast.class.should  == Array
-    ast.length.should == 1
-
-    token = ast[0]
-
-    token.class.should == RubyLint::Token::AssignmentToken
-    token.type.should  == :local_variable
-    token.name.should  == 'a'
-
-    token.value.class.should == RubyLint::Token::Token
-    token.value.type.should  == :integer
-    token.value.value.should == '10'
-  end
-
-  it 'Parse a left hand expand assignment' do
-    tokens = RubyLint::Parser.new('*numbers = 10').parse[0]
-
-    tokens.class.should  == Array
-    tokens.length.should == 1
-
-    token = tokens[0]
-
-    token.class.should == RubyLint::Token::AssignmentToken
-    token.type.should  == :local_variable
-    token.expand.should == true
-    token.name.should  == 'numbers'
-
-    token.value.class.should == RubyLint::Token::Token
-    token.value.type.should  == :array
-
-    token.value.value.class.should  == Array
-    token.value.value.length.should == 1
-
-    val = token.value.value[0]
-
-    val.class.should == RubyLint::Token::Token
-    val.type.should  == :integer
-    val.value.should == '10'
-  end
-
-  it 'Parse a left hand expand and local variable assignment' do
-    tokens = RubyLint::Parser.new('number, *numbers = 10').parse[0]
-
-    tokens.class.should  == Array
-    tokens.length.should == 2
-
-    number  = tokens[0]
-    numbers = tokens[1]
-
-    number.class.should == RubyLint::Token::AssignmentToken
-    number.name.should  == 'number'
-    number.type.should  == :local_variable
-    number.expand.should == false
-
-    number.value.class.should == RubyLint::Token::Token
-    number.value.type.should  == :integer
-    number.value.value.should == '10'
-
-    numbers.class.should == RubyLint::Token::AssignmentToken
-    numbers.name.should  == 'numbers'
-    numbers.type.should  == :local_variable
-    numbers.expand.should == true
-
-    numbers.value.class.should        == RubyLint::Token::Token
-    numbers.value.type.should         == :array
-    numbers.value.value.length.should == 0
-  end
-
-  it 'Parse a mass assignment using an expand variable on the left' do
-    tokens = RubyLint::Parser.new('*numbers, number, numberx = 10').parse[0]
-
-    tokens.class.should  == Array
-    tokens.length.should == 3
-
-    numbers = tokens[0]
-    number  = tokens[1]
-    numberx = tokens[2]
-
-    numbers.class.should == RubyLint::Token::AssignmentToken
-    numbers.name.should  == 'numbers'
-    numbers.type.should  == :local_variable
-    numbers.expand.should == true
-
-    numbers.value.class.should        == RubyLint::Token::Token
-    numbers.value.type.should         == :array
-    numbers.value.value.class.should  == Array
-    numbers.value.value.length.should == 0
-
-    number.class.should == RubyLint::Token::AssignmentToken
-    number.name.should  == 'number'
-    number.type.should  == :local_variable
-    number.expand.should == false
-
-    number.value.class.should == RubyLint::Token::Token
-    number.value.type.should  == :integer
-    number.value.value.should == '10'
-
-    numberx.class.should  == RubyLint::Token::AssignmentToken
-    numberx.name.should   == 'numberx'
-    numberx.type.should   == :local_variable
-    numberx.expand.should == false
-
-    numberx.value.nil?.should == true
-  end
-
-  it 'Parse the assignment of 2 values to 2 variables using an expander' do
-    tokens = RubyLint::Parser.new('*numbers, number = 10, 20').parse[0]
-
-    tokens.class.should  == Array
-    tokens.length.should == 2
-
-    numbers = tokens[0]
-    number  = tokens[1]
-
-    numbers.class.should == RubyLint::Token::AssignmentToken
-    numbers.name.should  == 'numbers'
-    numbers.type.should  == :local_variable
-
-    numbers.value.class.should == RubyLint::Token::Token
-    numbers.value.type.should  == :array
-
-    numbers.value.value.class.should  == Array
-    numbers.value.value.length.should == 1
-
-    numbers_val = numbers.value.value[0]
-
-    numbers_val.class.should == RubyLint::Token::Token
-    numbers_val.type.should  == :integer
-    numbers_val.value.should == '10'
-
-    number.class.should == RubyLint::Token::AssignmentToken
-    number.name.should  == 'number'
-    number.type.should  == :local_variable
-
-    number.value.class.should == RubyLint::Token::Token
-    number.value.type.should  == :integer
-    number.value.value.should == '20'
-  end
-
-  it 'Parse the assignment of 2 values to 3 variables using an expander' do
-    tokens = RubyLint::Parser.new('*numbers, number, numberx = 10, 20') \
-      .parse[0]
-
-    tokens.class.should  == Array
-    tokens.length.should == 3
-
-    numbers = tokens[0]
-    number  = tokens[1]
-    numberx = tokens[2]
-
-    numbers.class.should == RubyLint::Token::AssignmentToken
-    numbers.name.should  == 'numbers'
-    numbers.type.should  == :local_variable
-
-    numbers.value.class.should == RubyLint::Token::Token
-    numbers.value.type.should  == :array
-
-    numbers.value.value.class.should  == Array
-    numbers.value.value.length.should == 0
-
-    number.class.should == RubyLint::Token::AssignmentToken
-    number.name.should  == 'number'
-    number.type.should  == :local_variable
-
-    number.value.class.should == RubyLint::Token::Token
-    number.value.type.should  == :integer
-    number.value.value.should == '10'
-
-    numberx.class.should == RubyLint::Token::AssignmentToken
-    numberx.name.should  == 'numberx'
-    numberx.type.should  == :local_variable
-
-    numberx.value.class.should == RubyLint::Token::Token
-    numberx.value.type.should  == :integer
-    numberx.value.value.should == '20'
-  end
-end