parallel_rspec 2.1.2 → 2.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 73b85e313868cc3fb29697f15b99911c17a6ba818c80723c05ca6cf2cb450b00
-  data.tar.gz: a86b018f8fe152d0bf4203f3f076eb5e08583b65bbe4dd390220869a3292fb12
+  metadata.gz: 6d2e2199456f48d4b9726672b0c5e223b536350826d46b1bc14ff28e5d83ef45
+  data.tar.gz: ae4b625e1041147f8e6c4087196fe34bb97977fb0034549e9436b4056363f184
 SHA512:
-  metadata.gz: 8fa15fcdb155ba22163b604b8d2abaa7ba003e2b1c1169dd5565399b29757c714d7362626959024c26e32068fbf6974590533d0b37d7f55f7fd329d8d06755a9
-  data.tar.gz: 895f4135c2603f10a6e721c36d15cd0c0b9b7ba212e4f8b577fdc923030479332bcb55799bf12e78332649e6e6cecbca1946a7181416a567fffde27cfad50be4
+  metadata.gz: a27c85ac8cc1ef08b04ac61054848eeed0dfbe08465fd4814d0a6909dd18891df71d7bbe9011f2fe0281a028bdd62550988d2716f707d3ea1a11d12139d67772
+  data.tar.gz: 584531e1f3027132097cebb1a36e600ae7e470ce62b91bc23522352dc39f14a7c1c80b34b92a1c5298bbdb079fb51c1204962945577c6cdf549e14dd944f8a22

data/CHANGES.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 2.2.0
+
+* Fix incompatibility with `rspec --profile`.
+
 ## 2.1.2
 
 * Fix error when used in projects without ActiveSupport. Thanks @erikpaasonen.

data/README.md

@@ -24,7 +24,7 @@ And then execute:
 
     $ bundle
 
-This version of ParallelRSpec has been tested with RSpec 3.10.
+This version of ParallelRSpec has been tested with RSpec 3.12.
 
 ## Usage
 
