mutant-rspec 0.16.3 → 0.17.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1ed162d6c71157f2c6c5c9758b2e56616aeddb60cdf245cbf7928a20a9872efe
-  data.tar.gz: 053f85345daf93b3a9fb2be5986ebe96b1de88711af244dcfe6f5709689117d3
+  metadata.gz: '048a5d77aaec0aeb8cb8a3498b065a6d7e1fbc3691fc2913185a3c21698e98b7'
+  data.tar.gz: ae4405f217208163e6bf2822e5ebe313b8dde0a73989b1db3f674712439cc5ff
 SHA512:
-  metadata.gz: 679b303dbd729051c2b15665905a8f9ef6bcf3bf6d4780cb9e94cda033be4f9baab044f30fe710e33ced8db320a8e0c20d59fd3df2b951ed16efc203e4daba84
-  data.tar.gz: f300869bec5ad54080f693c586c62dbb4526130b59f4c8157702d7e97a393a32c41f85d4a16667a0f338a5b94bc0e72c4106d1c6edab290b5bf652cb50991319
+  metadata.gz: a2e47586585d3a592e0e3abd44056d6d454c81ef78eb9c192f9ecc44b4f118232815db1d3711bd8ecd7d3a007f65993892090b3ba4c769f77cf9609025912070
+  data.tar.gz: 3ee67f84ca7049f5bd6eda3e2abc7f055cdeeabf489c7378e830cef59249e98302e06114f60a01f50772ee114b7657d9a64c923d6d1b3725d53bc03ca4859e7e

data/lib/mutant/integration/rspec.rb

@@ -29,6 +29,50 @@ module Mutant
 
       private_constant(*constants(false))
 
+      # Rspec ordering strategy driven by the order mutant selected the tests in
+      #
+      # Rspec runs one group at a time, so a group is ranked by the best test it
+      # holds, and the ranking survives as group order plus example order within
+      # a group. Anything rspec asks about that was not selected sorts last.
+      class Ordering
+        UNRANKED = Float::INFINITY
+
+        private_constant(*constants(false))
+
+        def initialize = @rank = {}
+
+        # Rank examples, best first
+        #
+        # @param [Enumerable<RSpec::Core::Example>] examples
+        #
+        # @return [self]
+        def call(examples)
+          @rank.clear
+
+          examples.each_with_index do |example, rank|
+            @rank[example] = rank
+
+            example.example_group.parent_groups.each do |group|
+              @rank[group] = rank if @rank.fetch(group, UNRANKED) > rank
+            end
+          end
+
+          self
+        end
+
+        # Order the groups or the examples rspec is about to run
+        #
+        # Rspec's own id breaks a tie, so the order does not depend on the order
+        # rspec happened to hand the list over in.
+        #
+        # @param [Array<Object>] list
+        #
+        # @return [Array<Object>]
+        def order(list)
+          list.sort_by { |item| [@rank.fetch(item, UNRANKED), item.id] }
+        end
+      end # Ordering
+
       def freeze = tap { super if @setup_elapsed }
 
       # Initialize rspec integration
@@ -36,6 +80,7 @@ module Mutant
       # @return [undefined]
       def initialize(*)
         super
+        @ordering    = Ordering.new
         @runner      = RSpec::Core::Runner.new(RSpec::Core::ConfigurationOptions.new(effective_arguments))
         @rspec_world = RSpec.world
       end
@@ -56,7 +101,7 @@ module Mutant
       end
       memoize :setup
 
-      # Run a collection of tests
+      # Run a collection of tests, likeliest killer first
       #
       # @param [Enumerable<Mutant::Test>] tests
       #
@@ -65,8 +110,12 @@ module Mutant
       # rubocop:disable Metrics/AbcSize
       # rubocop:disable Metrics/MethodLength
       def call(tests)
+        examples = tests.map(&all_tests_index)
+
+        install_ordering
         reset_examples
-        setup_examples(tests.map(&all_tests_index))
+        @ordering.call(examples)
+        setup_examples(examples)
         @runner.configuration.start_time = world.time.now - @setup_elapsed
         start = timer.now
         passed = @runner.run_specs(@rspec_world.ordered_example_groups).equal?(EXIT_SUCCESS)
@@ -96,6 +145,18 @@ module Mutant
 
     private
 
+      # Rspec picks its own order for groups and for the examples inside them,
+      # which would throw away the selector's ranking. Its ordering registry is
+      # the seam for that, but the public `register_ordering(:global)` is a noop
+      # once `--order` was forced from the command line or `.rspec`, which most
+      # suites do. Registering with the registry is the only way in.
+      #
+      # Registered per run rather than at setup: the strategy is the same object
+      # every time, and a run is the moment its ranking is known.
+      def install_ordering
+        @runner.configuration.ordering_registry.register(:global, @ordering)
+      end
+
       def rspec_setup_failure?
         @rspec_world.wants_to_quit || rspec_is_quitting?
       end
@@ -124,16 +185,19 @@ module Mutant
       def parse_example(example, index)
         metadata = example.metadata
 
-        id = TEST_ID_FORMAT % {
+        Test.new(
+          expressions: parse_metadata(metadata),
+          id:          test_id(metadata, index),
+          location:    metadata.fetch(:location)
+        )
+      end
+
+      def test_id(metadata, index)
+        TEST_ID_FORMAT % {
           index:,
           location:    metadata.fetch(:location),
           description: metadata.fetch(:full_description)
         }
-
-        Test.new(
-          expressions: parse_metadata(metadata),
-          id:
-        )
       end
 
       def example_group_map

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mutant-rspec
 version: !ruby/object:Gem::Version
-  version: 0.16.3
+  version: 0.17.0
 platform: ruby
 authors:
 - Markus Schirp
@@ -15,14 +15,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.16.3
+        version: 0.17.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.16.3
+        version: 0.17.0
 - !ruby/object:Gem::Dependency
   name: rspec-core
   requirement: !ruby/object:Gem::Requirement