selective-ruby-rspec 0.1.4 → 0.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 357a3e88c5faad8cc7d188615b158094058f55b9342977bc999c14cfcdc82980
-  data.tar.gz: 6afe356356bd83ae83cce8267a3d98446489621dee5960eda86772bd29861273
+  metadata.gz: ab16214c977e214c1768f9151be4d4eb3f89f4400279c1e269947b353792981d
+  data.tar.gz: b19ed802819f65223e9ef8a2f262a1fd7c0d3baa70329d6e1de8fe9d5f3b7981
 SHA512:
-  metadata.gz: 47046465fbc086af40e51f7bf74e8ffd65c4d2b5c92ba914eff1e6a567e8cc005abad6b4f8e5e08492f0e35d076afd2c1f1cb3faa50f05ced435c72e5a910518
-  data.tar.gz: d7013aba148b15d7a9f7576cf8e99f7497cf1860610d32ad6d5a6d802903862152921fa3fe8b8104767320e7012606afb6d0f758ab887fd8ac59c26293c57f13
+  metadata.gz: d4ba21e4eee2ded289e1d2448401b8078c10eddecb4fa6acaf489954becd0a0166d17be966dce11d96489d63fb8da890c99e1fdb961d2b18853801d536167b02
+  data.tar.gz: 4f585e62586e20423913e0ad0940dc8c13901de771876de44766b9eb705c58e193ad87083d4df5c56be90bc6814722d6337860c99e20d46e58fa74ebc978d45e

data/.semaphore/semaphore.yml

@@ -0,0 +1,19 @@
+version: v1.0
+name: Ruby
+agent:
+  machine:
+    type: e1-standard-2
+    os_image: ubuntu2004
+blocks:
+  - name: Run RSpec tests
+    task:
+      jobs:
+        - name: Run RSpec tests
+          parallelism: 2
+          commands:
+            - sem-version ruby 3.3.0
+            - checkout
+            - bundle install
+            - bundle exec selective rspec
+      secrets:
+        - name: Selective

data/lib/selective/ruby/rspec/formatter.rb

@@ -0,0 +1,28 @@
+module Selective
+  module Ruby
+    module RSpec
+      class Formatter
+        ::RSpec::Core::Formatters.register self, :example_passed, :example_failed, :example_pending
+
+        def initialize(...); end
+
+        def self.runner_wrapper=(runner_wrapper)
+          @runner_wrapper = runner_wrapper
+        end
+
+        def self.runner_wrapper
+          @runner_wrapper
+        end
+        
+        %i(example_passed example_failed example_pending).each do |method|
+          define_method(method) do |notification|
+            self.class.runner_wrapper.report_example(notification.example)
+          rescue Selective::Ruby::Core::ConnectionLostError
+            ::RSpec.world.wants_to_quit = true
+            self.class.runner_wrapper.remove_test_case_result(notification.example.id)
+          end
+        end
+      end
+    end
+  end
+end

data/lib/selective/ruby/rspec/monkeypatches.rb

@@ -97,6 +97,16 @@ module Selective
             super
             @example_map = {}
           end
+
+          def prepare_example_filtering
+            @filtered_examples = Hash.new do |hash, group|
+              hash[group] = if ::RSpec.configuration.dry_run?
+                filter_manager.prune(group.examples)
+              else
+                []
+              end
+            end
+          end
         end
 
         module Runner
@@ -106,7 +116,7 @@ module Selective
         module Example
           def initialize(*args)
             super
-            ::RSpec.world.example_map[id] = example_group.parent_groups.last
+            ::RSpec.world.example_map[id] = self
           end
 
           def run(*args)

data/lib/selective/ruby/rspec/runner_wrapper.rb

@@ -7,12 +7,13 @@ module Selective
       class RunnerWrapper
         class TestManifestError < StandardError; end
 
-        attr_reader :rspec_runner, :config
+        attr_reader :rspec_runner, :config, :example_callback
 
         FRAMEWORK = "rspec"
         DEFAULT_SPEC_PATH = "./spec"
 
-        def initialize(args)
+        def initialize(args, example_callback)
+          @example_callback = example_callback
           rspec_args, wrapper_config_hash = parse_args(args)
           Selective::Ruby::RSpec::Monkeypatches.apply(wrapper_config_hash)
           apply_rspec_configuration
@@ -22,27 +23,12 @@ module Selective
             config.options[:files_or_directories_to_run] = [DEFAULT_SPEC_PATH]
           end
 
+          Formatter.runner_wrapper = self
+
           @rspec_runner = ::RSpec::Core::Runner.new(@config)
           @rspec_runner.setup($stderr, $stdout)
         end
 
-        def parse_args(args)
-          supported_wrapper_args = %w[--require-each-hooks]
-          wrapper_args, rspec_args = args.partition do |arg|
-            supported_wrapper_args.any? do |p|
-              arg.start_with?(p)
-            end
-          end
-
-          wrapper_config_hash = wrapper_args.each_with_object({}) do |arg, hash|
-            key = arg.sub('--', '').tr('-', '_').to_sym
-            hash[key] = true
-          end
-
-          rspec_args << "--format=progress" unless args.any? { |e| e.start_with?("--format") }
-          [rspec_args, wrapper_config_hash]
-        end
-
         def manifest
           output = nil
           Tempfile.create("selective-rspec-dry-run") do |f|
