ruby-lint 0.0.1a → 0.0.1a1

data/spec/{rlint → ruby-lint}/analyze/definitions/variables.rb

@@ -1,6 +1,6 @@
 require File.expand_path('../../../../helper', __FILE__)
 
-describe 'Rlint::Analyze::Definitions: variables' do
+describe 'RubyLint::Analyze::Definitions: variables' do
   it 'Build a list of variables defined in the global scope' do
     code = <<-CODE
 number   = 10
@@ -10,15 +10,15 @@ $number  = 10
 NUMBER   = 10
     CODE
 
-    tokens   = Rlint::Parser.new(code).parse
-    iterator = Rlint::Iterator.new
+    tokens   = RubyLint::Parser.new(code).parse
+    iterator = RubyLint::Iterator.new
 
-    iterator.bind(Rlint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::Definitions)
     iterator.run(tokens)
 
     scope = iterator.storage[:scope]
 
-    scope.class.should == Rlint::Definition
+    scope.class.should == RubyLint::Definition
 
     lvar  = scope.lookup(:local_variable, 'number')
     ivar  = scope.lookup(:instance_variable, '@number')
@@ -26,43 +26,43 @@ NUMBER   = 10
     gvar  = scope.lookup(:global_variable, '$number')
     const = scope.lookup(:constant, 'NUMBER')
 
-    lvar.class.should             == Rlint::Definition
-    lvar.token.class.should       == Rlint::Token::AssignmentToken
+    lvar.class.should             == RubyLint::Definition
+    lvar.token.class.should       == RubyLint::Token::AssignmentToken
     lvar.token.name.should        == 'number'
     lvar.token.type.should        == :local_variable
-    lvar.token.value.class.should == Rlint::Token::Token
+    lvar.token.value.class.should == RubyLint::Token::Token
     lvar.token.value.type.should  == :integer
     lvar.token.value.value.should == '10'
 
-    ivar.class.should             == Rlint::Definition
-    ivar.token.class.should       == Rlint::Token::AssignmentToken
+    ivar.class.should             == RubyLint::Definition
+    ivar.token.class.should       == RubyLint::Token::AssignmentToken
     ivar.token.name.should        == '@number'
     ivar.token.type.should        == :instance_variable
-    ivar.token.value.class.should == Rlint::Token::Token
+    ivar.token.value.class.should == RubyLint::Token::Token
     ivar.token.value.type.should  == :integer
     ivar.token.value.value.should == '10'
 
-    cvar.class.should             == Rlint::Definition
-    cvar.token.class.should       == Rlint::Token::AssignmentToken
+    cvar.class.should             == RubyLint::Definition
+    cvar.token.class.should       == RubyLint::Token::AssignmentToken
     cvar.token.name.should        == '@@number'
     cvar.token.type.should        == :class_variable
-    cvar.token.value.class.should == Rlint::Token::Token
+    cvar.token.value.class.should == RubyLint::Token::Token
     cvar.token.value.type.should  == :integer
     cvar.token.value.value.should == '10'
 
-    gvar.class.should             == Rlint::Definition
-    gvar.token.class.should       == Rlint::Token::AssignmentToken
+    gvar.class.should             == RubyLint::Definition
+    gvar.token.class.should       == RubyLint::Token::AssignmentToken
     gvar.token.name.should        == '$number'
     gvar.token.type.should        == :global_variable
-    gvar.token.value.class.should == Rlint::Token::Token
+    gvar.token.value.class.should == RubyLint::Token::Token
     gvar.token.value.type.should  == :integer
     gvar.token.value.value.should == '10'
 
-    const.class.should             == Rlint::Definition
-    const.token.class.should       == Rlint::Token::AssignmentToken
+    const.class.should             == RubyLint::Definition
+    const.token.class.should       == RubyLint::Token::AssignmentToken
     const.token.name.should        == 'NUMBER'
     const.token.type.should        == :constant
