rspec-multiprocess_runner 1.4.1 → 1.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9e0686de0f6ec92d9bfd7eddfb6a8c6ed3a781337dfa8557a49f3034eeff9fdf
-  data.tar.gz: 61a4bc72b7bff1eace52541606b2b1d9a38bfe2c0a0ae6e6b110947824ddeda8
+  metadata.gz: 2ee952287e85072086a4feab9909db4e61d6cafcc3bc6b574795f368e9b4010c
+  data.tar.gz: 65a2d11400a0604631eed4cb6dc7df62307b5bd9914a3d768a3bff3316960a11
 SHA512:
-  metadata.gz: 64e0d567808840b2672055500fa190e998fd66a08a8459e7456aaa3b3f417197ba4a763f36493a31513060384fb0b0a67bd3686865d503cac4ca6d707189b3eb
-  data.tar.gz: 72767efc1dfefeb4db2632dab59a9e098e7492475451f90589b5bbee54fcc93455088113ae37e3f514e005aa35e216a8e66bc22241e295bd1f1370d58c92d47c
+  metadata.gz: 52295ede507181b2b4f6a738131b3f279845076010421364e22aec501660a7e9ad442fda17723ec9f39f6247ee6f6d371ba0a32cafd8589404d8ce074ec40b1a
+  data.tar.gz: 86e482ce416b01185b3e4eed3bd3edaac850155c9d3353d0108cf5a14429329847ff1d741a3319faebcfd9fca22830569bafb87de7031c59b58fd01e2d97bb40

data/CHANGELOG.md

@@ -1,3 +1,10 @@
+# 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

data/Gemfile.lock

@@ -1,36 +1,36 @@
 PATH
   remote: .
   specs:
-    rspec-multiprocess_runner (1.4.1.pre)
+    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/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
@@ -221,6 +221,7 @@ module RSpec::MultiprocessRunner
         print_missing_files(output)
         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"
@@ -233,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
@@ -290,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
@@ -297,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
@@ -311,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

@@ -70,7 +70,7 @@ module RSpec::MultiprocessRunner
         else
           return nil # Malformed response, assume done, cease function
         end
-      rescue Exception => e
+      rescue StandardError => e
         puts("Got exception #{e} in get_file")
         return nil # If Error, assume done, cease function
       end

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.4.1"
+    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
 
@@ -281,8 +294,9 @@ module RSpec::MultiprocessRunner
       RSpec.world.example_groups.clear
 
       ReportingFormatter.worker = self
-      RSpec::Core::Runner.run(@rspec_arguments + [spec_file])
-      raise "Error outside of tests on #{spec_file}" if RSpec.world.non_example_failure
+      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
@@ -323,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
@@ -332,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.4.1
+  version: 1.4.2
 platform: ruby
 authors:
 - Rhett Sutphin
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-12-14 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,7 +131,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.6
+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