code_analyzer 0.5.0 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6c252e6214745ff2bcf507702a49553d0ca90c75a12614ee3e5c197c1994656c
-  data.tar.gz: 85bb0e3b8ea846132ba4a159f4477749751cb54a94611a7402d241a2e296f1c6
+  metadata.gz: 792496737344902c984457c4a153f5e7e5717e7f9568f130cdc039c1779e9627
+  data.tar.gz: 8b15a24c9ede966353b1100328fe1d5dcc14937fc6d2a4401a455cfe42176d27
 SHA512:
-  metadata.gz: 73fc0d1d771feafb646ff218837ae19e8de9556acd25b4fb3a3bad93b76899d950037145c603b1b5e48a38815b1be086102f65c3e77ad2bc2898931858988e6c
-  data.tar.gz: 3fd7c957fc9c2ceeb6714cf4a6a4f1db7b00f7ea6d4a9e73553b460c59f0bd42a4f5d4538e7bf00bc2f39d441f29bed8891a3d200a65b0653830d30c735b7dc0
+  metadata.gz: 2bd9cd8d09ffaab831f417207e0003082408ddb61e8a513884a0d137adb567c1c6f3aed5f68d2460d58a5799ef56a514cb49af4d1c5f2b92b017500193709e54
+  data.tar.gz: dd4bf2b587ca333c7eeabcd088586eeebc421c9e41931d1637a7dd0bb3f1b403959b95c0cc697eb81cd6d818e198f4648b1fc8e1d585c8896600f528ec2511fe

data/lib/code_analyzer/sexp.rb

@@ -844,7 +844,7 @@ class Sexp
     case sexp_type
     when :string_literal, :xstring_literal, :string_content, :const_ref,
          :symbol_literal, :symbol, :args_add_block, :var_ref, :vcall, :var_field,
-         :@ident, :@tstring_content, :@const, :@ivar, :@kw, :@gvar, :@cvar
+         :@ident, :@tstring_content, :@const, :@ivar, :@kw, :@gvar, :@cvar, :@period
       self[1].to_s
     when :string_add
       s(:string_content) == self[1] ? self[2].to_s : self[1].to_s

data/lib/code_analyzer/version.rb

@@ -1,4 +1,4 @@
 # encoding: utf-8
 module CodeAnalyzer
-  VERSION = '0.5.0'
+  VERSION = '0.5.1'
 end

data/spec/code_analyzer/sexp_spec.rb

@@ -87,25 +87,14 @@ describe Sexp do
       @node.grep_nodes(sexp_type: :call, receiver: 'current_user') do |node|
         nodes << node
       end
-      if RUBY_VERSION == '1.9.2'
-        expect(nodes).to eq [
-             s(
-               :call,
-               s(:var_ref, s(:@ident, 'current_user', s(2, 8))),
-               s(:@period, ".", s(2, 20)),
-               s(:@ident, 'posts', s(2, 21))
-             )
-           ]
-      else
-        expect(nodes).to eq [
-             s(
-               :call,
-               s(:vcall, s(:@ident, 'current_user', s(2, 8))),
-               s(:@period, ".", s(2, 20)),
-               s(:@ident, 'posts', s(2, 21))
-             )
-           ]
-      end
+      expect(nodes).to eq [
+           s(
+             :call,
+             s(:vcall, s(:@ident, 'current_user', s(2, 8))),
+             s(:@period, ".", s(2, 20)),
+             s(:@ident, 'posts', s(2, 21))
+           )
+         ]
     end
 
     it 'should get the call nodes with different messages' do
@@ -113,65 +102,33 @@ describe Sexp do
       @node.grep_nodes(sexp_type: :call, message: %w[posts find]) do |node|
         nodes << node
       end
-      if RUBY_VERSION == '1.9.2'
-        expect(nodes).to eq [
-             s(
-               :call,
-               s(
-                 :call,
-                 s(:var_ref, s(:@ident, 'current_user', s(2, 8))),
-                 s(:@period, ".", s(2, 20)),
-                 s(:@ident, 'posts', s(2, 21))
-               ),
-               s(:@period, ".", s(2, 26)),
-               s(:@ident, 'find', s(2, 27))
-             ),
-             s(
-               :call,
-               s(:var_ref, s(:@ident, 'current_user', s(2, 8))),
-               s(:@period, ".", s(2, 20)),
-               s(:@ident, 'posts', s(2, 21))
-             )
-           ]
-      else
-        expect(nodes).to eq [
-             s(
-               :call,
-               s(
-                 :call,
-                 s(:vcall, s(:@ident, 'current_user', s(2, 8))),
-                 s(:@period, ".", s(2, 20)),
-                 s(:@ident, 'posts', s(2, 21))
-               ),
-               s(:@period, ".", s(2, 26)),
-               s(:@ident, 'find', s(2, 27))
-             ),
+      expect(nodes).to eq [
+           s(
+             :call,
              s(
                :call,
                s(:vcall, s(:@ident, 'current_user', s(2, 8))),
                s(:@period, ".", s(2, 20)),
                s(:@ident, 'posts', s(2, 21))
-             )
-           ]
-      end
+             ),
+             s(:@period, ".", s(2, 26)),
+             s(:@ident, 'find', s(2, 27))
+           ),
+           s(
+             :call,
+             s(:vcall, s(:@ident, 'current_user', s(2, 8))),
+             s(:@period, ".", s(2, 20)),
+             s(:@ident, 'posts', s(2, 21))
+           )
+         ]
     end
 
