ruby-lint 0.0.5 → 0.9.0

data/spec/ruby-lint/node_hash_spec.rb

@@ -0,0 +1,56 @@
+require 'spec_helper'
+
+describe RubyLint::NodeHash do
+  before do
+    @node_hash = RubyLint::NodeHash.new
+    @node      = s(:integer, 10)
+  end
+
+  example 'set and get information associated with a node' do
+    @node_hash[@node] = 10
+
+    @node_hash[@node].should == 10
+  end
+
+  example 'check if a node has been added' do
+    @node_hash.key?(@node).should == false
+
+    @node_hash[@node] = 10
+
+    @node_hash.key?(@node).should == true
+  end
+
+  example 'return all keys' do
+    @node_hash[@node] = 10
+
+    @node_hash.keys.should == [@node.sha1]
+  end
+
+  example 'return all values' do
+    @node_hash[@node] = 10
+
+    @node_hash.values.should == [10]
+  end
+
+  example 'convert to an Array' do
+    @node_hash[@node] = 10
+
+    @node_hash.to_a.should == [[@node.sha1, 10]]
+  end
+
+  example 'merge with another node hash' do
+    other = RubyLint::NodeHash.new(@node.sha1 => 10)
+
+    @node_hash.merge!(other)
+
+    @node_hash.key?(@node).should == true
+  end
+
+  example 'merge with another hash' do
+    other = {@node => 10}
+
+    @node_hash.merge_hash!(other)
+
+    @node_hash.key?(@node).should == true
+  end
+end

data/spec/ruby-lint/runner_spec.rb

@@ -3,6 +3,11 @@ require 'spec_helper'
 describe RubyLint::Runner do
   before do
     @configuration = RubyLint::Configuration.new
+    @cache_dir     = Dir.mktmpdir('ruby-lint')
+  end
+
+  after do
+    FileUtils.rm_rf(@cache_dir)
   end
 
   example 'run analysis on a single file' do
@@ -42,7 +47,7 @@ describe RubyLint::Runner do
 
   example 'associating nodes with external definitions' do
     files  = [fixture_path('associating.rb')]
-    dirs   = [fixture_path('../../lib')]
+    dirs   = [fixture_path('file_scanner/lib/ruby-lint')]
     config = RubyLint::Configuration.new(:directories => dirs)
     runner = RubyLint::Runner.new(config)
     output = runner.analyze(files)

data/spec/ruby-lint/virtual_machine/assignments/variables_spec.rb

@@ -36,8 +36,8 @@ describe RubyLint::VirtualMachine do
     end
 
     example 'assign a value to a constant path' do
-      defs = build_definitions('Kernel::FOO = 10')
-      foo  = defs.lookup(:const, 'Kernel').lookup(:const, 'FOO')
+      defs = build_definitions('module Foo; end; Foo::FOO = 10')
+      foo  = defs.lookup(:const, 'Foo').lookup(:const, 'FOO')
 
       foo.is_a?(ruby_object).should == true
 

data/spec/ruby-lint/virtual_machine/location_spec.rb

@@ -0,0 +1,64 @@
+require 'spec_helper'
+
+describe RubyLint::VirtualMachine do
+  context 'definition location information' do
+    example 'store location information of a class' do
+      code = <<-CODE
+# Example class.
+class Example
+end
+      CODE
+
+      defs = build_definitions(code)
+      obj  = defs.lookup(:const, 'Example')
+
+      obj.line.should   == 2
+      obj.column.should == 0
+      obj.file.should   == '(ruby-lint)'
+    end
+
+    example 'store location information of a module' do
+      code = <<-CODE
+# Example module.
+module Example
+end
+      CODE
+
+      defs = build_definitions(code)
+      obj  = defs.lookup(:const, 'Example')
+
+      obj.line.should   == 2
+      obj.column.should == 0
+      obj.file.should   == '(ruby-lint)'
+    end
+
+    example 'store location information of a method' do
+      code = <<-CODE
+# Example module.
+def example
+end
+      CODE
+
+      defs = build_definitions(code)
+      obj  = defs.lookup(:instance_method, 'example')
+
+      obj.line.should   == 2
+      obj.column.should == 0
+      obj.file.should   == '(ruby-lint)'
+    end
+
+    example 'store location information of a variable' do
+      code = <<-CODE
+# Example assignment
+number = 10
+      CODE
+
+      defs = build_definitions(code)
+      obj  = defs.lookup(:lvar, 'number')
+
+      obj.line.should   == 2
+      obj.column.should == 0
+      obj.file.should   == '(ruby-lint)'
+    end
+  end
+end