-    const.token.value.class.should == Rlint::Token::Token
+    const.token.value.class.should == RubyLint::Token::Token
     const.token.value.type.should  == :integer
     const.token.value.value.should == '10'
   end
@@ -74,10 +74,10 @@ def example_method
 end
     CODE
 
-    tokens   = Rlint::Parser.new(code).parse
-    iterator = Rlint::Iterator.new
+    tokens   = RubyLint::Parser.new(code).parse
+    iterator = RubyLint::Iterator.new
 
-    iterator.bind(Rlint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::Definitions)
     iterator.run(tokens)
 
     scope = iterator.storage[:scope]
@@ -86,17 +86,17 @@ end
 
     method = scope.lookup(:instance_method, 'example_method')
 
-    method.class.should == Rlint::Definition
+    method.class.should == RubyLint::Definition
 
-    method.token.class.should      == Rlint::Token::MethodDefinitionToken
+    method.token.class.should      == RubyLint::Token::MethodDefinitionToken
     method.token.name.should       == 'example_method'
     method.token.value.nil?.should == true
 
     variable = method.lookup(:local_variable, 'number')
 
-    variable.class.should             == Rlint::Definition
+    variable.class.should             == RubyLint::Definition
     variable.token.name.should        == 'number'
-    variable.token.value.class.should == Rlint::Token::Token
+    variable.token.value.class.should == RubyLint::Token::Token
     variable.token.value.type.should  == :integer
     variable.token.value.value.should == '10'
   end

data/spec/{rlint → ruby-lint}/analyze/method_validation.rb

@@ -1,6 +1,6 @@
 require File.expand_path('../../../helper', __FILE__)
 
-describe 'Rlint::Analyze::MethodValidation' do
+describe 'RubyLint::Analyze::MethodValidation' do
   it 'Calling undefined methods' do
     code = <<-CODE
 def defined_method; end
@@ -9,13 +9,13 @@ undefined_method
 defined_method
     CODE
 
-    tokens   = Rlint::Parser.new(code).parse
-    report   = Rlint::Report.new
-    iterator = Rlint::Iterator.new(report)
+    tokens   = RubyLint::Parser.new(code).parse
+    report   = RubyLint::Report.new
+    iterator = RubyLint::Iterator.new(report)
 
-    iterator.bind(Rlint::Analyze::Definitions)
-    iterator.bind(Rlint::Analyze::MethodValidation)
-    iterator.bind(Rlint::Analyze::UndefinedVariables)
+    iterator.bind(RubyLint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::MethodValidation)
+    iterator.bind(RubyLint::Analyze::UndefinedVariables)
     iterator.run(tokens)
 
     report.messages[:error].class.should  == Array
@@ -37,13 +37,13 @@ String.newx
 Foobar.new
     CODE
 
-    tokens   = Rlint::Parser.new(code).parse
-    report   = Rlint::Report.new
-    iterator = Rlint::Iterator.new(report)
+    tokens   = RubyLint::Parser.new(code).parse
+    report   = RubyLint::Report.new
+    iterator = RubyLint::Iterator.new(report)
 
-    iterator.bind(Rlint::Analyze::Definitions)
-    iterator.bind(Rlint::Analyze::MethodValidation)
-    iterator.bind(Rlint::Analyze::UndefinedVariables)
+    iterator.bind(RubyLint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::MethodValidation)
+    iterator.bind(RubyLint::Analyze::UndefinedVariables)
     iterator.run(tokens)
 
     report.messages[:error].class.should  == Array
@@ -72,13 +72,13 @@ foo.upcase
 ''.downcasex
     CODE
 
-    tokens   = Rlint::Parser.new(code).parse
-    report   = Rlint::Report.new
-    iterator = Rlint::Iterator.new(report)
+    tokens   = RubyLint::Parser.new(code).parse
+    report   = RubyLint::Report.new
+    iterator = RubyLint::Iterator.new(report)
 