-    if RUBY_VERSION == '1.9.2'
-      it 'should get the var_ref node with to_s' do
-        nodes = []
-        @node.grep_nodes(sexp_type: :var_ref, to_s: 'current_user') do |node|
-          nodes << node
-        end
-        expect(nodes).to eq [s(:var_ref, s(:@ident, 'current_user', s(2, 8)))]
-      end
-    else
-      it 'should get the vcall node with to_s' do
-        nodes = []
-        @node.grep_nodes(sexp_type: :vcall, to_s: 'current_user') do |node|
-          nodes << node
-        end
-        expect(nodes).to eq [s(:vcall, s(:@ident, 'current_user', s(2, 8)))]
+    it 'should get the vcall node with to_s' do
+      nodes = []
+      @node.grep_nodes(sexp_type: :vcall, to_s: 'current_user') do |node|
+        nodes << node
       end
+      expect(nodes).to eq [s(:vcall, s(:@ident, 'current_user', s(2, 8)))]
     end
   end
 
@@ -187,21 +144,12 @@ describe Sexp do
 
     it 'should get first node with empty argument' do
       node = @node.grep_node(sexp_type: :call, receiver: 'current_user')
-      if RUBY_VERSION == '1.9.2'
-        expect(node).to eq s(
-             :call,
-             s(:var_ref, s(:@ident, 'current_user', s(2, 8))),
-             s(:@period, ".", s(2, 20)),
-             s(:@ident, 'posts', s(2, 21))
-           )
-      else
-        expect(node).to eq s(
-             :call,
-             s(:vcall, s(:@ident, 'current_user', s(2, 8))),
-             s(:@period, ".", s(2, 20)),
-             s(:@ident, 'posts', s(2, 21))
-           )
-      end
+      expect(node).to eq s(
+           :call,
+           s(:vcall, s(:@ident, 'current_user', s(2, 8))),
+           s(:@period, ".", s(2, 20)),
+           s(:@ident, 'posts', s(2, 21))
+         )
     end
   end
 
@@ -415,43 +363,23 @@ describe Sexp do
       node =
         parse_content('options_for_select(Account.get_business current_user)')
           .grep_node(sexp_type: :args_add)
-      if RUBY_VERSION == '1.9.2'
-        expect(node.all).to eq [
+      expect(node.all).to eq [
+           s(
+             :command_call,
+             s(:var_ref, s(:@const, 'Account', s(1, 19))),
+             s(:@period, ".", s(1, 26)),
+             s(:@ident, 'get_business', s(1, 27)),
              s(
-               :command_call,
-               s(:var_ref, s(:@const, 'Account', s(1, 19))),
-               :".",
-               s(:@ident, 'get_business', s(1, 27)),
+               :args_add_block,
                s(
-                 :args_add_block,
-                 s(
-                   :args_add,
-                   s(:args_new),
-                   s(:var_ref, s(:@ident, 'current_user', s(1, 40)))
-                 ),
-                 false
-               )
-             )
-           ]
-      else
-        expect(node.all).to eq [
-             s(
-               :command_call,
-               s(:var_ref, s(:@const, 'Account', s(1, 19))),
-               s(:@period, ".", s(1, 26)),
-               s(:@ident, 'get_business', s(1, 27)),
-               s(
-                 :args_add_block,
-                 s(
-                   :args_add,
-                   s(:args_new),
-                   s(:vcall, s(:@ident, 'current_user', s(1, 40)))
-                 ),
-                 false
-               )
+                 :args_add,
+                 s(:args_new),
+                 s(:vcall, s(:@ident, 'current_user', s(1, 40)))
+               ),
+               false
              )
-           ]
-      end
+           )
+         ]
     end
 
     it 'no error for args_add_star' do
@@ -885,6 +813,11 @@ describe Sexp do
       expect(node.to_s).to eq '@user'
     end
 
+    it 'should get to_s for period' do
+      node = parse_content('@user.name').grep_node(sexp_type: :@period)
+      expect(node.to_s).to eq '.'
+    end
+
     it 'should get to_s for class with module' do
       node =
         parse_content('ActiveRecord::Base').grep_node(

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: code_analyzer
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.5.1
 platform: ruby
 authors:
 - Richard Huang