ruby-lint 1.1.0 → 2.0.0

data/spec/ruby-lint/ast/node_spec.rb

@@ -1,39 +0,0 @@
-require 'spec_helper'
-
-describe RubyLint::AST::Node do
-  example 'return the value for a scalar' do
-    s(:str, 'hello').children[0].should == 'hello'
-  end
-
-  example 'return the value for an Array' do
-    numbers = s(:array, s(:int, 10), s(:int, 20)).children
-
-    numbers.is_a?(Array).should == true
-    numbers.length.should       == 2
-
-    numbers[0].children[0].should == 10
-    numbers[1].children[0].should == 20
-  end
-
-  example 'return the value for a Hash' do
-    pairs = s(:hash, s(:pair, s(:str, 'number'), s(:int, 10))).children
-
-    pairs.is_a?(Array).should == true
-
-    pairs[0].children[1].children[0].should == 10
-  end
-
-  example 'should try to guess the Ruby class of a node' do
-    s(:str, 'foo').ruby_class.should == 'String'
-  end
-
-  example 'check if a node is a constant path' do
-    s(:const, s(:const, nil, :A), :B).constant_path?.should == true
-  end
-
-  example 'inspect a node using #inspect_oneline' do
-    output = '(const (const nil :A) :B)'
-
-    s(:const, s(:const, nil, :A), :B).inspect_oneline.should == output
-  end
-end

data/spec/ruby-lint/cache_entry_spec.rb

@@ -1,25 +0,0 @@
-require 'spec_helper'
-
-describe RubyLint::CacheEntry do
-  before do
-    @tempfile = Tempfile.new('ruby-lint')
-  end
-
-  after do
-    File.unlink(@tempfile)
-  end
-
-  example 'should validate the file modification times' do
-    mtimes = {@tempfile.path => @tempfile.mtime}
-    entry  = RubyLint::CacheEntry.new(nil, nil, mtimes)
-
-    entry.valid?.should == true
-
-    @tempfile.write('hello')
-
-    # Bit of a hack to ensure that we don't have to use sleep()
-    entry.mtimes[@tempfile.path] = @tempfile.mtime - 10
-
-    entry.valid?.should == false
-  end
-end

data/spec/ruby-lint/cache_spec.rb

@@ -1,53 +0,0 @@
-require 'spec_helper'
-
-describe RubyLint::Cache do
-  before do
-    @directory = Dir.mktmpdir('ruby-lint')
-    @cache     = RubyLint::Cache.new(@directory)
-  end
-
-  after do
-    FileUtils.rm_rf(@directory)
-  end
-
-  example 'create the cache directory' do
-    directory = Dir::Tmpname.create('ruby-lint') { |name| name }
-    cache     = RubyLint::Cache.new(directory)
-
-    File.directory?(directory).should == false
-
-    cache.create_directory!
-
-    File.directory?(directory).should == true
-
-    FileUtils.rm_rf(directory)
-  end
-
-  example 'set and get a cache entry' do
-    value = 20
-
-    @cache.set('example', value)
-
-    @cache.get('example').should == value
-  end
-
-  example 'check if a cache entry exists' do
-    @cache.exists?('example').should == false
-
-    @cache.set('example', 20)
-
-    @cache.exists?('example').should == true
-  end
-
-  example 'return nothing if a cache entry does not exist' do
-    @cache.get('foo').nil?.should == true
-  end
-
-  example 'remove a cache entry' do
-    @cache.set('example', 20)
-
-    @cache.exists?('example').should == true
-    @cache.delete('example')
-    @cache.exists?('example').should == false
-  end
-end

data/spec/ruby-lint/cli/analyze_spec.rb