data/spec/ruby-lint/virtual_machine/method_call_tracking_spec.rb

@@ -0,0 +1,64 @@
+require 'spec_helper'
+
+describe RubyLint::VirtualMachine do
+  context 'method call tracking' do
+    before :all do
+      code = <<-CODE
+def first_method
+end
+
+def second_method
+end
+
+def third_method
+  first_method
+  second_method
+  String.new
+end
+      CODE
+
+      definitions = build_definitions(code)
+      @first      = definitions.lookup(:instance_method, 'first_method')
+      @second     = definitions.lookup(:instance_method, 'second_method')
+      @third      = definitions.lookup(:instance_method, 'third_method')
+      @string     = definitions.lookup(:const, 'String')
+    end
+
+    example 'track the right amount of method calls' do
+      @third.calls.length.should == 3
+    end
+
+    example 'track call location information' do
+      @third.calls[0].line.should   == 8
+      @third.calls[0].column.should == 2
+
+      @third.calls[1].line.should   == 9
+      @third.calls[1].column.should == 2
+    end
+
+    example 'track the call definitions' do
+      @third.calls[0].definition.should == @first
+      @third.calls[1].definition.should == @second
+    end
+
+    example 'track call location information in inverse direction' do
+      @first.callers[0].line.should   == 8
+      @first.callers[0].column.should == 2
+
+      @second.callers[0].line.should   == 9
+      @second.callers[0].column.should == 2
+    end
+
+    example 'track method calls in inverse direction' do
+      @first.callers[0].definition.should  == @third
+      @second.callers[0].definition.should == @third
+    end
+
+    example 'should not modify read-only definitions' do
+      method = @string.lookup(:method, 'new')
+
+      @third.calls[2].definition.should == method
+      method.callers.empty?.should      == true
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,6 +1,10 @@
 require 'rspec'
 require 'stringio'
 require 'redcard'
+require 'tmpdir'
+require 'fileutils'
+require 'tempfile'
+require 'benchmark'
 
 if ENV['COVERAGE']
   require_relative 'support/simplecov'
@@ -10,6 +14,9 @@ if ENV['COVERALLS']
   require_relative 'support/coveralls'
 end
 
+# Disable caching of RubyLint::Runner for all tests.
+ENV['RUBY_LINT_DISABLE_CACHE'] = '1'
+
 require_relative '../lib/ruby-lint'
 require_relative 'support/building'
 require_relative 'support/parsing'

data/task/generate.rake

@@ -25,7 +25,7 @@ namespace :generate do
     generator = RubyLint::DefinitionGenerator.new(
       Object,
       directory,
-      :ignore    => %w{BasicObject Class GEMSPEC Module},
+      :ignore    => %w{BasicObject Class GEMSPEC Module Kernel},
       :overwrite => !!args[:overwrite]
     )
 

data.tar.gz.asc

@@ -1,17 +1,17 @@
 -----BEGIN PGP SIGNATURE-----
-Version: GnuPG v2.0.21 (GNU/Linux)
+Version: GnuPG v2.0.22 (GNU/Linux)
 