@@ -82,6 +82,10 @@ You might also want to detect whether your spec suite is running under ParallelR
   do_something unless ParallelRSpec.running?
 ```
 
+## Limitations
+
+ParallelRSpec can't parallelize the specs in groups that use `before(:context)` or `after(:context)` (AKA `before(:all)` or `after(:all)`) hooks, since these may mean that there is state shared between examples.
+
 ## Contributing
 
 Bug reports and pull requests are welcome on GitHub at https://github.com/willbryant/parallel_rspec.

data/lib/parallel_rspec/client.rb

@@ -15,52 +15,38 @@ module ParallelRSpec
     end
 
     def example_started(example)
-      channel_to_server.write([:example_started, serialize_example(example)])
+      channel_to_server.write([:example_started, example.id, updates_from(example)])
     end
 
     def example_passed(example)
-      channel_to_server.write([:example_passed, serialize_example(example)])
+      channel_to_server.write([:example_passed, example.id, updates_from(example)])
     end
 
     def example_failed(example)
-      channel_to_server.write([:example_failed, serialize_example(example)])
+      channel_to_server.write([:example_failed, example.id, updates_from(example)])
     end
 
     def example_finished(example)
-      channel_to_server.write([:example_finished, serialize_example(example)])
+      channel_to_server.write([:example_finished, example.id, updates_from(example)])
     end
 
     def example_pending(example)
-      channel_to_server.write([:example_pending, serialize_example(example)])
+      channel_to_server.write([:example_pending, example.id, updates_from(example)])
     end
 
     def deprecation(hash)
       channel_to_server.write([:deprecation, hash])
     end
 
-    def serialize_example(example)
-      Example.new(
-        example.id,
-        example.description,
-        example.exception,
-        example.location_rerun_argument,
-        ExampleGroup.new([]),
-        example.metadata.slice(
-          :absolute_file_path,
-          :described_class,
-          :description,
-          :description_args,
+    def updates_from(example)
+      {
+        exception: example.exception,
+        metadata: example.metadata.slice(
           :execution_result,
-          :full_description,
-          :file_path,
-          :last_run_status,
-          :line_number,
-          :location,
           :pending,
-          :rerun_file_path,
-          :scoped_id,
-          :shared_group_inclusion_backtrace,
-          :type))
+          :skip,
+        )
+      }
     end
 
     def next_example_to_run

data/lib/parallel_rspec/runner.rb

@@ -58,7 +58,6 @@ module ParallelRSpec
     #   or the configured failure exit code (1 by default) if specs
     #   failed.
     def run_specs(example_groups)
-      puts "running #{@world.example_count(example_groups)}"
       @configuration.reporter.report(@world.example_count(example_groups)) do |reporter|
         @configuration.with_suite_hooks do
           with_context_hooks, without_context_hooks = example_groups.partition(&:any_context_hooks?)

data/lib/parallel_rspec/server.rb

@@ -1,34 +1,34 @@
 module ParallelRSpec
   class Server
-    attr_reader :remaining_example_group_indexes, :reporter
+    attr_reader :reporter, :remaining_examples_by_group, :running_examples
 
     def initialize(reporter)
-      @remaining_example_group_indexes = RSpec.world.filtered_examples.each_with_object({}) do |(example_group, examples), results|
-        results[example_group] = examples.size unless example_group.any_context_hooks? || examples.size.zero?
+      @remaining_examples_by_group = RSpec.world.filtered_examples.each_with_object({}) do |(example_group, examples), results|
+        results[example_group] = examples unless example_group.any_context_hooks? || examples.size.zero?
       end
       @reporter = reporter
       @success = true
+      @running_examples = {}
     end
 
-    def example_started(example, channel_to_client)
-      reporter.example_started(example)
+    def example_started(example_id, example_updates, channel_to_client)
+      reporter.example_started(update_example(running_examples[example_id], example_updates))
     end
 
-    def example_passed(example, channel_to_client)
-      reporter.example_passed(example)
+    def example_finished(example_id, example_updates, channel_to_client)
+      reporter.example_finished(update_example(running_examples[example_id], example_updates))
     end
 
-    def example_failed(example, channel_to_client)
-      reporter.example_failed(example)
+    def example_passed(example_id, example_updates, channel_to_client)
+      reporter.example_passed(update_example(running_examples.delete(example_id), example_updates))
     end
 
-    def example_finished(example, channel_to_client)
-      pp example
-      reporter.example_finished(example)
+    def example_failed(example_id, example_updates, channel_to_client)
+      reporter.example_failed(update_example(running_examples.delete(example_id), example_updates))
     end
 
-    def example_pending(example, channel_to_client)
-      reporter.example_pending(example)
+    def example_pending(example_id, example_updates, channel_to_client)
+      reporter.example_pending(update_example(running_examples.delete(example_id), example_updates))
     end
 
     def deprecation(hash, channel_to_client)
@@ -36,13 +36,14 @@ module ParallelRSpec
     end
 
     def next_example_to_run(channel_to_client)
-      if remaining_example_group_indexes.empty?
+      if remaining_examples_by_group.empty?
         channel_to_client.write(nil)
       else
-        klass = remaining_example_group_indexes.keys.first
-        remaining_example_group_indexes[klass] -= 1
-        channel_to_client.write([klass, remaining_example_group_indexes[klass]])
-        remaining_example_group_indexes.delete(klass) if remaining_example_group_indexes[klass].zero?
+        example_group = remaining_examples_by_group.keys.first
+        example = remaining_examples_by_group[example_group].pop
+        running_examples[example.id] = example # cache so we don't need to look through all the examples for each message
+        channel_to_client.write([example_group, remaining_examples_by_group[example_group].size])
+        remaining_examples_by_group.delete(example_group) if remaining_examples_by_group[example_group].empty?
       end
     end
 
@@ -53,5 +54,11 @@ module ParallelRSpec
     def success?
       @success
     end
+
+    def update_example(example, data)
+      example.set_exception(data[:exception])
+      example.metadata.merge!(data[:metadata])
+      example
+    end
   end
 end

data/lib/parallel_rspec/version.rb

@@ -1,3 +1,3 @@
 module ParallelRSpec
-  VERSION = "2.1.2"
+  VERSION = "2.2.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: parallel_rspec
 version: !ruby/object:Gem::Version
-  version: 2.1.2
+  version: 2.2.0
 platform: ruby
 authors:
 - Will Bryant, Powershop New Zealand Ltd