Simplify BisectFormatter using example_finished notification.

We did not offer it when the BisectFormatter was written but it
nicely simplifies it now.

Author: myronmarston

lib/rspec/core/formatters/bisect_formatter.rb

@@ -13,7 +13,7 @@ module Formatters
       # @private
       class BisectFormatter
         Formatters.register self, :start, :start_dump, :example_started,
-                            :example_failed, :example_passed, :example_pending
+                            :example_failed, :example_finished
 
         def initialize(_output)
           port = RSpec.configuration.drb_port
@@ -35,15 +35,15 @@ def example_started(notification)
 
         def example_failed(notification)
           @failed_example_ids << notification.example.id
-          example_finished(notification, :failed)
         end
 
-        def example_passed(notification)
-          example_finished(notification, :passed)
-        end
+        def example_finished(notification)
+          return unless @remaining_failures.include?(notification.example.id)
+          @remaining_failures.delete(notification.example.id)
 
-        def example_pending(notification)
-          example_finished(notification, :pending)
+          status = notification.example.execution_result.status
+          return if status == :failed && !@remaining_failures.empty?
+          RSpec.world.wants_to_quit = true
         end
 
         def start_dump(_notification)
@@ -53,16 +53,6 @@ def start_dump(_notification)
         end
 
         RunResults = Struct.new(:all_example_ids, :failed_example_ids)
-
-      private
-
-        def example_finished(notification, status)
-          return unless @remaining_failures.include?(notification.example.id)
-          @remaining_failures.delete(notification.example.id)
-
-          return if status == :failed && !@remaining_failures.empty?
-          RSpec.world.wants_to_quit = true
-        end
       end
     end
   end