-iQIcBAABAgAGBQJSI1ZdAAoJEPMbESk2SfREfKkP/izxfJRPVk8TzuiXs3lc6lmR
-siLgOCzBwLJK9Pm5iAENQ6D8jyOcWBjOK/NMLgccMyg6KbW54TGFWfZ2n2X3k0QI
-38+HjeDGym92tE1LPOtIHslFKpojRJqUZxFNXhwMtmh1i8Ieqpwq/yA+hyeyFF5Y
-S8i2utYzuWs32bcCSs25wG9ke9rwtO2HiGoDLeIWysYcoAWdSWe0HyAcL9ip3KF+
-1tnKkfRaiFflSWh2o8fmuhc73/yL7vNSTpZtaxDuE7Dsg3Dm/OrIyw/h3ud8LHsa
-uV0SJHMDBg2xJo7JqYJDnazzAqAja/N2IfvywGT1CJfE5qsDdrHyFBPXSuBYRAZy
-dphjD6whpUkuJLGS6vMm7WAFo2xFOaHM7r6ulO5LWZkaiJLEEIJ/34NijKbS7xsm
-L5XH6Zhwoli3rMVJBPjnhT+blOXYSP1+Lw84xbI8OpbQMoGtWtErIOJuHNPrP5YP
-B5x/OnE8KaT4gvyakSjRl/lTeAfTtvI+TFI5Atmye2+Q0eSOhnpxHQaqtNp/HFUl
-ECDmkkVH2lc8L/7H9EqjgNTJiNLtPfPWxDBAzBIVyT2i4WzKKQC4eJjEhZuzAvcX
-SZNcPoUIXf4P2mE4APGILziitM3k+azdiIvHZ2FzU2ruVA5g5nvD86VEPczw5zDL
-LQCMOKMtg15IP5HWWrWB
-=x7n/
+iQIcBAABAgAGBQJSWugYAAoJEPMbESk2SfREyWwQAK79Sx6k+kT89iV25d17afIF
+8VwlL7+gqQV9NiSeK8hgWHpvy7A+sP1cYifsm5R1nxesN350m3QhdZO9HeJbr97e
+MSWPToL23rwCNj/J0YMcKDJpRZgU4qQLbPG/ZXprv2mmkxeWziNZnPMdRD1eVXmE
+rMAIlvCHgzyGpxrNrZ7BFGCrBSZp8jjIEvWoURSQJT7i8JcVE8Pi5KIHuiCIgbyM
+qp3DPM/K5Gln/7uRyoFjphIz+pOT978kTJzKQwg1dEwttvrW0jAD5K3BxiIy5IF9
+exO2Ngd+9e/2efDQ5rXK8JCJ7WtChplmuaj1pGn1J3YHfyzZecqV5ei7t2SfP5Pa
+vMOr1z09V2/IAOi43hWsNm+AUZj+Bg8+AXUmW2bFNxmbPw1J96FJbmRaVJLUcxyf
+TWibwq+z4q7d8wEcoOEPmzqdHQG9mHf+mvLlhDUKZrgcF0asKJwHNpHt6ppxFdpd
+X9AXWGUFvKd4/LVxBJVTYNN4ynBzyMe23thVIbcxbaXj6oM1oV6Jz6y3CSnApQm9
+3hpzLLr4eCak35uHatTWmur4n8XQI8S7W5H6h7VfYTOKdVG5vRa4daYtRMyRSDcz
+J7L7Y5DgL9GZMF59kuSo1gQFYWLQOxg1zt/t1V/YAubX6A3VU4bcll/CWB7D3eR0
+o0VlVQPGIgrfvr4SXlBq
+=ZCLE
 -----END PGP SIGNATURE-----

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ruby-lint
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.9.0
 platform: ruby
 authors:
 - Yorick Peterse
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-01 00:00:00.000000000 Z
+date: 2013-10-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: parser
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
-        version: 2.0.0.pre6
+        version: 2.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
-        version: 2.0.0.pre6
+        version: 2.0.0
 - !ruby/object:Gem::Dependency
   name: slop
   requirement: !ruby/object:Gem::Requirement
@@ -178,6 +178,20 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: gnuplot
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: A linter and static code analysis tool for Ruby.
 email: yorickpeterse@gmail.com
 executables:
@@ -195,12 +209,12 @@ files:
 - MANIFEST
 - README.md
 - Rakefile
+- benchmark/virtual_machine.rb
 - bin/ruby-lint
 - checksum/.gitkeep
 - checksum/ruby-lint-0.0.3.gem.sha512
 - checksum/ruby-lint-0.0.4.gem.sha512
-- debug/memory_usage.rb
-- debug/profile.rb
+- checksum/ruby-lint-0.0.5.gem.sha512
 - doc/.gitkeep
 - doc/DCO.md
 - doc/architecture.md
@@ -222,10 +236,13 @@ files:
 - lib/ruby-lint/analysis/unused_variables.rb
 - lib/ruby-lint/ast/builder.rb
 - lib/ruby-lint/ast/node.rb
