Merge pull request #2532 from garybernhardt/dont_write_status_file_during_dry_runs

Don't write status file during dry runs.

Author: myronmarston

lib/rspec/core/runner.rb

@@ -184,6 +184,7 @@ def self.handle_interrupt
     private
 
       def persist_example_statuses
+        return if @configuration.dry_run
         return unless (path = @configuration.example_status_persistence_file_path)
 
         ExampleStatusPersister.persist(@world.all_examples, path)

spec/rspec/core/runner_spec.rb

@@ -406,6 +406,15 @@ def run
             end
           end
 
+          context "with --dry-run" do
+            it "doesn't persist example status" do
+              config.example_status_persistence_file_path = "examples.txt"
+              config.dry_run = true
+              run
+              expect(ExampleStatusPersister).not_to have_received(:persist)
+            end
+          end
+
           context "when `example_status_persistence_file_path` is not configured" do
             it "doesn't persist example status" do
               config.example_status_persistence_file_path = nil