@@ -1,52 +0,0 @@
-require 'spec_helper'
-require File.expand_path('../../../../lib/ruby-lint/cli', __FILE__)
-
-describe RubyLint::CLI do
-  describe 'analyze' do
-    before do
-      @command = RubyLint::CLI.options.commands['analyze']
-      @output  = StringIO.new
-
-      @command.output_destination = @output
-    end
-
-    example 'analyze a valid Ruby file' do
-      file = fixture_path('valid.rb')
-
-      @command.parse([file])
-
-      @output.rewind
-      @output.read.empty?.should == true
-    end
-
-    example 'analyze an invalid Ruby file' do
-      file = fixture_path('invalid.rb')
-
-      @command.parse([file])
-
-      @output.rewind
-      @output.read.should =~ /undefined method foobar/
-    end
-
-    example 'include benchmarking output' do
-      @command.parse(['-b', fixture_path('valid.rb')])
-
-      @output.rewind
-
-      output = @output.read
-
-      output.should =~ /Execution time:/
-      output.should =~ /Memory usage:/
-    end
-
-    example 'run analysis on an entire directory' do
-      @command.parse([fixture_path('deeply')])
-
-      @output.rewind
-
-      output = @output.read
-
-      output.should =~ /undefined method foobar/
-    end
-  end
-end

data/spec/ruby-lint/configuration_spec.rb

@@ -1,126 +0,0 @@
-require 'spec_helper'
-
-describe RubyLint::Configuration do
-  context 'default options' do
-    before do
-      @configuration = RubyLint::Configuration.new
-    end
-
-    example 'set the default report levels' do
-      @configuration.report_levels.should == RubyLint::Report::DEFAULT_LEVELS
-    end
-
-    example 'set the default presenter' do
-      @configuration.default_presenter.should == RubyLint::Presenter::Text
-    end
-
-    example 'set the default analysis classes' do
-      @configuration.default_analysis_classes.empty?.should == false
-    end
-
-    example 'set the default debug mode' do
-      @configuration.debug.should == false
-    end
-  end
-
-  context 'empty and invalid arguments' do
-    before do
-      @configuration = RubyLint::Configuration.new
-    end
-
-    example 'set the report levels' do
-      @configuration.report_levels = []
-
-      @configuration.report_levels
-        .should == @configuration.default_report_levels
-    end
-
-    example 'set an invalid presenter' do
-      blk = lambda { @configuration.presenter = String }
-
-      blk.should raise_error(ArgumentError, /Invalid presenter/)
-    end
-
-    example 'set the analysis classes' do
-      @configuration.analysis_classes = []
-
-      @configuration.analysis_classes
-        .should == @configuration.default_analysis_classes
-    end
-
-    example 'set an invalid directory' do
-      blk = lambda do
-        @configuration.directories = ['idoubtthisdirectoryexists']
-      end
-
-      blk.should raise_error(ArgumentError, /does not exist/)
-    end
-
-    example 'setting the directories to nil' do
-      @configuration.directories = nil
-
-      @configuration.directories.should == []
-    end
-  end
-
-  context 'setting options via the constructor' do
-    example 'set the report levels' do
-      conf = RubyLint::Configuration.new(:report_levels => ['info'])
-
-      conf.report_levels.should == [:info]
-    end
-
-    example 'set the presenter' do
-      conf = RubyLint::Configuration.new(:presenter => 'json')
-
-      conf.presenter.should == RubyLint::Presenter::JSON
-    end
-
-    example 'set the analysis classes' do
-      conf = RubyLint::Configuration.new(
-        :analysis_classes => ['undefined_methods']
-      )
-
-      conf.analysis_classes.should == [RubyLint::Analysis::UndefinedMethods]
-    end
-
-    example 'set the debug mode' do
-      conf = RubyLint::Configuration.new(:debug => true)
-
-      conf.debug.should == true
-    end
-  end
-
-  context 'setting options directly' do
-    before do
-      @configuration = RubyLint::Configuration.new
-    end
-
-    example 'expand the directories set' do
-      @configuration.directories = ['lib']
-
-      @configuration.directories.should == [
-        File.expand_path('lib')
-      ]
-    end
-  end
-
-  context 'setting options via a configuration file' do
-    example 'set the options from a YAML file' do
-      paths = [fixture_path('config.yml')]
-      conf  = RubyLint::Configuration.load_from_file(paths)
-
-      conf.report_levels.should    == [:info]
-      conf.analysis_classes.should == [RubyLint::Analysis::UndefinedMethods]
-      conf.presenter.should        == RubyLint::Presenter::JSON
-    end
-
-    example 'return a default configuration file' do
-      conf = RubyLint::Configuration.load_from_file([])
-
-      conf.is_a?(RubyLint::Configuration).should == true
-
-      conf.report_levels.should == RubyLint::Report::DEFAULT_LEVELS
-    end
-  end
-end