+- lib/ruby-lint/cache.rb
+- lib/ruby-lint/cache_entry.rb
 - lib/ruby-lint/cli.rb
 - lib/ruby-lint/cli/analyze.rb
 - lib/ruby-lint/cli/ast.rb
 - lib/ruby-lint/cli/base.rb
+- lib/ruby-lint/cli/plot.rb
 - lib/ruby-lint/configuration.rb
 - lib/ruby-lint/constant_loader.rb
 - lib/ruby-lint/default_names.rb
@@ -377,6 +394,7 @@ files:
 - lib/ruby-lint/definitions/core/weak_ref.rb
 - lib/ruby-lint/definitions/core/yaml.rb
 - lib/ruby-lint/definitions/core/zero_division_error.rb
+- lib/ruby-lint/definitions/core/zlib.rb
 - lib/ruby-lint/definitions/global_variables.rb
 - lib/ruby-lint/definitions/rails.rb
 - lib/ruby-lint/definitions/rails/abstract_controller.rb
@@ -409,7 +427,9 @@ files:
 - lib/ruby-lint/method_call/base.rb
 - lib/ruby-lint/method_call/define_method.rb
 - lib/ruby-lint/method_call/include.rb
+- lib/ruby-lint/method_call_info.rb
 - lib/ruby-lint/nested_stack.rb
+- lib/ruby-lint/node_hash.rb
 - lib/ruby-lint/parser.rb
 - lib/ruby-lint/presenter/json.rb
 - lib/ruby-lint/presenter/text.rb
@@ -421,6 +441,7 @@ files:
 - lib/ruby-lint/variable_predicates.rb
 - lib/ruby-lint/version.rb
 - lib/ruby-lint/virtual_machine.rb
+- profiling/virtual_machine.rb
 - ruby-lint.gemspec
 - ruby-lint.yml
 - spec/fixtures/associating.rb
@@ -431,6 +452,8 @@ files:
 - spec/fixtures/file_scanner/lib/example/recursive/source.rb
 - spec/fixtures/file_scanner/lib/example/recursive/target.rb
 - spec/fixtures/file_scanner/lib/example/user.rb
+- spec/fixtures/file_scanner/lib/ruby-lint/definition/constant_proxy.rb
+- spec/fixtures/file_scanner/lib/ruby-lint/global_scope.rb
 - spec/fixtures/file_scanner/lib/test-dashes/foo.rb
 - spec/fixtures/file_scanner/rails/app/models/example/user.rb
 - spec/fixtures/file_scanner/rails/app/models/user.rb
@@ -449,6 +472,8 @@ files:
 - spec/ruby-lint/analysis/undefined_variables_spec.rb
 - spec/ruby-lint/analysis/unused_variables_spec.rb
 - spec/ruby-lint/ast/node_spec.rb
+- spec/ruby-lint/cache_entry_spec.rb
+- spec/ruby-lint/cache_spec.rb
 - spec/ruby-lint/cli/analyze_spec.rb
 - spec/ruby-lint/cli/ast_spec.rb
 - spec/ruby-lint/configuration_spec.rb
@@ -468,6 +493,7 @@ files:
 - spec/ruby-lint/inspector_spec.rb
 - spec/ruby-lint/iterator_spec.rb
 - spec/ruby-lint/nested_stack_spec.rb
+- spec/ruby-lint/node_hash_spec.rb
 - spec/ruby-lint/parser_spec.rb
 - spec/ruby-lint/presenter/json_spec.rb
 - spec/ruby-lint/presenter/text_spec.rb
@@ -497,6 +523,8 @@ files:
 - spec/ruby-lint/virtual_machine/global_variables_spec.rb
 - spec/ruby-lint/virtual_machine/inherit_kernel_spec.rb
 - spec/ruby-lint/virtual_machine/interpolation_spec.rb
+- spec/ruby-lint/virtual_machine/location_spec.rb
+- spec/ruby-lint/virtual_machine/method_call_tracking_spec.rb
 - spec/ruby-lint/virtual_machine/methods/attr_spec.rb
 - spec/ruby-lint/virtual_machine/methods/define_method_spec.rb
 - spec/ruby-lint/virtual_machine/methods/defining_spec.rb
