synvert-core 0.29.0 → 0.33.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 31ee6b89cd48a4accb2dc71fdec2e487c534e8d6eb4a9bd6050817cd88f7ba3c
-  data.tar.gz: cd4c005e0f7a21e52e984e5279a45d4325d341b92d67feedbbc455ddc062e0fc
+  metadata.gz: a1f74df180f3b7583a026a0549fe82d57a075fae69990b2dd4a20d22f391fcc5
+  data.tar.gz: 0a2159eb88c03c812b3a3a5a2d45e6cdfa79f899c70e820d538a90a6aa17cf9b
 SHA512:
-  metadata.gz: 4d1e8642e682a155eedb03c6cdbabca55f53f22c258d4774da7200877159ec18c0a7d568823c5c379fd93810d0baf945a2bc139f8444423855c6279241acb0bc
-  data.tar.gz: 73a369f23d1add90b21ce578faf37a3136a6ffcb384f47fa5b3881fd4241af4b087b19fb0d854143ed1e3d9b76f350fc41b2415cddc29671cde3afe0e46d4709
+  metadata.gz: bd505776d1d2f97f5acae2def15b808ee05536193f076e1d3ce683bdbf8ada56e89acf261db5247fc535bdf3444fb7ee8c58f2b142427ec20ac1325abdc31940
+  data.tar.gz: 971fcf29e04c1aebc6e32d5c434adc0ec12e18a6732ff415970e08318b4af885598b4925567c691cd7ce1d8dbd7b7802bed4426cf71ae6dfb21f5546d18c2c73

data/CHANGELOG.md

@@ -1,5 +1,22 @@
 # CHANGELOG
 
+## 0.33.0 (2021-05-10)
+
+* Add `body` for `class` node
+
+## 0.32.0 (2021-05-07)
+
+* Remove `ArgumentsNode`
+
+## 0.31.0 (2021-04-27)
+
+* Add `in` and `not_in` rules
+
+## 0.30.0 (2021-04-26)
+
+* `goto_node` accepts multiple child node names
+* Match any_value
+
 ## 0.29.0 (2021-04-25)
 
 * Make `child_name_range` support [:block, :pipe]

data/lib/synvert/core/node_ext.rb

@@ -1,29 +1,6 @@
 # frozen_string_literal: true
 
 module Parser::AST
-  # ArgumentsNode allows to handle all args as one node or handle all args as an array.
-  class ArgumentsNode
-    # Initialize
-    #
-    # @param node [Parser::AST::Node] args node.
-    def initialize(node)
-      @node = node
-    end
-
-    # If args node responds method itself, call method on args node.
-    # If args children (array) responds method,  call method on args children.
-    # Otherwise raise method missing error.
-    def method_missing(meth, *args, &block)
-      if @node.respond_to?(meth)
-        @node.send meth, *args, &block
-      elsif @node.children.respond_to?(meth)
-        @node.children.send meth, *args, &block
-      else
-        super
-      end
-    end
-  end
-
   # Parser::AST::Node monkey patch.
   class Node
     # Get name node of :class, :module, :const, :mlhs, :def and :defs node.
@@ -101,9 +78,9 @@ module Parser::AST
     def arguments
       case type
       when :def, :block
-        ArgumentsNode.new(children[1])
+        children[1]
       when :defs
-        ArgumentsNode.new(children[2])
+        children[2]
       when :send
         children[2..-1]
       when :defined?
@@ -133,7 +110,7 @@ module Parser::AST
       case type
       when :begin
         children
-      when :def, :block
+      when :def, :block, :class
         return [] if children[2].nil?
 
         :begin == children[2].type ? children[2].body : children[2..-1]
@@ -285,17 +262,23 @@ module Parser::AST
     # Current node is s(:hash, s(:pair, s(:sym, :number), s(:int, 10)))
     # node.number_value is 10
     def method_missing(method_name, *args, &block)
-      if :hash == type && method_name.to_s.include?('_value')
+      if :args == type && children.respond_to?(method_name)
+        return children.send(method_name, *args, &block)
+      elsif :hash == type && method_name.to_s.include?('_value')
         key = method_name.to_s.sub('_value', '')
         return hash_value(key.to_sym)&.to_value if key?(key.to_sym)
         return hash_value(key.to_s)&.to_value if key?(key.to_s)
