rspec-multiprocess_runner 1.3.0 → 1.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2f561425e9f9c62eba1610b84278f1f3d86a5448d0c671324125bb88c0150f81
-  data.tar.gz: 3b79c2b0e07224cc2fe64204b0adf5e7ca90099846e84cc2078ad771c617c05d
+  metadata.gz: 2ee952287e85072086a4feab9909db4e61d6cafcc3bc6b574795f368e9b4010c
+  data.tar.gz: 65a2d11400a0604631eed4cb6dc7df62307b5bd9914a3d768a3bff3316960a11
 SHA512:
-  metadata.gz: ea71c613e8309da1d384f2c04778dce6871131bfa623ab4edabd4df6226d3393b39b6a1d3f688146881827650c56490a5b688139ba5c8bbcb76a199bced318d5
-  data.tar.gz: 0db27a58d409e7485c4c4ae934f59f53ea86d63fa2125194813fbbefdd28f39db90ea59355cf205e6b39610f344b99447d112d72adc293a12b303f29b2df1f1c
+  metadata.gz: 52295ede507181b2b4f6a738131b3f279845076010421364e22aec501660a7e9ad442fda17723ec9f39f6247ee6f6d371ba0a32cafd8589404d8ce074ec40b1a
+  data.tar.gz: 86e482ce416b01185b3e4eed3bd3edaac850155c9d3353d0108cf5a14429329847ff1d741a3319faebcfd9fca22830569bafb87de7031c59b58fd01e2d97bb40

data/CHANGELOG.md

@@ -1,9 +1,30 @@
+# 1.4.2
+
+* Add better logging for errors that occur outside of tests
+* Add docker tests for all current ruby versions and more ruby versions to test in
+* Fix some minor bugs that caused incorrect error codes (still failure codes)
+* Fixed the error code spec file
+
+# 1.4.1
+
+* Fix file coordinator helpers failing due to unknown method on command line args
+
+# 1.4.0
+
+* Fix issue catching spec files failing outside of tests (it will now die loudly instead of passing silently).
+
+# 1.3.3
+
+* File output bug fix
+
+# 1.3.1
+
+* Or use an environment variable for summary file.
+
 # 1.3.0
 
 * Add file output option
 
-# 1.2.4
-
 # 1.2.3
 
 * Fix time library sporadic failure

data/Gemfile.lock

@@ -1,36 +1,36 @@
 PATH
   remote: .
   specs:
-    rspec-multiprocess_runner (1.3.0)
+    rspec-multiprocess_runner (1.4.2)
       rspec (>= 3.0)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    byebug (11.0.1)
+    byebug (11.1.3)
     coderay (1.1.2)
     diff-lcs (1.3)
-    method_source (0.9.2)
-    pry (0.12.2)
-      coderay (~> 1.1.0)
-      method_source (~> 0.9.0)
-    pry-byebug (3.7.0)
+    method_source (1.0.0)
+    pry (0.13.1)
+      coderay (~> 1.1)
+      method_source (~> 1.0)
+    pry-byebug (3.9.0)
       byebug (~> 11.0)
-      pry (~> 0.10)
-    rake (10.5.0)
-    rspec (3.8.0)
-      rspec-core (~> 3.8.0)
-      rspec-expectations (~> 3.8.0)
-      rspec-mocks (~> 3.8.0)
-    rspec-core (3.8.2)
-      rspec-support (~> 3.8.0)
-    rspec-expectations (3.8.5)
+      pry (~> 0.13.0)
+    rake (13.0.1)
+    rspec (3.9.0)
+      rspec-core (~> 3.9.0)
+      rspec-expectations (~> 3.9.0)
+      rspec-mocks (~> 3.9.0)
+    rspec-core (3.9.2)
+      rspec-support (~> 3.9.3)
+    rspec-expectations (3.9.2)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.8.0)
-    rspec-mocks (3.8.2)
+      rspec-support (~> 3.9.0)
+    rspec-mocks (3.9.1)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.8.0)
-    rspec-support (3.8.3)
+      rspec-support (~> 3.9.0)
+    rspec-support (3.9.3)
     stub_env (1.0.4)
       rspec (>= 2.0, < 4.0)
 