@@ -524,7 +552,6 @@ files:
 - task/generate.rake
 - task/graphviz.rake
 - task/manifest.rake
-- task/profile.rake
 - task/tag.rake
 - task/test.rake
 - task/todo.rake

metadata.gz.asc

@@ -1,17 +1,17 @@
 -----BEGIN PGP SIGNATURE-----
-Version: GnuPG v2.0.21 (GNU/Linux)
+Version: GnuPG v2.0.22 (GNU/Linux)
 
-iQIcBAABAgAGBQJSI1ZdAAoJEPMbESk2SfRED5gQAKyjQeBLhUlFdrBqItO2uQTt
-tlZoZWSrprz/yHDOI7uVAbPaE/LmwaRQgSy8lbdp0d8M/m9UAdEyOT3COB+LcTtt
-YwN0X4MZ6OckJfQnS+ZTIfgnj+R4tI430Ifq1U11D3uleJDLrjL7/6yvVz5Z6ZqB
-haeFIc0oNSQUxSg6c7So18BdTPDi1dEC4+xRBkFi4F0s+JTIyMrzvPQdPjYDFC3v
-c1smx7DqLIO58DLNVcvPivouFYIrfuX4hubu2TkS4HFCh4Swn6r1gNpFg8A5Z+qV
-t5hzX3PQoN1OK88GCJxVjcoJEtZ3lVAU1G61Pt29LrKZHkrpWXYFsU8LHkpatvFX
-f3q+HbvddrToaR8nTxZAor3Xf4rb9+RwVYAEeePRySXObcfEd+ZTL6ljQ6RsbWqS
-IR5zmKbMitEH/d1qfH8D3oKoe8FVhP6l8Cwi7SGmjlRJ/HrJInTn0sge1kw2jDet
-YXCBGj3jxizmD5nAPhPcDXeul704VmgBsoQF3pNOGD7Rd1sjxKiKpWMfTqZShSPu
-DHXJ2sBCOP3QoWxYuRyw6mMXMpt57RZ/e37KrRHIBsCEIh2Qpgvkv3x443rCSo+X
-zVKIW5hR7T2kkY9tziIe6SmOkPz3JXQCzVuk/o9F9pOnbSORS6kuJc1haQ6vnFXN
-lG3YOvdp/Bomj64R+wSP
-=73rz
+iQIcBAABAgAGBQJSWugYAAoJEPMbESk2SfREyAwP/Rr9jTup5rlAjLVhAYYtXnjO
+pUrPFcsUUwYS/5N/o3p2HlQK3bbXehfQVJ235N4faF7eyS8V07Ak4BDz3aYkYny3
+vFwYbkbPOcptbNRbSyNo+tF6OzXuZ+dSRB9XXbZ/OadTAKDyGfcouGvMLcP01CeF
+1KaQOTzn/Mi2Jg6gXbEoGzoG1AjHLp7r2IVYZ97anGbQD2nprcMVKGZwXP4qTfZH
+qB6Tm2DAogoziMDcqXRbOLFrEJe5LU88SAqR9Ks7x158D40dAbw/9WcoQBgdRZBQ
+97hxbOa5pgOPMI+TMojeDsT1PEASScVxO6OuNsMnZt/30roxv6Lb+G1MpeYk4HV0
+7oaYAvTwKF+WQeFSmPvnkJ3dWaJwq3danWlm4L51YNP4ulGZf86ApoSEura4Kjpa
+BXVIPlzjNESrQ8Lo7wu+h+ZZKGJzFFbHliF3s6wkdRB5WInjSb5ajSkhZIOyLi8b
+p4OxxR7aSiQfbSlrcdmhVCHpMAARG7rQbYvQTp01NbwJmzmmsx6PiYLzLQNrW+v+
+pQMCqkmIlbIKrZfb/YyXkB70/nBiPMLOM3oz/X1t8EyuOJd2Tc6IO5n8n3ilvZdW
+WhljFlEj11DGWeKhrRiWZcTOkU7SMHG9IVbPfhZrXBwsn2MzzBjSB78oFPbjHAFq
+xNmx+U9HE3H7mMpce8r5
+=Zjwg
 -----END PGP SIGNATURE-----