-    iterator.bind(Rlint::Analyze::Definitions)
-    iterator.bind(Rlint::Analyze::MethodValidation)
-    iterator.bind(Rlint::Analyze::UndefinedVariables)
+    iterator.bind(RubyLint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::MethodValidation)
+    iterator.bind(RubyLint::Analyze::UndefinedVariables)
     iterator.run(tokens)
 
     report.messages[:error].class.should  == Array
@@ -109,13 +109,13 @@ def uppercase(name = 'Ruby', *args, &block)
 end
     CODE
 
-    tokens   = Rlint::Parser.new(code).parse
-    report   = Rlint::Report.new
-    iterator = Rlint::Iterator.new(report)
+    tokens   = RubyLint::Parser.new(code).parse
+    report   = RubyLint::Report.new
+    iterator = RubyLint::Iterator.new(report)
 
-    iterator.bind(Rlint::Analyze::Definitions)
-    iterator.bind(Rlint::Analyze::MethodValidation)
-    iterator.bind(Rlint::Analyze::UndefinedVariables)
+    iterator.bind(RubyLint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::MethodValidation)
+    iterator.bind(RubyLint::Analyze::UndefinedVariables)
     iterator.run(tokens)
 
     report.messages[:error].class.should  == Array
@@ -148,13 +148,13 @@ A::B.invalid_method
 A::B.valid_method
     CODE
 
-    tokens   = Rlint::Parser.new(code).parse
-    report   = Rlint::Report.new
-    iterator = Rlint::Iterator.new(report)
+    tokens   = RubyLint::Parser.new(code).parse
+    report   = RubyLint::Report.new
+    iterator = RubyLint::Iterator.new(report)
 
-    iterator.bind(Rlint::Analyze::Definitions)
-    iterator.bind(Rlint::Analyze::UndefinedVariables)
-    iterator.bind(Rlint::Analyze::MethodValidation)
+    iterator.bind(RubyLint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::UndefinedVariables)
+    iterator.bind(RubyLint::Analyze::MethodValidation)
     iterator.run(tokens)
 
     report.messages[:error].class.should  == Array

data/spec/{rlint → ruby-lint}/analyze/shadowing_variables.rb

@@ -1,6 +1,6 @@
 require File.expand_path('../../../helper', __FILE__)
 
-describe 'Rlint::Analyze::ShadowingVariables' do
+describe 'RubyLint::Analyze::ShadowingVariables' do
   it 'Warn for shadowing outer variables' do
     code = <<-CODE
 number = 10
@@ -10,12 +10,12 @@ number = 10
 end
     CODE
 
-    tokens   = Rlint::Parser.new(code).parse
-    report   = Rlint::Report.new
-    iterator = Rlint::Iterator.new(report)
+    tokens   = RubyLint::Parser.new(code).parse
+    report   = RubyLint::Report.new
+    iterator = RubyLint::Iterator.new(report)
 
-    iterator.bind(Rlint::Analyze::Definitions)
-    iterator.bind(Rlint::Analyze::ShadowingVariables)
+    iterator.bind(RubyLint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::ShadowingVariables)
     iterator.run(tokens)
 
     report.messages[:warning].class.should  == Array

data/spec/{rlint → ruby-lint}/analyze/undefined_variables.rb

@@ -1,6 +1,6 @@
 require File.expand_path('../../../helper', __FILE__)
 
-describe 'Rlint::Analyze::UndefinedVariables' do
+describe 'RubyLint::Analyze::UndefinedVariables' do
   it 'Use of undefined variables' do
     code = <<-CODE
 number = 10
@@ -25,12 +25,12 @@ puts $number
 puts NUMBER
     CODE
 
-    tokens   = Rlint::Parser.new(code).parse
-    report   = Rlint::Report.new
-    iterator = Rlint::Iterator.new(report)
+    tokens   = RubyLint::Parser.new(code).parse
+    report   = RubyLint::Report.new
+    iterator = RubyLint::Iterator.new(report)
 