data/spec/ruby-lint/constant_loader_spec.rb

@@ -1,79 +0,0 @@
-require 'spec_helper'
-
-describe RubyLint::ConstantLoader do
-  before do
-    @definitions = ruby_object.new
-    @loader      = RubyLint::ConstantLoader.new(:definitions => @definitions)
-  end
-
-  context 'bootstrapping definitions' do
-    before do
-      @loader.bootstrap
-    end
-
-    example 'bootstrap Module' do
-      @definitions.has_definition?(:const, 'Module').should == true
-    end
-
-    example 'bootstrap Fixnum' do
-      @definitions.has_definition?(:const, 'Fixnum').should == true
-    end
-
-    example 'bootstrap global variables' do
-      @definitions.has_definition?(:gvar, '$LOAD_PATH').should == true
-    end
-  end
-
-  context 'loading constants' do
-    before do
-      @loader.load_constant('PP')
-    end
-
-    example 'mark bootstrapped constants as loaded' do
-      @loader.bootstrap
-      @loader.loaded?('Module').should == true
-    end
-
-    example 'load a constant' do
-      @loader.loaded?('PP').should == true
-    end
-
-    example 'apply a constant to a definition' do
-      @definitions.has_definition?(:const, 'PP').should == true
-    end
-
-    example 'update the registry' do
-      @loader.registry.include?('PP').should == true
-    end
-  end
-
-  context 'dealing with case sensitivity' do
-    before do
-      @loader.bootstrap
-    end
-
-    example 'do not raise when loading process.rb for the PROCESS constant' do
-      block = lambda { @loader.load_constant('PROCESS') }
-
-      block.should_not raise_error
-    end
-  end
-
-  context 'iterating over an AST' do
-    before do
-      @ast = s(:root, s(:const, nil, 'PP'))
-    end
-
-    example 'load a constant' do
-      @loader.run([@ast])
-      @loader.loaded?('PP').should == true
-    end
-
-    example 'call the correct callbacks' do
-      @loader.should_receive(:on_const)
-        .with(an_instance_of(RubyLint::AST::Node))
-
-      @loader.run([@ast])
-    end
-  end
-end

data/spec/ruby-lint/constant_path.rb

