Throw translated Exception on SRP conn failure

DATAREDIS-234

Author: jencompgeek

src/main/java/org/springframework/data/redis/connection/srp/SrpConnection.java

@@ -210,6 +210,8 @@ public SrpConnection(String host, int port, BlockingQueue<SrpConnection> queue)
 this.queue = queue;
 } catch (IOException e) {
 throw new RedisConnectionFailureException("Could not connect", e);
+} catch(RedisException e) {
+throw new RedisConnectionFailureException("Could not connect", e);
 }
 }
 

src/test/java/org/springframework/data/redis/connection/srp/SrpConnectionFactoryTests.java

@@ -17,8 +17,10 @@
 
 import org.junit.Ignore;
 import org.junit.Test;
+import org.springframework.data.redis.RedisConnectionFailureException;
 import org.springframework.data.redis.SettingsUtils;
 import org.springframework.data.redis.connection.RedisConnection;
+import static org.junit.Assert.fail;
 
 /**
  * Integration test of {@link SrpConnectionFactory}
@@ -37,6 +39,18 @@ public void testConnect() {
 connection.ping();
 }
 
+@Test
+public void testConnectInvalidHost() {
+SrpConnectionFactory factory = new SrpConnectionFactory();
+factory.setHostName("fakeHost");
+factory.afterPropertiesSet();
+try {
+factory.getConnection();
+fail("Expected a connection failure exception");
+} catch(RedisConnectionFailureException e) {
+}
+}
+
 @Ignore("Redis must have requirepass set to run this test")
 @Test
 public void testConnectWithPassword() {