synvert-core 1.4.0 → 1.6.0

data/spec/synvert/core/rewriter/action/replace_erb_stmt_with_expr_action_spec.rb

@@ -3,6 +3,26 @@
 require 'spec_helper'
 
 module Synvert::Core
-  describe Rewriter::ReplaceErbStmtWithExprAction do
+  RSpec.describe Rewriter::ReplaceErbStmtWithExprAction do
+    context 'replace with single line' do
+      subject {
+        source = "<% form_for post do |f| %>\n<% end %>"
+        source = Engine::Erb.encode(source)
+        node = Parser::CurrentRuby.parse(source).children[1]
+        described_class.new(node).process
+      }
+
+      it 'gets start' do
+        expect(subject.start).to eq '@output_buffer = output_buffer || ActionView::OutputBuffer.new;'.length
+      end
+
+      it 'gets end' do
+        expect(subject.end).to eq '@output_buffer = output_buffer || ActionView::OutputBuffer.new;;   ; form_for post do |f| ;   ;'.length
+      end
+
+      it 'gets new_code' do
+        expect(subject.new_code).to eq '@output_buffer.append=  form_for post do |f| ;  ;'
+      end
+    end
   end
 end

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

@@ -4,8 +4,6 @@ require 'spec_helper'
 
 module Synvert::Core
   describe Rewriter::Instance do