@@ -59,19 +45,17 @@ module Selective
           raise_test_manifest_error(e.message)
         end
 
-        def run_test_cases(test_case_ids, callback)
-          ::RSpec.world.reporter.send(:start, nil)
-          Selective::Ruby::Core::Controller.suppress_reporting!
-          test_case_ids.flatten.each do |test_case_id|
-            run_test_case(test_case_id, callback)
-          end
+        def run_test_cases(test_case_ids)
+          ensure_test_phase
+          configure(test_case_ids)
+          rspec_runner.run_specs(optimize_test_filtering(test_case_ids).to_a)
         end
 
         def exec
           rspec_runner.run($stderr, $stdout)
         end
 
-        def remove_failed_test_case_result(test_case_id)
+        def remove_test_case_result(test_case_id)
           failure = ::RSpec.world.reporter.failed_examples.detect { |e| e.id == test_case_id }
           if (failed_example_index = ::RSpec.world.reporter.failed_examples.index(failure))
             ::RSpec.world.reporter.failed_examples.delete_at(failed_example_index)
@@ -106,18 +90,27 @@ module Selective
           RSpec::VERSION
         end
 
+        def report_example(example)
+          example_callback.call(format_example(example))
+        end
+
         private
 
-        def run_test_case(test_case_id, callback)
-          ::RSpec.configuration.reset_filters
-          ::RSpec.world.prepare_example_filtering
-          config.options[:files_or_directories_to_run] = test_case_id
-          rspec_runner.options = config
-          rspec_runner.setup($stderr, $stdout)
+        def parse_args(args)
+          supported_wrapper_args = %w[--require-each-hooks]
+          wrapper_args, rspec_args = args.partition do |arg|
+            supported_wrapper_args.any? do |p|
+              arg.start_with?(p)
+            end
+          end
 
-          # $rspec_rerun_debug = true if test_case == './spec/selective_spec.rb[1:2]' && ::RSpec.world.reporter.examples.length == 5
-          rspec_runner.run_specs([::RSpec.world.example_map[test_case_id]])
-          callback.call(format_example(get_example_from_reporter(test_case_id)))
+          wrapper_config_hash = wrapper_args.each_with_object({}) do |arg, hash|
+            key = arg.sub('--', '').tr('-', '_').to_sym
+            hash[key] = true
+          end
+
+          rspec_args << "--format=progress" unless args.any? { |e| e.start_with?("--format") }
+          [rspec_args, wrapper_config_hash]
         end
 
         def get_example_from_reporter(test_case_id)
@@ -161,6 +154,42 @@ module Selective
         def raise_test_manifest_error(output)
           raise TestManifestError.new("Selective could not generate a test manifest. The output was:\n#{output}")
         end
+
+        def configure(test_case_ids)
+          ::RSpec.world.wants_to_quit = false
+          ::RSpec.configuration.reset_filters
+          ::RSpec.world.filtered_examples.clear
+
+          config.options[:files_or_directories_to_run] = test_case_ids
+          rspec_runner.configure($stderr, $stdout)
+          ::RSpec.configuration.load_spec_files
+        end
+
+        def optimize_test_filtering(test_case_ids)
+          # The following is an optimization to avoid RSpec's usual filtering mechanism
+          # which has a significant performance overhead.
+          test_case_ids.each_with_object(Set.new) do |test_case_id, set|
+            example = ::RSpec.world.example_map[test_case_id]
+            if ::RSpec.world.filtered_examples.key?(example.example_group)
+              ::RSpec.world.filtered_examples[example.example_group] << example
+            else
+              ::RSpec.world.filtered_examples[example.example_group] = [example]
+            end
+            set << example.example_group.parent_groups.last
+          end
+        end
+
+        def ensure_test_phase
+          @test_phase_initialized ||= begin
+            ::RSpec.world.reporter.send(:start, nil)
+            Selective::Ruby::Core::Controller.suppress_reporting!
+            apply_formatter
+          end
+        end
+
+        def apply_formatter
+          config.options[:formatters] << [Selective::Ruby::RSpec::Formatter.to_s]
+        end
       end
     end
   end

data/lib/selective/ruby/rspec/version.rb

@@ -3,7 +3,7 @@
 module Selective
   module Ruby
     module RSpec
-      VERSION = "0.1.4"
+      VERSION = "0.1.6"
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: selective-ruby-rspec
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.1.6
 platform: ruby
 authors:
 - Benjamin Wood
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2024-01-04 00:00:00.000000000 Z
+date: 2024-02-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: zeitwerk
@@ -31,14 +31,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.2.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.2.3
 description: Selective is an intelligent test runner for your current CI provider.
   Get real-time test results, intelligent ordering based on code changes, shorter
   run times, automatic flake detection, the ability to re-enqueue failed tests, and
@@ -50,9 +50,11 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".semaphore/semaphore.yml"
 - LICENSE
 - Rakefile
 - lib/selective-ruby-rspec.rb
+- lib/selective/ruby/rspec/formatter.rb
 - lib/selective/ruby/rspec/monkeypatches.rb
 - lib/selective/ruby/rspec/runner_wrapper.rb
 - lib/selective/ruby/rspec/version.rb