-    iterator.bind(Rlint::Analyze::Definitions)
-    iterator.bind(Rlint::Analyze::UndefinedVariables)
+    iterator.bind(RubyLint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::UndefinedVariables)
     iterator.run(tokens)
 
     report.messages[:error].class.should  == Array
@@ -61,12 +61,12 @@ puts NUMBER
 
   it 'Default global variables should not trigger errors' do
     code     = Kernel.global_variables.join("\n")
-    tokens   = Rlint::Parser.new(code).parse
-    report   = Rlint::Report.new
-    iterator = Rlint::Iterator.new(report)
+    tokens   = RubyLint::Parser.new(code).parse
+    report   = RubyLint::Report.new
+    iterator = RubyLint::Iterator.new(report)
 
-    iterator.bind(Rlint::Analyze::Definitions)
-    iterator.bind(Rlint::Analyze::UndefinedVariables)
+    iterator.bind(RubyLint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::UndefinedVariables)
     iterator.run(tokens)
 
     report.messages[:error].nil?.should == true
@@ -89,12 +89,12 @@ puts a
 puts b # b was defined inside the method and isn't available outside it
     CODE
 
-    tokens   = Rlint::Parser.new(code).parse
-    report   = Rlint::Report.new
-    iterator = Rlint::Iterator.new(report)
+    tokens   = RubyLint::Parser.new(code).parse
+    report   = RubyLint::Report.new
+    iterator = RubyLint::Iterator.new(report)
 
-    iterator.bind(Rlint::Analyze::Definitions)
-    iterator.bind(Rlint::Analyze::UndefinedVariables)
+    iterator.bind(RubyLint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::UndefinedVariables)
     iterator.run(tokens)
 
     report.messages[:error].class.should  == Array
@@ -124,12 +124,12 @@ end
 puts @number
     CODE
 
-    tokens   = Rlint::Parser.new(code).parse
-    report   = Rlint::Report.new
-    iterator = Rlint::Iterator.new(report)
+    tokens   = RubyLint::Parser.new(code).parse
+    report   = RubyLint::Report.new
+    iterator = RubyLint::Iterator.new(report)
 
-    iterator.bind(Rlint::Analyze::Definitions)
-    iterator.bind(Rlint::Analyze::UndefinedVariables)
+    iterator.bind(RubyLint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::UndefinedVariables)
     iterator.run(tokens)
 
     report.messages[:error].nil?.should == true
@@ -149,12 +149,12 @@ class Person
 end
     CODE
 
-    tokens   = Rlint::Parser.new(code).parse
-    report   = Rlint::Report.new
-    iterator = Rlint::Iterator.new(report)
+    tokens   = RubyLint::Parser.new(code).parse
+    report   = RubyLint::Report.new
+    iterator = RubyLint::Iterator.new(report)
 
-    iterator.bind(Rlint::Analyze::Definitions)
-    iterator.bind(Rlint::Analyze::UndefinedVariables)
+    iterator.bind(RubyLint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::UndefinedVariables)
     iterator.run(tokens)
 
     report.messages[:error].class.should  == Array
@@ -178,12 +178,12 @@ end
 puts A::B
     CODE
 
-    tokens   = Rlint::Parser.new(code).parse
-    report   = Rlint::Report.new
-    iterator = Rlint::Iterator.new(report)
+    tokens   = RubyLint::Parser.new(code).parse
+    report   = RubyLint::Report.new
+    iterator = RubyLint::Iterator.new(report)
 
-    iterator.bind(Rlint::Analyze::Definitions)
-    iterator.bind(Rlint::Analyze::UndefinedVariables)
+    iterator.bind(RubyLint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::UndefinedVariables)
     iterator.run(tokens)
 
     report.messages[:error].class.should  == Array
@@ -202,7 +202,7 @@ puts A::B
 
   it 'Look up a constant using an implicit constant path' do
     code = <<-CODE
