From fad593ddb881d02e25b1b9d2485685f38d64f866 Mon Sep 17 00:00:00 2001
From: Benjamin Quorning <benjamin@quorning.net>
Date: Thu, 25 May 2023 22:01:14 +0200
Subject: [PATCH] Remove compatibility with RSpec 2
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

RSpec 3 was released 9 years ago. Most people will have upgraded by now 😅
---
 CHANGELOG.md                                   |  2 ++
 lib/parallel_tests/rspec/failures_logger.rb    | 18 +++++-------------
 lib/parallel_tests/rspec/logger_base.rb        |  2 --
 lib/parallel_tests/rspec/runtime_logger.rb     |  5 ++---
 lib/parallel_tests/rspec/summary_logger.rb     |  2 +-
 .../rspec/runtime_logger_spec.rb               |  2 +-
 6 files changed, 11 insertions(+), 20 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index e74fbd3e..6832c309 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -4,6 +4,8 @@
 
 ### Breaking Changes
 
+- Drop support for RSpec 2.
+
 ### Added
 
 ### Fixed
diff --git a/lib/parallel_tests/rspec/failures_logger.rb b/lib/parallel_tests/rspec/failures_logger.rb
index 7047d43d..3c979f69 100644
--- a/lib/parallel_tests/rspec/failures_logger.rb
+++ b/lib/parallel_tests/rspec/failures_logger.rb
@@ -3,22 +3,14 @@
 require 'parallel_tests/rspec/runner'
 
 class ParallelTests::RSpec::FailuresLogger < ParallelTests::RSpec::LoggerBase
-  if RSPEC_2
-    def dump_failures(*args); end
-  else
-    RSpec::Core::Formatters.register self, :dump_summary
-  end
+  RSpec::Core::Formatters.register(self, :dump_summary)
 
   def dump_summary(*args)
     lock_output do
-      if RSPEC_2
-        dump_commands_to_rerun_failed_examples
-      else
-        notification = args.first
-        unless notification.failed_examples.empty?
-          colorizer = ::RSpec::Core::Formatters::ConsoleCodes
-          output.puts notification.colorized_rerun_commands(colorizer)
-        end
+      notification = args.first
+      unless notification.failed_examples.empty?
+        colorizer = ::RSpec::Core::Formatters::ConsoleCodes
+        output.puts notification.colorized_rerun_commands(colorizer)
       end
     end
     @output.flush
diff --git a/lib/parallel_tests/rspec/logger_base.rb b/lib/parallel_tests/rspec/logger_base.rb
index ab1356bb..4b3b1bff 100644
--- a/lib/parallel_tests/rspec/logger_base.rb
+++ b/lib/parallel_tests/rspec/logger_base.rb
@@ -7,8 +7,6 @@ module RSpec
 require 'rspec/core/formatters/base_text_formatter'
 
 class ParallelTests::RSpec::LoggerBase < RSpec::Core::Formatters::BaseTextFormatter
-  RSPEC_2 = RSpec::Core::Version::STRING.start_with?('2')
-
   def initialize(*args)
     super
 
diff --git a/lib/parallel_tests/rspec/runtime_logger.rb b/lib/parallel_tests/rspec/runtime_logger.rb
index 14f9b883..bfcccfa1 100644
--- a/lib/parallel_tests/rspec/runtime_logger.rb
+++ b/lib/parallel_tests/rspec/runtime_logger.rb
@@ -9,7 +9,7 @@ def initialize(*args)
     @group_nesting = 0
   end
 
-  RSpec::Core::Formatters.register self, :example_group_started, :example_group_finished, :start_dump unless RSPEC_2
+  RSpec::Core::Formatters.register(self, :example_group_started, :example_group_finished, :start_dump)
 
   def example_group_started(example_group)
     @time = ParallelTests.now if @group_nesting == 0
@@ -20,8 +20,7 @@ def example_group_started(example_group)
   def example_group_finished(notification)
     @group_nesting -= 1
     if @group_nesting == 0
-      path = (RSPEC_2 ? notification.file_path : notification.group.file_path)
-      @example_times[path] += ParallelTests.now - @time
+      @example_times[notification.group.file_path] += ParallelTests.now - @time
     end
     super if defined?(super)
   end
diff --git a/lib/parallel_tests/rspec/summary_logger.rb b/lib/parallel_tests/rspec/summary_logger.rb
index 339a6013..8b30cd6c 100644
--- a/lib/parallel_tests/rspec/summary_logger.rb
+++ b/lib/parallel_tests/rspec/summary_logger.rb
@@ -2,7 +2,7 @@
 require 'parallel_tests/rspec/failures_logger'
 
 class ParallelTests::RSpec::SummaryLogger < ParallelTests::RSpec::LoggerBase
-  RSpec::Core::Formatters.register self, :dump_failures unless RSPEC_2
+  RSpec::Core::Formatters.register(self, :dump_failures)
 
   def dump_failures(*args)
     lock_output { super }
diff --git a/spec/parallel_tests/rspec/runtime_logger_spec.rb b/spec/parallel_tests/rspec/runtime_logger_spec.rb
index 23f62e5c..ec100080 100644
--- a/spec/parallel_tests/rspec/runtime_logger_spec.rb
+++ b/spec/parallel_tests/rspec/runtime_logger_spec.rb
@@ -19,7 +19,7 @@ def log_for_a_file(_options = {})
       end
 
       example = double(file_path: "#{Dir.pwd}/spec/foo.rb")
-      example = double(group: example) unless ParallelTests::RSpec::RuntimeLogger::RSPEC_2
+      example = double(group: example)
 
       logger.example_group_started example
       logger.example_group_finished example