Fix SRP exception on select in transaction

DATAREDIS-220

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

@@ -592,6 +592,7 @@ public class SrpConnection implements RedisConnection {
 		try {
 			if (isPipelined()) {
 				pipeline(pipeline.select(dbIndex));
+				return;
 			}
 			client.select(dbIndex);
 		} catch (Exception ex) {

src/test/java/org/springframework/data/redis/connection/AbstractConnectionIntegrationTests.java

@@ -98,6 +98,12 @@ public abstract class AbstractConnectionIntegrationTests {
 		connection = null;
 	}
 
+	@Test
+	public void testSelect() {
+		// Make sure this doesn't throw Exception
+		connection.select(1);
+	}
+
 	@Test
 	@IfProfileValue(name = "runLongTests", value = "true")
 	public void testExpire() throws Exception {

src/test/java/org/springframework/data/redis/connection/jedis/JedisConnectionPipelineIntegrationTests.java

@@ -141,6 +141,11 @@ public class JedisConnectionPipelineIntegrationTests extends
 		super.testZRevRangeByScoreWithScoresOffsetCount();
 	}
 
+	@Test(expected = RedisSystemException.class)
+	public void testSelect() {
+		super.testSelect();
+	}
+
 	// Overrides, usually due to return values being Long vs Boolean or Set vs
 	// List
 

src/test/java/org/springframework/data/redis/connection/jredis/JRedisConnectionIntegrationTests.java

@@ -274,6 +274,10 @@ public class JRedisConnectionIntegrationTests extends AbstractConnectionIntegrat
 		super.testZRevRangeByScoreWithScoresOffsetCount();
 	}
 
+	@Test(expected = UnsupportedOperationException.class)
+	public void testSelect() {
+		super.testSelect();
+	}
 
 	// Jredis returns null for rPush
 	@Test

src/test/java/org/springframework/data/redis/connection/lettuce/LettuceConnectionIntegrationTests.java

@@ -123,7 +123,7 @@ public class LettuceConnectionIntegrationTests extends AbstractConnectionIntegra
 
 	@Test(expected = UnsupportedOperationException.class)
 	public void testSelect() {
-		connection.select(1);
+		super.testSelect();
 	}
 
 	@Test

src/test/java/org/springframework/data/redis/connection/lettuce/LettuceConnectionPipelineIntegrationTests.java

@@ -60,6 +60,11 @@ public class LettuceConnectionPipelineIntegrationTests extends
 	public void testMultiDiscard() {
 	}
 
+	@Test(expected=UnsupportedOperationException.class)
+	public void testSelect() {
+		super.testSelect();
+	}
+
 	// Overrides, usually due to return values being Long vs Boolean or Set vs
 	// List