@@ -1,63 +0,0 @@
-require 'spec_helper'
-
-describe RubyLint::ConstantPath do
-  context 'resolving definitions' do
-    before :all do
-      @scope = ruby_object.new(:type => :const, :name => 'Example')
-      @foo   = @scope.define_constant('Foo')
-      @bar   = @foo.define_constant('Bar')
-
-      lvar = ruby_object.new(
-        :type  => :lvar,
-        :name  => 'example',
-        :value => @foo
-      )
-
-      @scope.add_definition(lvar)
-    end
-
-    example 'resolve a path of purely constants' do
-      node = s(:const, s(:const, nil, :Foo), :Bar)
-      defs = RubyLint::ConstantPath.new(node).resolve(@scope)
-
-      defs.is_a?(ruby_object).should == true
-
-      defs.name.should == 'Bar'
-      defs.type.should == :const
-    end
-
-    example 'resolve a path containing a variable' do
-      node = s(:const, s(:lvar, :example), :Bar)
-      defs = RubyLint::ConstantPath.new(node).resolve(@scope)
-
-      defs.is_a?(ruby_object).should == true
-
-      defs.name.should == 'Bar'
-      defs.type.should == :const
-    end
-
-    example 'resolve a path for a constant assignment' do
-      node = s(:casgn, s(:const, nil, :Foo), :Bar, s(:int, 10))
-      defs = RubyLint::ConstantPath.new(node).resolve(@scope)
-
-      defs.is_a?(ruby_object).should == true
-
-      defs.name.should == 'Bar'
-      defs.type.should == :const
-    end
-  end
-
-  example 'return the root node of a constant path' do
-    node = s(:const, s(:const, nil, :Foo), :Bar)
-    root = RubyLint::ConstantPath.new(node).root_node
-
-    root.should == [:const, 'Foo']
-  end
-
-  example 'generate the name of a constant path' do
-    node = s(:const, s(:const, nil, :Foo), :Bar)
-    name = RubyLint::ConstantPath.new(node).to_s
-
-    name.should == 'Foo::Bar'
-  end
-end

data/spec/ruby-lint/definition/constant_proxy_spec.rb

@@ -1,85 +0,0 @@
-require 'spec_helper'
-
-describe RubyLint::Definition::ConstantProxy do
-  context 'without a registry' do
-    before do
-      @source = ruby_object.new(:name => 'global', :type => :global)
-      @proxy  = RubyLint::Definition::ConstantProxy.new(@source, 'Foo')
-    end
-
-    example 'create a new constant proxy' do
-      @proxy.proxy_source.should == @source
-      @proxy.proxy_name.should   == 'Foo'
-    end
-
-    example 'act like a RubyObject definition' do
-      @proxy.lookup(:const, 'Bar').nil?.should == true
-    end
-
-    example 'cache the definition when it exists' do
-      @proxy.name.nil?.should == true
-      @proxy.type.nil?.should == true
-
-      @proxy.lookup(:const, 'VERSION').nil?.should == true
-
-      @source.define_constant('Foo').define_constant('VERSION')
-
-      @proxy.name.should == 'Foo'
-      @proxy.type.should == :const
-
-      @proxy.lookup(:const, 'VERSION').is_a?(ruby_object).should == true
-    end
-
-    example 'delegating #inspect' do
-      foo = @source.define_constant('Foo')
-
-      @proxy.inspect.should == foo.inspect
-    end
-  end
-
-  context 'with a registry' do
-    before do
-      @source   = ruby_object.new
-      @registry = RubyLint::Definition::Registry.new
-
-      @proxy = RubyLint::Definition::ConstantProxy
-        .new(@source, 'Foo', @registry)
-    end
-
-    example 'do not load anything when the registry is empty' do
-      @proxy.lookup(:const, 'Foo').nil?.should == true
-    end
-
-    example 'autoload a definition when possible' do
-      @registry.register('Foo') { |defs| defs.define_constant('Foo') }
-
-      @proxy.lookup(:const, 'Foo').is_a?(ruby_object).should == true
-    end
-  end
-
-  context 'loading definitions automatically' do
-    before do
-      @source   = ruby_object.new
-      @registry = RubyLint::Definition::Registry.new
-
-      @proxy = RubyLint::Definition::ConstantProxy
-        .new(@source, 'ActionController', @registry)
-    end
-
-    example 'load a definition from the filesystem' do
-      @registry.should_receive(:load).with('ActionController')
-
-      @proxy.lookup(:const, 'Foo')
-    end
-
-    example 'do not load the definition when it is already loaded' do
-      @registry.should_not receive(:load)
-
-      @registry.register('ActionController') do |defs|
-        defs.define_constant('ActionController')
-      end
-
-      @proxy.lookup(:const, 'Foo')
-    end
-  end
-end