@@ -40,9 +40,9 @@ PLATFORMS
 DEPENDENCIES
   bundler (>= 1.10)
   pry-byebug
-  rake (~> 10.0)
+  rake (>= 12.3.3)
   rspec-multiprocess_runner!
   stub_env
 
 BUNDLED WITH
-   2.0.2
+   2.1.4

data/lib/rspec/multiprocess_runner/command_line_options.rb

@@ -24,7 +24,7 @@ module RSpec::MultiprocessRunner
       self.hostname = "localhost"
       self.head_node = true
       self.max_nodes = 5
-      self.summary_filename = nil
+      self.summary_filename = ENV['SUMMARY_FILE']
     end
 
     def parse(command_line_args, error_stream=$stderr)

data/lib/rspec/multiprocess_runner/coordinator.rb

@@ -41,7 +41,7 @@ module RSpec::MultiprocessRunner
     def exit_code
       exit_code = 0
       exit_code |= 1 if any_example_failed?
-      exit_code |= 2 if !failed_workers.empty?
+      exit_code |= 2 if failed_workers.any? || error_messages.any?
       exit_code |= 4 if work_left_to_do? || @file_coordinator.missing_files.any?
       exit_code
     end
@@ -52,10 +52,10 @@ module RSpec::MultiprocessRunner
         end_workers_in_parallel(@workers.dup, :kill_now)
       else
         @shutting_down = true
-        print "Shutting down #{pluralize(@workers.size, "worker")} …" if options.print_summary
+        print "Shutting down #{pluralize(@workers.size, "worker")} …" if options[:print_summary]
         # end_workers_in_parallel modifies @workers, so dup before sending in
         end_workers_in_parallel(@workers.dup, :shutdown_now)
-        if options.print_summary
+        if options[:print_summary]
           puts " done"
           print_summary
         end
@@ -203,25 +203,30 @@ module RSpec::MultiprocessRunner
       end
     end
 
+    def summary_file
+      File.new(options.summary_filename, 'w') if (options.summary_filename)
+    end
+
     def print_summary
       elapsed = Time.now - @start_time
       by_status_and_time = combine_example_results.each_with_object(Hash.new { |h, k| h[k] = [] }) do |result, idx|
         idx[result.status] << result
       end
 
-      outputs = [STDOUT]
-      outputs << File.new(options.summary_filename, 'w') if (options.summary_filename)
+      outputs = [$stdout, summary_file].compact
       outputs.each do |output|
         print_skipped_files_details(output)
         print_pending_example_details(output, by_status_and_time["pending"])
         print_failed_example_details(output, by_status_and_time["failed"])
         print_missing_files(output)
-        log_failed_files(by_status_and_time["failed"].map(&:filename).uniq  + @file_coordinator.missing_files.to_a) if options.log_failing_files
+        log_failed_files(output, by_status_and_time["failed"].map(&:filename).uniq  + @file_coordinator.missing_files.to_a) if options.log_failing_files
         print_failed_process_details(output)
+        print_error_messages(output)
         output.puts
         print_elapsed_time(output, elapsed)
         output.puts failed? ? "FAILURE" : "SUCCESS"
         print_example_counts(output, by_status_and_time)
+        output.close unless output == $stdout
       end
     end
 
@@ -229,6 +234,10 @@ module RSpec::MultiprocessRunner
       @file_coordinator.results.select { |r| r.run_status == "example_complete" }.sort_by { |r| r.time_finished }
     end
 
+    def error_messages
+      @file_coordinator.results.select { |r| r.run_status == "error_running" }.sort_by { |r| r.time_finished }
+    end
+
     def any_example_failed?
       @file_coordinator.results.detect { |r| r.status == "failed" }
     end
@@ -263,7 +272,7 @@ module RSpec::MultiprocessRunner
       end
     end
 
