Merge pull request #2166 from rspec/remove-logged-messages

Remove logged messages

Author: JonRowe

lib/rspec/core/runner.rb

@@ -184,7 +184,7 @@ def self.handle_interrupt
           exit!(1)
         else
           RSpec.world.wants_to_quit = true
-          STDERR.puts "\nRSpec is shutting down and will print the summary report... Interrupt again to force quit."
+          $stderr.puts "\nRSpec is shutting down and will print the summary report... Interrupt again to force quit."
         end
       end
     end

spec/rspec/core/runner_spec.rb

@@ -115,7 +115,7 @@ module RSpec::Core
       let(:interrupt_handlers) { [] }
 
       before do
-        allow(Object).to receive(:trap).with("INT", any_args) do |&block|
+        allow(Runner).to receive(:trap).with("INT", any_args) do |&block|
           interrupt_handlers << block
         end
       end
@@ -137,19 +137,20 @@ def interrupt
           interrupt
         end
 
-        it "does not exit immediately" do
+        it "does not exit immediately, but notifies the user" do
           Runner.send(:trap_interrupt)
-          expect_any_instance_of(Object).not_to receive(:exit)
-          expect_any_instance_of(Object).not_to receive(:exit!)
-          interrupt
+          expect(Runner).not_to receive(:exit)
+          expect(Runner).not_to receive(:exit!)
+
+          expect { interrupt }.to output(/RSpec is shutting down/).to_stderr
         end
       end
 
       context "with SIGINT twice" do
         it "exits immediately" do
           Runner.send(:trap_interrupt)
-          expect_any_instance_of(Object).to receive(:exit!).with(1)
-          interrupt
+          expect(Runner).to receive(:exit!).with(1)
+          expect { interrupt }.to output(//).to_stderr
           interrupt
         end
       end