@@ -80,6 +80,7 @@ module Integrity begin notifier.notify_of_build last_build rescue Timeout::Error + Integrity.logger.info "#{notifier.name} notifier timed out" next end end lib/integrity/project.rb @@ -273,10 +273,17 @@ describe Integrity::Project do end it "should protect itself from eventual timeout error" do + Integrity.logger.stub!(:info) @email_notifier.stub!(:notify_of_build).and_raise(Timeout::Error) lambda do @project.send(:send_notifications) end.should_not raise_error end + + it "should log timeout" do + @email_notifier.stub!(:notify_of_build).and_raise(Timeout::Error) + Integrity.logger.should_receive(:info).with("Email notifier timed out") + @project.send(:send_notifications) + end end end spec/project_spec.rb 88f7e1f484bdb933d306efcea60d34107d834eb6 Simon Rozet simon@rozet.name http://github.com/foca/integrity/commit/6cc92bfc9712440ed45e3630c081fb3e776ace6b 6cc92bfc9712440ed45e3630c081fb3e776ace6b 2008-11-21T14:12:48-08:00 2008-11-21T14:09:26-08:00 log time out of notifiers 954aa3fae448c25783857dd93a7a278a3046bfe1 Simon Rozet simon@rozet.name