Default example status to unknown if corrupt

lib/rspec/core/configuration.rb

@@ -980,7 +980,9 @@ def last_run_statuses
           if (path = example_status_persistence_file_path)
             begin
               ExampleStatusPersister.load_from(path).inject(statuses) do |hash, example|
-                hash[example.fetch(:example_id)] = example.fetch(:status)
+                status = example[:status]
+                status = UNKNOWN_STATUS unless VALID_STATUSES.include?(status)
+                hash[example.fetch(:example_id)] = status
                 hash
               end
             rescue SystemCallError => e
@@ -1000,6 +1002,12 @@ def last_run_statuses
       # @private
       FAILED_STATUS = "failed".freeze
 
+      # @private
+      PASSED_STATUS = "passed".freeze
+
+      # @private
+      VALID_STATUSES = [UNKNOWN_STATUS, FAILED_STATUS, PASSED_STATUS]
+
       # @private
       def spec_files_with_failures
         @spec_files_with_failures ||= last_run_statuses.inject(Set.new) do |files, (id, status)|

spec/rspec/core/configuration/only_failures_support_spec.rb

@@ -110,6 +110,44 @@ def spec_files_with_failures
         end
       end
 
+      context 'when the file at `example_status_persistence_file_path` is corrupt' do
+        before do
+          simulate_persisted_examples(
+            { :example_id => "./spec_1.rb[1:1]" },
+            { :example_id => "./spec_1.rb[1:2]", :status => ""  },
+            { :example_id => "./spec_2.rb[1:2]", :status => nil },
+            { :example_id => "./spec_3.rb[1:2]", :status => "wrong" },
+            { :example_id => "./spec_4.rb[1:2]", :status => "unknown" },
+            :example_id => "./spec_5.rb[1:2]", :status => "failed"
+          )
+        end
+
+        it 'should not crash' do
+          Array.new(9) do |i|
+            RSpec.describe("group") { it "example #{i}" do; end }
+          end
+          expectation = expect do
+            RSpec.world.example_groups
+          end
+          expectation.not_to raise_error
+        end
+        it 'should default invalid statuses to unknown' do
+          expect(spec_files_with_failures).to(
+            be_an(Array) &
+            contain_exactly("./spec_5.rb")
+          )
+          # Check that each example has the correct status
+          expect(config.last_run_statuses).to eq(
+            './spec_1.rb[1:1]' => 'unknown',
+            './spec_1.rb[1:2]' => 'unknown',
+            './spec_2.rb[1:2]' => 'unknown',
+            './spec_3.rb[1:2]' => 'unknown',
+            './spec_4.rb[1:2]' => 'unknown',
+            './spec_5.rb[1:2]' => 'failed'
+          )
+        end
+      end
+
       context "when `example_status_persistence_file_path` is not configured" do
         it "returns a memoized blank array" do
           config.example_status_persistence_file_path = nil

spec/rspec/core/configuration_spec.rb

@@ -2,7 +2,6 @@
 require 'rspec/support/spec/in_sub_process'
 
 module RSpec::Core
-
   RSpec.describe Configuration do
     include RSpec::Support::InSubProcess