Merge pull request #119 from soupmatt/handle_network_timeouts_on_sign…

…_out_notifications

Handle Errno::ETIMEOUT when sending single sign out notifications

app/models/casino/service_ticket/single_sign_out_notifier.rb

@@ -37,7 +37,7 @@ def send_notification(url, xml)
       Rails.logger.warn "Service #{url} responded to logout notification with code '#{result.status}'!"
       false
     end
-  rescue Faraday::Error::ClientError => error
+  rescue Faraday::Error::ClientError, Errno::ETIMEDOUT => error
     Rails.logger.warn "Failed to send logout notification to service #{url} due to #{error}"
     false
   end

spec/model/service_ticket/single_sign_out_notifier_spec.rb

@@ -56,6 +56,16 @@
           notifier.notify.should == false
         end
       end
+
+      context 'network timeout' do
+        before(:each) do
+          stub_request(:post, service).to_raise Errno::ETIMEDOUT
+        end
+
+        it 'returns false' do
+          notifier.notify.should == false
+        end
+      end
     end
   end
-end
+end