synvert-core 1.10.0 → 1.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 16de1af456b15373297aebe409711e1725bb329e3da7d4fa8d84023348ceb236
-  data.tar.gz: e034ccf44cfce96335f10a38d5aa36793faa08c84e8d75509fc658824cf64676
+  metadata.gz: 0663ba61695ae5b47f27e9a23f1edbfe31d3746f9b3e02a79315a827fae6f36c
+  data.tar.gz: 8199c6c8a122555bfbf0df0ef79f6bd96ade0cdfe472d81d6abe4cccde3b6c59
 SHA512:
-  metadata.gz: 3f745e87c308b417c5938c2731af00254abe5beba70472bca1ebfdf534127b7df8f6ab33261b8eb5d31d33deaa471927a780a14720f0ee406eea44f5a7e06495
-  data.tar.gz: ac2c35d21a4e6558b14c861966c6326446926bc27cc9e2d278757df31cf68a33142e62d47f9d40e5ec72c1d3fd8efde6574046e00506c9a12d0a800580a00023
+  metadata.gz: 3609b9bdb845681c9dd71d9d0eca4985d43203ce28fb1e874e9538322f8205995d77f5b8c2b43fb11caf698a69772c3e7bdd5114a709e4513313339870576a5e
+  data.tar.gz: 952578268a2c8cba6dcd2883141fe5977917594ea01b76e51e93032fc71225c813b9021ec1852c3a93d76f6438f30f9975ac47b36c5da3d7c5805fa15763fa9e

data/CHANGELOG.md

@@ -1,5 +1,15 @@
 # CHANGELOG
 
+## 1.11.0 (2022-10-11)
+
+* Add `Configuration.number_of_workers`
+* Test rewriter in parallel
+
+## 1.10.1 (2022-10-09)
+
+* Do not reset `@options`
+* `Rewriter.fetch` does not raise error if rewriter not found
+
 ## 1.10.0 (2022-10-09)
 
 * Eval snippet by http url, file path or snippet name

data/Gemfile.lock

@@ -1,11 +1,12 @@
 PATH
   remote: .
   specs:
-    synvert-core (1.10.0)
+    synvert-core (1.11.0)
       activesupport (< 7.0.0)
       erubis
       node_mutation
       node_query
+      parallel
       parser
       parser_node_ext
 
@@ -57,6 +58,7 @@ GEM
     notiffany (0.1.3)
       nenv (~> 0.1)
       shellany (~> 0.0)
+    parallel (1.22.1)
     parser (3.1.2.1)
       ast (~> 2.4.1)
     parser_node_ext (0.4.0)

data/lib/synvert/core/configuration.rb

@@ -8,7 +8,8 @@ module Synvert::Core
       # @!attribute [w] skip_paths
       # @!attribute [w] only_paths
       # @!attribute [w] show_run_process
-      attr_writer :root_path, :skip_paths, :only_paths, :show_run_process
+      # @!attribute [w] number_of_workers
+      attr_writer :root_path, :skip_paths, :only_paths, :show_run_process, :number_of_workers
 
       # Get the path.
       #
@@ -37,6 +38,13 @@ module Synvert::Core
       def show_run_process
         @show_run_process || false
       end
+
+      # Number of workers
+      #
+      # @return [Integer] default is 1
+      def number_of_workers
+        @number_of_workers || 1
+      end
     end
   end
 end

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+require 'parallel'
 
 module Synvert::Core
   # Instance is an execution unit, it finds specified ast nodes,
@@ -41,8 +42,14 @@ module Synvert::Core
     # It finds specified files, for each file, it executes the block code, tests the original code,
     # then returns the actions.
     def test
-      get_file_paths.map do |file_path|
-        test_file(file_path)
+      if Configuration.number_of_workers > 1
+        Parallel.map(get_file_paths, in_processes: Configuration.number_of_workers) do |file_path|
+          test_file(file_path)
+        end
+      else
+        get_file_paths.map do |file_path|
+          test_file(file_path)
+        end
       end
     end
 

data/lib/synvert/core/rewriter.rb

@@ -51,14 +51,10 @@ module Synvert::Core
       # @param group [String] rewrtier group.
       # @param name [String] rewrtier name.
       # @return [Synvert::Core::Rewriter] the matching rewriter.
-      # @raise [Synvert::Core::RewriterNotFound] if the registered rewriter is not found.
       def fetch(group, name)
         group = group.to_s
         name = name.to_s
-        rewriter = rewriters.dig(group, name)
-        raise RewriterNotFound, "Rewriter #{group} #{name} not found" unless rewriter
-
-        rewriter
+        rewriters.dig(group, name)
       end
 
       # Get all available rewriters
@@ -136,24 +132,16 @@ module Synvert::Core
     # It will call the block but doesn't change any file.
     def process_with_sandbox
       @options[:run_instance] = false
-      begin
-        process
-      ensure
-        @options[:run_instance] = true
-      end
+      process
     end
 
     def test
       @options[:write_to_file] = false
-      begin
-        @affected_files = Set.new
-        instance_eval(&@block)
+      @affected_files = Set.new
+      instance_eval(&@block)
 
-        if !@affected_files.empty? && @redo_until_no_change # redo
-          test
-        end
-      ensure
-        @options[:write_to_file] = true
+      if !@affected_files.empty? && @redo_until_no_change # redo
+        test
       end
       @test_results
     end

data/lib/synvert/core/version.rb

@@ -2,6 +2,6 @@
 
 module Synvert
   module Core
-    VERSION = '1.10.0'
+    VERSION = '1.11.0'
   end
 end

data/lib/synvert/core.rb

@@ -17,8 +17,6 @@ module Synvert
     autoload :Configuration, 'synvert/core/configuration'
     autoload :Rewriter, 'synvert/core/rewriter'
     autoload :Engine, 'synvert/core/engine'
-    autoload :RewriterNotFound, 'synvert/core/exceptions'
-    autoload :MethodNotSupported, 'synvert/core/exceptions'
     autoload :Utils, 'synvert/core/utils'
   end
 end

data/spec/synvert/core/rewriter_spec.rb

@@ -238,15 +238,6 @@ module Synvert::Core
         rewriter.process
         expect(Rewriter.fetch('group', 'sub_rewriter')).not_to be_nil
       end
-
-      it 'raises RewriterNotFound' do
-        rewriter =
-          Rewriter.new 'group', 'name' do
-            add_snippet :group, :not_exist
-          end
-        expect { rewriter.process }
-          .to raise_error(RewriterNotFound)
-      end
     end
 
     it 'parses helper_method' do

data/synvert-core-ruby.gemspec

@@ -25,4 +25,5 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency "node_mutation"
   spec.add_runtime_dependency "parser"
   spec.add_runtime_dependency "parser_node_ext"
+  spec.add_runtime_dependency "parallel"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: synvert-core
 version: !ruby/object:Gem::Version
-  version: 1.10.0
+  version: 1.11.0
 platform: ruby
 authors:
 - Richard Huang
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-10-09 00:00:00.000000000 Z
+date: 2022-10-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -94,6 +94,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: parallel
+  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
@@ -116,7 +130,6 @@ files:
 - 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/rewriter.rb
 - lib/synvert/core/rewriter/action/replace_erb_stmt_with_expr_action.rb

data/lib/synvert/core/exceptions.rb

@@ -1,11 +0,0 @@
-# frozen_string_literal: true
-
-module Synvert::Core
-  # Rewriter not found exception.
-  class RewriterNotFound < RuntimeError
-  end
-
-  # Method not supported exception.
-  class MethodNotSupported < RuntimeError
-  end
-end