-module Rlint
+module RubyLint
   module Derp
     Foobar.name
     ConstantImporter.name
@@ -210,12 +210,12 @@ module Rlint
 end
     CODE
 
-    tokens   = Rlint::Parser.new(code).parse
-    report   = Rlint::Report.new
-    iterator = Rlint::Iterator.new(report)
+    tokens   = RubyLint::Parser.new(code).parse
+    report   = RubyLint::Report.new
+    iterator = RubyLint::Iterator.new(report)
 
-    iterator.bind(Rlint::Analyze::Definitions)
-    iterator.bind(Rlint::Analyze::UndefinedVariables)
+    iterator.bind(RubyLint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::UndefinedVariables)
     iterator.run(tokens)
 
     report.messages[:error].class.should  == Array

data/spec/{rlint → ruby-lint}/analyze/unused_variables.rb

@@ -1,6 +1,6 @@
 require File.expand_path('../../../helper', __FILE__)
 
-describe 'Rlint::Analyze::UnusedVariables' do
+describe 'RubyLint::Analyze::UnusedVariables' do
   it 'Add warnings for unused variables in the global scope' do
     code = <<-CODE
 number   = 10
@@ -19,12 +19,12 @@ puts @@number_2
 puts $number_2
     CODE
 
-    tokens   = Rlint::Parser.new(code).parse
-    report   = Rlint::Report.new
-    iterator = Rlint::Iterator.new(report)
+    tokens   = RubyLint::Parser.new(code).parse
+    report   = RubyLint::Report.new
+    iterator = RubyLint::Iterator.new(report)
 
-    iterator.bind(Rlint::Analyze::Definitions)
-    iterator.bind(Rlint::Analyze::UnusedVariables)
+    iterator.bind(RubyLint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::UnusedVariables)
     iterator.run(tokens)
 
     report.messages[:warning].class.should  == Array
@@ -75,12 +75,12 @@ def example
 end
     CODE
 
-    tokens   = Rlint::Parser.new(code).parse
-    report   = Rlint::Report.new
-    iterator = Rlint::Iterator.new(report)
+    tokens   = RubyLint::Parser.new(code).parse
+    report   = RubyLint::Report.new
+    iterator = RubyLint::Iterator.new(report)
 
-    iterator.bind(Rlint::Analyze::Definitions)
-    iterator.bind(Rlint::Analyze::UnusedVariables)
+    iterator.bind(RubyLint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::UnusedVariables)
     iterator.run(tokens)
 
     report.messages[:warning].class.should  == Array
@@ -131,12 +131,12 @@ class Example
 end
     CODE
 
-    tokens   = Rlint::Parser.new(code).parse
-    report   = Rlint::Report.new
-    iterator = Rlint::Iterator.new(report)
+    tokens   = RubyLint::Parser.new(code).parse
+    report   = RubyLint::Report.new
+    iterator = RubyLint::Iterator.new(report)
 
-    iterator.bind(Rlint::Analyze::Definitions)
-    iterator.bind(Rlint::Analyze::UnusedVariables)
+    iterator.bind(RubyLint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::UnusedVariables)
     iterator.run(tokens)
 
     report.messages[:warning].class.should  == Array
@@ -187,12 +187,12 @@ module Example
 end
     CODE
 
-    tokens   = Rlint::Parser.new(code).parse
-    report   = Rlint::Report.new
-    iterator = Rlint::Iterator.new(report)
+    tokens   = RubyLint::Parser.new(code).parse
+    report   = RubyLint::Report.new
+    iterator = RubyLint::Iterator.new(report)
 
-    iterator.bind(Rlint::Analyze::Definitions)
-    iterator.bind(Rlint::Analyze::UnusedVariables)
+    iterator.bind(RubyLint::Analyze::Definitions)
+    iterator.bind(RubyLint::Analyze::UnusedVariables)
     iterator.run(tokens)
 
     report.messages[:warning].class.should  == Array