-    before { Rewriter::Instance.reset }
-
     let(:instance) {
       rewriter = Rewriter.new('foo', 'bar')
       Rewriter::Instance.new(rewriter, ['file pattern'])
@@ -14,16 +12,27 @@ module Synvert::Core
     it 'parses find_node' do
       scope = double
       block = proc {}
-      expect(Rewriter::QueryScope).to receive(:new).with(instance, '.send[message=create]', &block).and_return(scope)
+      expect(Rewriter::QueryScope).to receive(:new).with(instance, '.send[message=create]', {}, &block).and_return(scope)
       expect(scope).to receive(:process)
       instance.find_node('.send[message=create]', &block)
     end
 
+    it 'raises ParseError when parsing invalid nql' do
+      block = proc {}
+      expect {
+        instance.find_node('synvert', &block)
+      }.to raise_error(NodeQuery::Compiler::ParseError)
+
+      expect {
+        instance.find_node('.block <caller.arguments> .hash', &block)
+      }.to raise_error(NodeQuery::Compiler::ParseError)
+    end
+
     it 'parses within_node' do
       scope = double
       block = proc {}
       expect(Rewriter::WithinScope).to receive(:new)
-        .with(instance, { type: 'send', message: 'create' }, { stop_when_match: false, direct: false }, &block)
+        .with(instance, { type: 'send', message: 'create' }, {}, &block)
         .and_return(scope)
       expect(scope).to receive(:process)
       instance.within_node(type: 'send', message: 'create', &block)
@@ -33,30 +42,20 @@ module Synvert::Core
       scope = double
       block = proc {}
       expect(Rewriter::WithinScope).to receive(:new)
-        .with(instance, { type: 'send', message: 'create' }, { stop_when_match: false, direct: false }, &block)
+        .with(instance, { type: 'send', message: 'create' }, {}, &block)
         .and_return(scope)
       expect(scope).to receive(:process)
       instance.with_node(type: 'send', message: 'create', &block)
     end
 
-    it 'parses within_node with stop_when_match true' do
-      scope = double
-      block = proc {}
-      expect(Rewriter::WithinScope).to receive(:new)
-        .with(instance, { type: 'send', message: 'create' }, { stop_when_match: true, direct: false }, &block)
-        .and_return(scope)
-      expect(scope).to receive(:process)
-      instance.within_node({ type: 'send', message: 'create' }, { stop_when_match: true }, &block)
-    end
-
-    it 'parses within_node with direct true' do
+    it 'parses within_node with stop_at_first_match true' do
       scope = double
       block = proc {}
       expect(Rewriter::WithinScope).to receive(:new)
-        .with(instance, { type: 'send', message: 'create' }, { stop_when_match: false, direct: true }, &block)
+        .with(instance, { type: 'send', message: 'create' }, { stop_at_first_match: true }, &block)
         .and_return(scope)
       expect(scope).to receive(:process)
-      instance.within_node({ type: 'send', message: 'create' }, { direct: true }, &block)
+      instance.within_node({ type: 'send', message: 'create' }, { stop_at_first_match: true }, &block)
     end
 
     it 'parses goto_node' do
@@ -98,97 +97,83 @@ module Synvert::Core
     end
 
     it 'parses append' do
-      action = double
-      expect(Rewriter::AppendAction).to receive(:new).with(
-        instance,
-        'include FactoryGirl::Syntax::Methods'
-      ).and_return(action)
-      expect(action).to receive(:process)
-      instance.append 'include FactoryGirl::Syntax::Methods'
+      instance.current_mutation = double
+      instance.current_node = double
+      expect(instance.current_mutation).to receive(:append).with(instance.current_node, 'Foobar')
+      instance.append 'Foobar'
     end
 
     it 'parses prepend' do
-      action = double
-      expect(Rewriter::PrependAction).to receive(:new).with(
-        instance,
-        '{{arguments.first}}.include FactoryGirl::Syntax::Methods'
-      ).and_return(action)
-      expect(action).to receive(:process)
-      instance.prepend '{{arguments.first}}.include FactoryGirl::Syntax::Methods'
+      instance.current_mutation = double
+      instance.current_node = double
+      expect(instance.current_mutation).to receive(:prepend).with(instance.current_node, 'Foobar')
+      instance.prepend 'Foobar'
     end
 
     it 'parses insert at end' do
-      action = double
-      expect(Rewriter::InsertAction).to receive(:new).with(
-        instance,
-        '.first',
-        at: 'end',
-        to: 'receiver'
-      ).and_return(action)
-      expect(action).to receive(:process)
-      instance.insert '.first', to: 'receiver'
+      instance.current_mutation = double
+      instance.current_node = double
+      expect(instance.current_mutation).to receive(:insert).with(instance.current_node, 'Foobar', at: 'end', to: 'receiver')
+      instance.insert 'Foobar', to: 'receiver'
     end
 
     it 'parses insert at beginning' do
-      action = double
-      expect(Rewriter::InsertAction).to receive(:new).with(
-        instance,
-        'URI.',
-        at: 'beginning',
-        to: nil
-      ).and_return(action)
-      expect(action).to receive(:process)
-      instance.insert 'URI.', at: 'beginning'
+      instance.current_mutation = double
+      instance.current_node = double
+      expect(instance.current_mutation).to receive(:insert).with(instance.current_node, 'Foobar', at: 'beginning', to: nil)
+      instance.insert 'Foobar', at: 'beginning'
     end
 
     it 'parses insert_after' do
+      instance.current_mutation = double
+      instance.current_node = double
+      expect(instance.current_mutation).to receive(:insert_after).with(instance.current_node, 'Foobar')
+      instance.insert_after 'Foobar'
+    end
+
+    it 'parses replace_erb_stmt_with_expr' do
+      instance.current_mutation = double
+      instance.current_node = double
       action = double
-      expect(Rewriter::InsertAfterAction).to receive(:new).with(
-        instance,
-        '{{arguments.first}}.include FactoryGirl::Syntax::Methods'
-      ).and_return(action)
+      expect(instance.current_mutation).to receive(:actions).and_return([])
+      expect(Rewriter::ReplaceErbStmtWithExprAction).to receive(:new).with(instance.current_node).and_return(action)
       expect(action).to receive(:process)
-      instance.insert_after '{{arguments.first}}.include FactoryGirl::Syntax::Methods'
+      instance.replace_erb_stmt_with_expr
     end
 
     it 'parses replace_with' do
-      action = double
-      expect(Rewriter::ReplaceWithAction).to receive(:new).with(instance, 'create {{arguments}}').and_return(action)
-      expect(action).to receive(:process)
-      instance.replace_with 'create {{arguments}}'
+      instance.current_mutation = double
+      instance.current_node = double
+      expect(instance.current_mutation).to receive(:replace_with).with(instance.current_node, 'Foobar')
+      instance.replace_with 'Foobar'
     end
 
     it 'parses replace with' do
-      action = double
-      expect(Rewriter::ReplaceAction).to receive(:new).with(instance, :message, with: 'test').and_return(action)
-      expect(action).to receive(:process)
-      instance.replace :message, with: 'test'
+      instance.current_mutation = double
+      instance.current_node = double
+      expect(instance.current_mutation).to receive(:replace).with(instance.current_node, :message, with: 'Foobar')
+      instance.replace :message, with: 'Foobar'
     end
 
     it 'parses remove' do
-      action = double
-      expect(Rewriter::RemoveAction).to receive(:new).with(instance, { and_comma: true }).and_return(action)
-      expect(action).to receive(:process)
+      instance.current_mutation = double
+      instance.current_node = double
+      expect(instance.current_mutation).to receive(:remove).with(instance.current_node, and_comma: true)
       instance.remove and_comma: true
     end
 
-    it 'parses remove' do
-      action = double
-      expect(Rewriter::DeleteAction).to receive(:new).with(
-        instance,
-        :dot,
-        :message,
-        { and_comma: true }
-      ).and_return(action)
-      expect(action).to receive(:process)
+    it 'parses delete' do
+      instance.current_mutation = double
+      instance.current_node = double
+      expect(instance.current_mutation).to receive(:delete).with(instance.current_node, :dot, :message, and_comma: true)
       instance.delete :dot, :message, and_comma: true
     end
 
     it 'parses wrap with' do
-      action = double
-      expect(Rewriter::WrapAction).to receive(:new).with(instance, with: 'module Foo', indent: nil).and_return(action)
-      expect(action).to receive(:process)
-      instance.wrap with: 'module Foo'
+      instance.current_mutation = double
+      instance.current_node = double
+      expect(instance.current_mutation).to receive(:wrap).with(instance.current_node, with: 'module Foobar')
+      instance.wrap with: 'module Foobar'
     end
 
     it 'parses warn' do
@@ -251,32 +236,6 @@ module Synvert::Core
         instance.process
       end
 
-      it 'does not read file if already read' do
-        instance =
-          Rewriter::Instance.new rewriter, ['spec/spec_helper.rb'] do
-            with_node type: 'block', caller: { receiver: 'RSpec', message: 'configure' } do
-              unless_exist_node type: 'send', message: 'include', arguments: ['FactoryGirl::Syntax::Methods'] do
-                insert '{{arguments.first}}.include FactoryGirl::Syntax::Methods'
-              end
-            end
-          end
-        input = <<~EOS
-          RSpec.configure do |config|
-            config.include FactoryGirl::Syntax::Methods
-          end
-        EOS
-        output = <<~EOS
-          RSpec.configure do |config|
-            config.include FactoryGirl::Syntax::Methods
-          end
-        EOS
-        expect(Dir).to receive(:glob).with('./spec/spec_helper.rb').and_return(['spec/spec_helper.rb']).twice
-        expect(File).to receive(:read).with('spec/spec_helper.rb', encoding: 'UTF-8').and_return(input).once
-        expect(File).not_to receive(:write).with('spec/spec_helper.rb', output)
-        instance.process
-        instance.process
-      end
-
       it 'updates file_source and file_ast when writing a file' do
         instance =
           Rewriter::Instance.new rewriter, ['spec/**/*_spec.rb'] do
@@ -308,32 +267,6 @@ module Synvert::Core
       end
     end
 
-    describe '#get_conflict_actions' do
-      let(:rewriter) { Rewriter.new('foo', 'bar') }
-
-      it 'has no conflict' do
-        action1 = double(begin_pos: 10, end_pos: 20)
-        action2 = double(begin_pos: 30, end_pos: 40)
-        action3 = double(begin_pos: 50, end_pos: 60)
-        instance = Rewriter::Instance.new rewriter, ['spec/spec_helper.rb']
-        instance.instance_variable_set :@actions, [action1, action2, action3]
-        conflict_actions = instance.send(:get_conflict_actions)
-        expect(conflict_actions).to eq []
-        expect(instance.instance_variable_get(:@actions)).to eq [action1, action2, action3]
-      end
-
-      it 'has no conflict' do
-        action1 = double(begin_pos: 30, end_pos: 40)
-        action2 = double(begin_pos: 50, end_pos: 60)
-        action3 = double(begin_pos: 10, end_pos: 20)
-        instance = Rewriter::Instance.new rewriter, ['spec/spec_helper.rb']
-        instance.instance_variable_set :@actions, [action1, action2, action3]
-        conflict_actions = instance.send(:get_conflict_actions)
-        expect(conflict_actions).to eq [action2, action1]
-        expect(instance.instance_variable_get(:@actions)).to eq [action3]
-      end
-    end
-
     describe '#process_with_node' do
       it 'resets current_node' do
         node1 = double

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

@@ -14,10 +14,7 @@ module Synvert::Core
     EOS
 
     let(:node) { Parser::CurrentRuby.parse(source) }
-    before do
-      Rewriter::Instance.reset
-      instance.current_node = node
-    end
+    before { instance.current_node = node }
 
     describe '#process' do
       it 'call block with child node' do

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

@@ -18,10 +18,7 @@ module Synvert::Core
 
     let(:node) { Parser::CurrentRuby.parse(source) }
 
-    before do
-      Rewriter::Instance.reset
-      instance.current_node = node
-    end
+    before { instance.current_node = node }
 
     describe '#process' do
       it 'not call block if no matching node' do
@@ -58,18 +55,6 @@ module Synvert::Core
         expect(block_nodes.size).to eq 2
       end
 
-      it 'raises ParseError' do
-        scope = described_class.new(instance, 'synvert') {}
-        expect {
-          scope.process
-        }.to raise_error(NodeQuery::Compiler::ParseError)
-
-        scope = described_class.new(instance, '.block <caller.arguments> .hash') {}
-        expect {
-          scope.process
-        }.to raise_error(NodeQuery::Compiler::ParseError)
-      end
-
       it 'raises InvalidOperatorError' do
         scope = described_class.new(instance, '.send[receiver IN FactoryGirl]') {}
         expect {

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

@@ -9,19 +9,18 @@ module Synvert::Core
       Rewriter::Instance.new(rewriter, 'file pattern')
     }
     let(:source) { <<~EOS }
-      describe Post do
-        it 'gets post' do
-          FactoryGirl.create :post
+      describe User do
+        describe 'create' do
+          it 'creates user' do
+            FactoryGirl.create :user
+          end
         end
       end
     EOS
 
     let(:node) { Parser::CurrentRuby.parse(source) }
 
-    before do
-      Rewriter::Instance.reset
-      instance.current_node = node
-    end
+    before { instance.current_node = node }
 
     describe '#process' do
       it 'not call block if no matching node' do
@@ -42,7 +41,7 @@ module Synvert::Core
                                     type: 'send',
                                     receiver: 'FactoryGirl',
                                     message: 'create',
-                                    arguments: [':post'] do
+                                    arguments: [':user'] do
             run = true
             type_in_scope = node.type
           end
@@ -55,27 +54,37 @@ module Synvert::Core
       it 'matches multiple block nodes' do
         block_nodes = []
         scope =
-          Rewriter::WithinScope.new(instance, { type: 'block' }, { stop_when_match: false }) do
+          Rewriter::WithinScope.new(instance, { type: 'block' }) do
+            block_nodes << node
+          end
+        scope.process
+        expect(block_nodes.size).to eq 3
+      end
+
+      it 'matches only 2 block nodes if including_self is false' do
+        block_nodes = []
+        scope =
+          Rewriter::WithinScope.new(instance, { type: 'block' }, { including_self: false }) do
             block_nodes << node
           end
         scope.process
         expect(block_nodes.size).to eq 2
       end
 
-      it 'matches only one block node if no recursive' do
+      it 'matches only one block node if recursive is false' do
         block_nodes = []
         scope =
-          Rewriter::WithinScope.new(instance, { type: 'block' }, { stop_when_match: true }) do
+          Rewriter::WithinScope.new(instance, { type: 'block' }, { recursive: false }) do
             block_nodes << node
           end
         scope.process
         expect(block_nodes.size).to eq 1
       end
 
-      it 'matches only one direct node' do
+      it 'matches only one block node if stop_at_first_match is true' do
         block_nodes = []
         scope =
-          Rewriter::WithinScope.new(instance, { type: 'block' }, { direct: true }) do
+          Rewriter::WithinScope.new(instance, { type: 'block' }, { stop_at_first_match: true }) do
             block_nodes << node
           end
         scope.process

data/synvert-core-ruby.gemspec

@@ -14,13 +14,15 @@ Gem::Specification.new do |spec|
   spec.homepage      = "https://github.com/xinminlabs/synvert-core-ruby"
   spec.license       = "MIT"
 
-  spec.files         = `git ls-files -z`.split("\x0") +
-                       %w[lib/synvert/core/node_query/lexer.rex.rb lib/synvert/core/node_query/parser.racc.rb]
+  spec.files         = `git ls-files -z`.split("\x0")
   spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
-  spec.add_runtime_dependency "activesupport"
+  spec.add_runtime_dependency "activesupport", "< 7.0.0"
   spec.add_runtime_dependency "erubis"
+  spec.add_runtime_dependency "node_query"
+  spec.add_runtime_dependency "node_mutation"
   spec.add_runtime_dependency "parser"
+  spec.add_runtime_dependency "parser_node_ext"
 end

metadata

@@ -1,17 +1,31 @@
 --- !ruby/object:Gem::Specification
 name: synvert-core
 version: !ruby/object:Gem::Version
-  version: 1.4.0
+  version: 1.6.0
 platform: ruby
 authors:
 - Richard Huang
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-05-21 00:00:00.000000000 Z
+date: 2022-09-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 7.0.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 7.0.0
+- !ruby/object:Gem::Dependency
+  name: erubis
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -25,7 +39,21 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: erubis
+  name: node_query
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: node_mutation
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -52,6 +80,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: parser_node_ext
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: convert ruby code to better syntax automatically.
 email:
 - flyerhzm@gmail.com
@@ -64,53 +106,19 @@ files:
 - ".rspec"
 - CHANGELOG.md
 - Gemfile
+- Gemfile.lock
 - Guardfile
 - LICENSE.txt
 - README.md
 - Rakefile
 - lib/synvert/core.rb
-- lib/synvert/core/array_ext.rb
 - lib/synvert/core/configuration.rb
 - lib/synvert/core/engine.rb
 - lib/synvert/core/engine/erb.rb
 - lib/synvert/core/exceptions.rb
 - lib/synvert/core/node_ext.rb
-- lib/synvert/core/node_query.rb
-- lib/synvert/core/node_query/compiler.rb
-- lib/synvert/core/node_query/compiler/array.rb
-- lib/synvert/core/node_query/compiler/attribute.rb
-- lib/synvert/core/node_query/compiler/attribute_list.rb
-- lib/synvert/core/node_query/compiler/basic_selector.rb
-- lib/synvert/core/node_query/compiler/boolean.rb
-- lib/synvert/core/node_query/compiler/comparable.rb
-- lib/synvert/core/node_query/compiler/dynamic_attribute.rb
-- lib/synvert/core/node_query/compiler/expression.rb
-- lib/synvert/core/node_query/compiler/float.rb
-- lib/synvert/core/node_query/compiler/identifier.rb
-- lib/synvert/core/node_query/compiler/integer.rb
-- lib/synvert/core/node_query/compiler/invalid_operator_error.rb
-- lib/synvert/core/node_query/compiler/nil.rb
-- lib/synvert/core/node_query/compiler/parse_error.rb
-- lib/synvert/core/node_query/compiler/regexp.rb
-- lib/synvert/core/node_query/compiler/selector.rb
-- lib/synvert/core/node_query/compiler/string.rb
-- lib/synvert/core/node_query/compiler/symbol.rb
-- lib/synvert/core/node_query/lexer.rex
-- lib/synvert/core/node_query/lexer.rex.rb
-- lib/synvert/core/node_query/parser.racc.rb
-- lib/synvert/core/node_query/parser.y
 - lib/synvert/core/rewriter.rb
-- lib/synvert/core/rewriter/action.rb
-- lib/synvert/core/rewriter/action/append_action.rb
-- lib/synvert/core/rewriter/action/delete_action.rb
-- lib/synvert/core/rewriter/action/insert_action.rb
-- lib/synvert/core/rewriter/action/insert_after_action.rb
-- lib/synvert/core/rewriter/action/prepend_action.rb
-- lib/synvert/core/rewriter/action/remove_action.rb
-- 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/action/wrap_action.rb
 - lib/synvert/core/rewriter/any_value.rb
 - lib/synvert/core/rewriter/condition.rb
 - lib/synvert/core/rewriter/condition/if_exist_condition.rb
@@ -130,19 +138,7 @@ files:
 - spec/support/parser_helper.rb
 - spec/synvert/core/engine/erb_spec.rb
 - spec/synvert/core/node_ext_spec.rb
-- spec/synvert/core/node_query/lexer_spec.rb
-- spec/synvert/core/node_query/parser_spec.rb
-- spec/synvert/core/rewriter/action/append_action_spec.rb
-- spec/synvert/core/rewriter/action/delete_action_spec.rb
-- spec/synvert/core/rewriter/action/insert_action_spec.rb
-- spec/synvert/core/rewriter/action/insert_after_action_spec.rb
-- spec/synvert/core/rewriter/action/prepend_action_spec.rb
-- spec/synvert/core/rewriter/action/remove_action_spec.rb
-- spec/synvert/core/rewriter/action/replace_action_spec.rb
 - spec/synvert/core/rewriter/action/replace_erb_stmt_with_expr_action_spec.rb
-- spec/synvert/core/rewriter/action/replace_with_action_spec.rb
-- spec/synvert/core/rewriter/action/wrap_action_spec.rb
-- spec/synvert/core/rewriter/action_spec.rb
 - spec/synvert/core/rewriter/condition/if_exist_condition_spec.rb
 - spec/synvert/core/rewriter/condition/if_only_exist_condition_spec.rb
 - spec/synvert/core/rewriter/condition/unless_exist_condition_spec.rb
@@ -186,19 +182,7 @@ test_files:
 - spec/support/parser_helper.rb
 - spec/synvert/core/engine/erb_spec.rb
 - spec/synvert/core/node_ext_spec.rb
-- spec/synvert/core/node_query/lexer_spec.rb
-- spec/synvert/core/node_query/parser_spec.rb
-- spec/synvert/core/rewriter/action/append_action_spec.rb
-- spec/synvert/core/rewriter/action/delete_action_spec.rb
-- spec/synvert/core/rewriter/action/insert_action_spec.rb
-- spec/synvert/core/rewriter/action/insert_after_action_spec.rb
-- spec/synvert/core/rewriter/action/prepend_action_spec.rb
-- spec/synvert/core/rewriter/action/remove_action_spec.rb
-- spec/synvert/core/rewriter/action/replace_action_spec.rb
 - spec/synvert/core/rewriter/action/replace_erb_stmt_with_expr_action_spec.rb
-- spec/synvert/core/rewriter/action/replace_with_action_spec.rb
-- spec/synvert/core/rewriter/action/wrap_action_spec.rb
-- spec/synvert/core/rewriter/action_spec.rb
 - spec/synvert/core/rewriter/condition/if_exist_condition_spec.rb
 - spec/synvert/core/rewriter/condition/if_only_exist_condition_spec.rb
 - spec/synvert/core/rewriter/condition/unless_exist_condition_spec.rb

data/lib/synvert/core/array_ext.rb

@@ -1,48 +0,0 @@
-# frozen_string_literal: true
-
-# Extend Array.
-class Array
-  # Get child node by the name.
-  #
-  # @param child_name [String] name of child node.
-  # @return [Parser::AST::Node] the child node.
-  def child_node_by_name(child_name)
-    direct_child_name, nested_child_name = child_name.split('.', 2)
-    child_direct_child_node = direct_child_name =~ /\A\d+\z/ ? self[direct_child_name.to_i - 1] : self.send(direct_child_name)
-    return child_direct_child_node.child_node_by_name(nested_child_name) if nested_child_name
-    return child_direct_child_node if child_direct_child_node
-
-    raise Synvert::Core::MethodNotSupported,
-          "child_node_by_name is not handled for #{debug_info}, child_name: #{child_name}"
-  end
-
-  # Get the source range of child node.
-  #
-  # @param child_name [String] name of child node.
-  # @return [Parser::Source::Range] source range of child node.
-  def child_node_range(child_name)
-    direct_child_name, nested_child_name = child_name.split('.', 2)
-    child_direct_child_node = direct_child_name =~ /\A\d+\z/ ? self[direct_child_name.to_i - 1] : self.send(direct_child_name)
-    if nested_child_name
-      return child_direct_child_node.child_node_range(nested_child_name)
-    elsif child_direct_child_node
-      return (
-        Parser::Source::Range.new(
-          '(string)',
-          child_direct_child_node.loc.expression.begin_pos,
-          child_direct_child_node.loc.expression.end_pos
-        )
-      )
-    else
-      raise Synvert::Core::MethodNotSupported,
-            "child_node_range is not handled for #{debug_info}, child_name: #{child_name}"
-    end
-  end
-
-  # Return the debug info.
-  #
-  # @return [String] file, line, source and node.
-  def debug_info
-    map(&:debug_info).join("\n")
-  end
-end