-    def log_failed_files(failed_files)
+    def log_failed_files(output, failed_files)
       return if failed_files.nil?
       output.puts
       output.puts "Writing failures to file: #{options.log_failing_files}"
@@ -286,6 +295,7 @@ module RSpec::MultiprocessRunner
       pending_count = by_status_and_time["pending"] ? by_status_and_time["pending"].size : 0
       missing_count = @file_coordinator.missing_files.size
       process_failure_count = failed_workers.size
+      error_count = error_messages.size
       skipped_count = @file_coordinator.remaining_files.size
 
       # Copied from RSpec
@@ -293,6 +303,7 @@ module RSpec::MultiprocessRunner
       summary << ", " << pluralize(failure_count, "failure")
       summary << ", #{pending_count} pending" if pending_count > 0
       summary << ", " << pluralize(process_failure_count, "failed proc") if process_failure_count > 0
+      summary << ", " << pluralize(error_count, "error") if error_count > 0
       summary << ", " << pluralize(skipped_count, "skipped file") if skipped_count > 0
       summary << ", " << pluralize(missing_count, "missing file") if missing_count > 0
       output.puts summary
@@ -307,6 +318,17 @@ module RSpec::MultiprocessRunner
       end
     end
 
+    def print_error_messages(output)
+      return if error_messages.nil?
+      output.puts
+      output.puts "Errors:"
+      error_messages.each_with_index do |error, i|
+        output.puts
+        output.puts "  #{i.next}) #{error.filename}"
+        output.print error.message
+      end
+    end
+
     def print_missing_files(output)
       return if @file_coordinator.missing_files.empty?
       output.puts

data/lib/rspec/multiprocess_runner/file_coordinator.rb

@@ -1,6 +1,7 @@
 # encoding: utf-8
 require 'rspec/multiprocess_runner'
 require 'socket'
+require 'json'
 require 'set'
 
 module RSpec::MultiprocessRunner
@@ -14,7 +15,7 @@ module RSpec::MultiprocessRunner
     COMMAND_FINISHED = "finished"
     COMMAND_START = "start"
 
-    def initialize(files, options={})
+    def initialize(files, options=RSpec::MultiprocessRunner::CommandLineOptions.new)
       @spec_files = []
       @results = Set.new
       @threads = []
@@ -69,7 +70,8 @@ module RSpec::MultiprocessRunner
         else
           return nil # Malformed response, assume done, cease function
         end
-      rescue
+      rescue StandardError => e
+        puts("Got exception #{e} in get_file")
         return nil # If Error, assume done, cease function
       end
     end
@@ -122,7 +124,7 @@ module RSpec::MultiprocessRunner
         break unless raw_response
         command, results, node = JSON.parse(raw_response)
         if command == COMMAND_START
-          if results == options.start_string
+          if results == options.run_identifier
             socket.puts COMMAND_START
           else
             socket.puts COMMAND_FINISHED
@@ -143,7 +145,7 @@ module RSpec::MultiprocessRunner
 
     def start?
       begin
-        @node_socket.puts [COMMAND_START, options.start_string].to_json
+        @node_socket.puts [COMMAND_START, options.run_identifier].to_json
         response = @node_socket.gets
         response = response.chomp if response
         raise BadStartStringError if response == COMMAND_FINISHED

data/lib/rspec/multiprocess_runner/reporting_formatter.rb

@@ -59,6 +59,13 @@ module RSpec::MultiprocessRunner
       )
     end
 
+    def message(struct)
+      message = struct.message
+      return if message =~ /^No examples found./
+
+      worker.report_error(message)
+    end
+
     private
 
     def capture_output

data/lib/rspec/multiprocess_runner/version.rb

@@ -1,5 +1,5 @@
 module RSpec
   module MultiprocessRunner
-    VERSION = "1.3.0"
+    VERSION = "1.4.2"
   end
 end

data/lib/rspec/multiprocess_runner/worker.rb