+
+        return nil
       end
 
       super
     end
 
     def respond_to_missing?(method_name, *args)
-      if :hash == type && method_name.to_s.include?('_value')
+      if :args == type && children.respond_to?(method_name)
+        return true
+      elsif :hash == type && method_name.to_s.include?('_value')
         key = method_name.to_s.sub('_value', '')
         return true if key?(key.to_sym) || key?(key.to_s)
       end
@@ -348,6 +331,8 @@ module Parser::AST
       case [type, child_name]
       when %i[block pipe]
         Parser::Source::Range.new('(string)', arguments.loc.expression.begin_pos, arguments.loc.expression.end_pos)
+      when %i[block arguments], %i[def arguments], %i[defs arguments]
+        Parser::Source::Range.new('(string)', arguments.first.loc.expression.begin_pos, arguments.last.loc.expression.end_pos)
       when %i[class name]
         loc.name
       when %i[def name]
@@ -357,7 +342,7 @@ module Parser::AST
       when %i[defs dot]
         loc.operator
       when %i[defs self]
-        Parser::Source::Range.new('(string)', loc.operator.begin_pos - 4, loc.operator.begin_pos)
+        Parser::Source::Range.new('(string)', loc.operator.begin_pos - 'self'.length, loc.operator.begin_pos)
       when %i[send dot]
         loc.dot
       when %i[send message]
@@ -426,6 +411,14 @@ module Parser::AST
           actual = actual_value(self, multi_keys[0...-1])
           expected = expected_value(rules, multi_keys)
           !match_value?(actual, expected)
+        when :in
+          actual = actual_value(self, multi_keys[0...-1])
+          expected_values = expected_value(rules, multi_keys)
+          expected_values.any? { |expected| match_value?(actual, expected) }
+        when :not_in
+          actual = actual_value(self, multi_keys[0...-1])
+          expected_values = expected_value(rules, multi_keys)
+          expected_values.all? { |expected| !match_value?(actual, expected) }
         else
           actual = actual_value(self, multi_keys)
           expected = expected_value(rules, multi_keys)
@@ -448,8 +441,12 @@ module Parser::AST
           evaluated = instance_eval old_code
           case evaluated
           when Parser::AST::Node
-            evaluated.loc.expression.source
-          when Array, ArgumentsNode
+            if evaluated.type == :args
+              evaluated.loc.expression.source[1...-1]
+            else
+              evaluated.loc.expression.source
+            end
+          when Array
             if evaluated.size > 0
               file_source = evaluated.first.loc.expression.source_buffer.source
               source = file_source[evaluated.first.loc.expression.begin_pos...evaluated.last.loc.expression.end_pos]
@@ -465,7 +462,7 @@ module Parser::AST
                 source
               end
             end
-          when String, Symbol
+          when String, Symbol, Integer, Float
             evaluated
           when NilClass
             'nil'
@@ -525,6 +522,8 @@ module Parser::AST
         :false == actual.type
       when Parser::AST::Node
         actual == expected
+      when Synvert::Core::Rewriter::AnyValue
+        !actual.nil?
       else
         raise Synvert::Core::MethodNotSupported, "#{expected.class} is not handled for match_value?"
       end

data/lib/synvert/core/rewriter.rb

@@ -44,6 +44,8 @@ module Synvert::Core
     autoload :RubyVersion, 'synvert/core/rewriter/ruby_version'
     autoload :GemSpec, 'synvert/core/rewriter/gem_spec'
 
+    autoload :AnyValue, 'synvert/core/rewriter/any_value'
+
     class << self
       # Execute the temporary rewriter without group and name.
       #

data/lib/synvert/core/rewriter/any_value.rb

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+module Synvert::Core
+  class Rewriter::AnyValue
+  end
+end

data/lib/synvert/core/rewriter/instance.rb

@@ -181,10 +181,10 @@ module Synvert::Core
     # Parse goto_node dsl, it creates a [Synvert::Core::Rewriter::GotoScope] to go to a child node,
     # then continue operating on the child node.
     #
