ripper-tags 0.1.2 → 0.1.3

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    MTY5YzNjZGUwOTNkOGMwMzM0NTY1MjE3ZDc4MDkyNjVkNjM2ZDBhNw==
+    Yzc2MzlhZjllZTM4NTk1OWMzNmU3ZjkyNDIxNjVmYTk1NmJiOGJhZA==
   data.tar.gz: !binary |-
-    MWM1M2IzYTk2ODgyZjEwZjQ5ODVlZDYzN2E1Nzc1NGEzOWRmYTI0Ng==
+    ZDlmM2U4YmUyNzZkMGY3OTkxNTRmNWRjMGQxZTk1YjQzN2NiZGZlZA==
 SHA512:
   metadata.gz: !binary |-
-    NDM5MWNkMjk2Y2YxODE2ZTMxODcyY2I3ODcwN2M1YmY5ODFjZGU1ZWI3Mjhj
-    YmIzMzAwMDcyZWM0ZmMxZTA3MTkxYTA0ODk1N2MwY2EwMzUxMTZmZDg1ZDQz
-    YzljYWE2NzEzZDY1YTVlOGQ0MDViNjJmYzBlMzJkNmYzZThkOTU=
+    Zjg2MDJmOTg4Y2UwYTA4MzI0ZDUwNWEzYWY3ZGZkYmE5ZTQ5YTRkN2FkMWZm
+    MmQ1ODdmNmZhNWYwYzA1MzM4ZWY1OWNlMTU2M2VmNjI3MWNiYWVlZGJhY2M3
+    MTEzNGQ3NDQ1ZmUyOGVlNjRlM2I0Njc4YzdjN2M3ZjNiOWQ3YjU=
   data.tar.gz: !binary |-
-    ZWU2ZDQ5Nzc0MzAyNjNjYjI0YjU5MTczMjRkMDUwZTJiZWRhN2Y2Y2YwYmQz
-    Yzg5MDg3MDJkOTMzNzIyMmI2ZjQzNzliYmIwMzIyNzliNzYzMWQxNjFlZTMz
-    MDAxNzVlZjk2YmRjMDlhYTZiYTAzMDRkYjViNjFjYTNlNDgwZGI=
+    NGY3N2YxMzU0NDM2MmIyNjRiZGU4MmUwNThmYzE2YjVjMWFlMjdmMzMwMTkx
+    NzkxZWViZWZjMzRhNmRjZDYxYjQ2ZWNhNGI2MWI1ZjNhZTlmODBjN2VmYjcy
+    ZjNkYjU0MjE4MTlhZmI5Nzc1ZDBkZTcwNDYxYTA4NmFhZDI5NTc=

data/.gitignore

@@ -1,3 +1,5 @@
 tags
 TAGS
 ripper-tags-*gem
+*.swp
+Gemfile.lock

data/lib/ripper-tags/parser.rb

@@ -144,22 +144,26 @@ class Parser < Ripper
           gen
         end
       when "has_many", "has_and_belongs_to_many"
-        a = args[1][0]
+        a = args[1][0].to_s
         kind = name.to_sym
         gen = []
-        gen << [:rails_def, kind, a, line]
-        gen << [:rails_def, kind, "#{a}=", line]
-        if (sing = a.chomp('s')) != a
-          # poor man's singularize
-          gen << [:rails_def, kind, "#{sing}_ids", line]
-          gen << [:rails_def, kind, "#{sing}_ids=", line]
+        unless a.is_a?(Enumerable)
+          gen << [:rails_def, kind, a, line]
+          gen << [:rails_def, kind, "#{a}=", line]
+          if a.respond_to?(:chomp) && (sing = a.chomp('s')) != a
+            # poor man's singularize
+            gen << [:rails_def, kind, "#{sing}_ids", line]
+            gen << [:rails_def, kind, "#{sing}_ids=", line]
+          end
         end
         gen
       when "belongs_to", "has_one"
         a = args[1][0]
-        kind = name.to_sym
-        %W[ #{a} #{a}= build_#{a} create_#{a} create_#{a}! ].inject([]) do |gen, ident|
-          gen << [:rails_def, kind, ident, line]
+        unless a.is_a?(Enumerable)
+          kind = name.to_sym
+          %W[ #{a} #{a}= build_#{a} create_#{a} create_#{a}! ].inject([]) do |gen, ident|
+            gen << [:rails_def, kind, ident, line]
+          end
         end
       end
     else
@@ -244,7 +248,8 @@ end
         sexp.each{ |child| process(child) }
       when Symbol
         name, *args = sexp
-        __send__("on_#{name}", *args)
+        handler = "on_#{name}"
+        __send__(handler, *args) if respond_to?(handler)
       when String, nil
         # nothing
       end
@@ -283,7 +288,7 @@ end
       on_module_or_class(:module, name, nil, body)
     end
 
-    def on_class(name, superclass, body)
+    def on_class(name, superclass, body = nil)
       on_module_or_class(:class, name, superclass, body)
     end
 

data/ripper-tags.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name = 'ripper-tags'
-  s.version = '0.1.2'
+  s.version = '0.1.3'
 
   s.summary = 'ctags generator for ruby code'
   s.description = 'fast, accurate ctags generator for ruby source code using Ripper'

data/test/test_data_reader.rb

@@ -41,7 +41,7 @@ class DataReaderTest < Test::Unit::TestCase
       very/deep/script.rb
       very/inter.rb
     ]
-    assert_equal expected, files
+    assert_equal Set.new(expected), Set.new(files)
   end
 
   def test_file_finder_no_exclude
@@ -61,7 +61,7 @@ class DataReaderTest < Test::Unit::TestCase
       encoding.rb
       very/inter.rb
     ]
-    assert_equal expected, files
+    assert_equal Set.new(expected), Set.new(files)
   end
 
   def with_default_encoding(name)

data/test/test_ripper_tags.rb

@@ -295,4 +295,48 @@ class TagRipperTest < Test::Unit::TestCase
     assert_equal '2: scope C.red',  inspect(tags[1])
     assert_equal '3: scope C.red',  inspect(tags[2])
   end
+
+  def test_extract_from_erb
+    tags = extract(<<-EOC)
+      class NavigationTest < ActionDispatch::IntegrationTest
+      <% unless options[:skip_active_record] -%>
+        fixtures :all
+      <% end -%>
+
+        # test "the truth" do
+        #   assert true
+        # end
+      end
+    EOC
+
+    assert_equal 1, tags.size
+    assert_equal 'NavigationTest', tags[0][:name]
+  end
+
+  def test_extract_with_keyword_variables
+    tags = extract(<<-EOC)
+      class Foo
+        @public
+        @protected
+        @private
+      end
+    EOC
+
+    assert_equal 1, tags.size
+    assert_equal 'Foo', tags[0][:name]
+  end
+
+  def test_extract_associations_with_class_name
+    tags = extract(<<-EOC)
+      class Foo
+        belongs_to Bar
+        has_one Bar
+        has_and_belongs_to_many Bar
+        has_many Bar
+      end
+    EOC
+
+    assert_equal 1, tags.size
+    assert_equal 'Foo', tags[0][:name]
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ripper-tags
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Aman Gupta
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-11-28 00:00:00.000000000 Z
+date: 2013-12-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: yajl-ruby