@@ -29,6 +29,7 @@ module RSpec::MultiprocessRunner
 
     STATUS_EXAMPLE_COMPLETE = "example_complete"
     STATUS_RUN_COMPLETE = "run_complete"
+    ERROR_RUNNING = "error_running"
 
     def initialize(environment_number, options)
       @environment_number = environment_number
@@ -196,6 +197,10 @@ module RSpec::MultiprocessRunner
         end
         $stdout.puts "#{environment_number} (#{pid}): #{message_hash["description"]} (#{location})#{suffix}"
         @current_example_started_at = Time.now
+      when ERROR_RUNNING
+        example_results << Result.new(message_hash)
+        $stdout.puts "Error in file: #{message_hash["filename"]}"
+        $stdout.print message_hash["message"]
       else
         $stderr.puts "Received unsupported status #{message_hash["status"].inspect} in worker #{pid}"
       end
@@ -219,7 +224,15 @@ module RSpec::MultiprocessRunner
         description: description,
         line_number: line_number,
         details: details,
-        filename: @current_file
+        filename: @current_file,
+      )
+    end
+
+    def report_error(message)
+      send_message_to_coordinator(
+        status: ERROR_RUNNING,
+        message: message,
+        filename: @current_file,
       )
     end
 
@@ -263,9 +276,9 @@ module RSpec::MultiprocessRunner
     def act_on_message_from_coordinator(message_hash)
       return handle_closed_coordinator_socket unless message_hash # EOF
       case message_hash["command"]
-      when "quit"
+      when COMMAND_QUIT
         exit
-      when "run_file"
+      when COMMAND_RUN_FILE
         execute_spec(message_hash["filename"])
       else
         $stderr.puts "Received unsupported command #{message_hash["command"].inspect} in worker #{pid}"
@@ -281,7 +294,9 @@ module RSpec::MultiprocessRunner
       RSpec.world.example_groups.clear
 
       ReportingFormatter.worker = self
-      RSpec::Core::Runner.run(@rspec_arguments + [spec_file])
+      error_stream = StringIO.new
+      output_stream = StringIO.new
+      RSpec::Core::Runner.run(@rspec_arguments + [spec_file], error_stream, output_stream)
       send_message_to_coordinator(status: STATUS_RUN_COMPLETE, filename: spec_file)
     ensure
       @current_file = nil
@@ -322,7 +337,7 @@ module RSpec::MultiprocessRunner
 
   # @private
   class Result
-    attr_reader :run_status, :status, :description, :details, :filename, :time_finished
+    attr_reader :description, :details, :filename, :message, :run_status, :status, :time_finished
 
     def initialize(complete_message, time = Time.now)
       @hash = complete_message
@@ -331,6 +346,7 @@ module RSpec::MultiprocessRunner
       @description = complete_message["description"]
       @details = complete_message["details"]
       @filename = complete_message["filename"]
+      @message = complete_message["message"]
       @time_finished = time
     end
 

data/rspec-multiprocess_runner.gemspec

@@ -22,6 +22,7 @@ Gem::Specification.new do |spec|
 
   spec.add_development_dependency "bundler", ">= 1.10"
   spec.add_development_dependency "pry-byebug"
-  spec.add_development_dependency "rake", "~> 10.0"
+  spec.add_development_dependency "byebug", "<11" if RUBY_VERSION.to_f < 2.4
+  spec.add_development_dependency "rake", ">= 12.3.3"
   spec.add_development_dependency "stub_env"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rspec-multiprocess_runner
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.4.2
 platform: ruby
 authors:
 - Rhett Sutphin
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-10-02 00:00:00.000000000 Z
+date: 2020-05-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -59,16 +59,16 @@ dependencies:
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: 12.3.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: 12.3.3
 - !ruby/object:Gem::Dependency
   name: stub_env
   requirement: !ruby/object:Gem::Requirement
@@ -131,8 +131,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6.2
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: A runner for RSpec 3 that uses multiple processes to execute specs in parallel