-    # @param child_node_name [String] the name of the child node.
+    # @param *child_node_names [Array] the name of the child nodes.
     # @param block [Block] block code to continue operating on the matching nodes.
-    def goto_node(child_node_name, &block)
-      Rewriter::GotoScope.new(self, child_node_name, &block).process
+    def goto_node(*child_node_names, &block)
+      Rewriter::GotoScope.new(self, *child_node_names, &block).process
     end
 
     # Parse if_exist_node dsl, it creates a [Synvert::Core::Rewriter::IfExistCondition] to check
@@ -285,6 +285,11 @@ module Synvert::Core
       @rewriter.add_warning Rewriter::Warning.new(self, message)
     end
 
+    # Any value but nil.
+    def any_value
+      Rewriter::AnyValue.new
+    end
+
     private
 
     # It changes source code from bottom to top, and it can change source code twice at the same time,
@@ -297,7 +302,7 @@ module Synvert::Core
 
       begin_pos = @actions[i].begin_pos
       while j > -1
-        if begin_pos <= @actions[j].end_pos
+        if begin_pos < @actions[j].end_pos
           conflict_actions << @actions.delete_at(j)
         else
           i = j

data/lib/synvert/core/rewriter/scope/goto_scope.rb

@@ -6,11 +6,11 @@ module Synvert::Core
     # Initialize a scope
     #
     # @param instance [Synvert::Core::Rewriter::Instance]
-    # @param child_node_name [String]
+    # @param *child_node_names [Array]
     # @param block [Block]
-    def initialize(instance, child_node_name, &block)
+    def initialize(instance, *child_node_names, &block)
       @instance = instance
-      @child_node_name = child_node_name
+      @child_node_names = child_node_names
       @block = block
     end
 
@@ -19,7 +19,10 @@ module Synvert::Core
       current_node = @instance.current_node
       return unless current_node
 
-      child_node = @child_node_name.is_a?(Parser::AST::Node) ? @child_node_name : current_node.send(@child_node_name)
+      child_node = current_node
+      @child_node_names.each do |child_node_name|
+        child_node = child_node_name.is_a?(Parser::AST::Node) ? child_node_name : child_node.send(child_node_name)
+      end
       @instance.process_with_other_node child_node do
         @instance.instance_eval(&@block)
       end

data/lib/synvert/core/version.rb

@@ -2,6 +2,6 @@
 
 module Synvert
   module Core
-    VERSION = '0.29.0'
+    VERSION = '0.33.1'
   end
 end

data/spec/synvert/core/node_ext_spec.rb

@@ -104,17 +104,17 @@ describe Parser::AST::Node do
   describe '#arguments' do
     it 'gets for def node' do
       node = parse('def test(foo, bar); foo + bar; end')
-      expect(node.arguments.map { |argument| argument.to_source }).to eq %w[foo bar]
+      expect(node.arguments.type).to eq :args
     end
 
     it 'gets for defs node' do
       node = parse('def self.test(foo, bar); foo + bar; end')
-      expect(node.arguments.map { |argument| argument.to_source }).to eq %w[foo bar]
+      expect(node.arguments.type).to eq :args
     end
 
     it 'gets for block node' do
       node = parse('RSpec.configure do |config|; end')
-      expect(node.arguments.map { |argument| argument.to_source }).to eq ['config']
+      expect(node.arguments.type).to eq :args
     end
 
     it 'gets for send node' do
@@ -146,6 +146,21 @@ describe Parser::AST::Node do
       expect(node.body).to eq [parse('include EmailSpec::Helpers'), parse('include EmailSpec::Matchers')]
     end
 
+    it 'gets empty for class node' do
+      node = parse('class User; end')
+      expect(node.body).to be_empty
+    end
+
+    it 'gets one line for class node' do
+      node = parse('class User; attr_accessor :email; end')
+      expect(node.body).to eq [parse('attr_accessor :email')]
+    end
+
+    it 'gets one line for class node' do
+      node = parse('class User; attr_accessor :email; attr_accessor :username; end')
+      expect(node.body).to eq [parse('attr_accessor :email'), parse('attr_accessor :username')]
+    end
+
     it 'gets for begin node' do
       node = parse('foo; bar')
       expect(node.body).to eq [parse('foo'), parse('bar')]
@@ -335,11 +350,13 @@ describe Parser::AST::Node do
 
   describe 'key value by method_missing' do
     it 'gets for key value' do
-      node = parse("{:foo => :bar}")
+      node = parse('{:foo => :bar}')
       expect(node.foo_value).to eq :bar
 
       node = parse("{'foo' => 'bar'}")
       expect(node.foo_value).to eq 'bar'
+
+      expect(node.bar_value).to be_nil
     end
   end
 
@@ -402,6 +419,18 @@ describe Parser::AST::Node do
       node = parse(source)
       expect(node).not_to be_match(type: 'class', name: { not: 'Synvert' })
     end
+
+    it 'matches in' do
+      source = 'FactoryBot.create(:user)'
+      node = parse(source)
+      expect(node).to be_match(type: 'send', message: { in: %i[create build] })
+    end
+
+    it 'matches not_in' do
+      source = 'FactoryBot.create(:user)'
+      node = parse(source)
+      expect(node).not_to be_match(type: 'send', message: { not_in: %i[create build] })
+    end
   end
 
   describe '#child_node_range' do
@@ -552,7 +581,7 @@ describe Parser::AST::Node do
       expect(node.rewritten_source('{{name}}')).to eq 'Synvert'
     end
 
-    it 'rewrites for ArgumentsNode' do
+    it 'rewrites for arguments' do
       source = 'test { |a, b| }'
       node = parse(source)
       expect(node.rewritten_source('{{arguments}}')).to eq 'a, b'

data/spec/synvert/core/rewriter/instance_spec.rb

@@ -54,9 +54,9 @@ module Synvert::Core
     it 'parses goto_node' do
       scope = double
       block = proc {}
-      expect(Rewriter::GotoScope).to receive(:new).with(instance, :caller, &block).and_return(scope)
+      expect(Rewriter::GotoScope).to receive(:new).with(instance, :caller, :receiver, &block).and_return(scope)
       expect(scope).to receive(:process)
-      instance.goto_node(:caller, &block)
+      instance.goto_node(:caller, :receiver, &block)
     end
 
     it 'parses if_exist_node' do

data/spec/synvert/core/rewriter/scope/goto_scope_spec.rb

@@ -8,12 +8,11 @@ module Synvert::Core
       rewriter = Rewriter.new('foo', 'bar')
       Rewriter::Instance.new(rewriter, 'file pattern')
     }
-    let(:source) {
-      '
-Factory.define :user do |user|
-end
-    '
-    }
+    let(:source) { <<~EOS }
+      Factory.define :user do |user|
+      end
+    EOS
+
     let(:node) { Parser::CurrentRuby.parse(source) }
     before do
       Rewriter::Instance.reset
@@ -25,13 +24,13 @@ end
         run = false
         type_in_scope = nil
         scope =
-          Rewriter::GotoScope.new instance, :caller do
+          Rewriter::GotoScope.new instance, :caller, :receiver do
             run = true
             type_in_scope = node.type
           end
         scope.process
         expect(run).to be_truthy
-        expect(type_in_scope).to eq :send
+        expect(type_in_scope).to eq :const
         expect(instance.current_node.type).to eq :block
       end
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: synvert-core
 version: !ruby/object:Gem::Version
-  version: 0.29.0
+  version: 0.33.1
 platform: ruby
 authors:
 - Richard Huang
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-04-25 00:00:00.000000000 Z
+date: 2021-05-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -154,6 +154,7 @@ files:
 - lib/synvert/core/rewriter/action/replace_action.rb
 - lib/synvert/core/rewriter/action/replace_erb_stmt_with_expr_action.rb
 - lib/synvert/core/rewriter/action/replace_with_action.rb
+- lib/synvert/core/rewriter/any_value.rb
 - lib/synvert/core/rewriter/condition.rb
 - lib/synvert/core/rewriter/condition/if_exist_condition.rb
 - lib/synvert/core/rewriter/condition/if_only_exist_condition.rb