DATAREDIS-289 - Avoid NPE in DefaultValueOperations#append() in case of pipelining or/and multi/exec calls.

We now return null if DefaultValueOperations#append() was applied to a non-existing list and thus returned null. Previously a NPE was thrown in case of pipelining or/and multi/exec calls.

Original pull request: #48.

Author: reta

src/main/java/org/springframework/data/redis/core/DefaultValueOperations.java

@@ -84,7 +84,8 @@ public Integer append(K key, String value) {
 		return execute(new RedisCallback<Integer>() {
 
 			public Integer doInRedis(RedisConnection connection) {
-				return connection.append(rawKey, rawString).intValue();
+				final Long result = connection.append(rawKey, rawString); 				
+				return ( result != null ) ? result.intValue() : null; 
 			}
 		}, true);
 	}

src/test/java/org/springframework/data/redis/core/RedisTemplateTests.java

@@ -218,6 +218,7 @@ public List<Object> execute(RedisOperations operations) throws DataAccessExcepti
 				operations.multi();
 				operations.opsForValue().set("foo", "5");
 				operations.opsForValue().get("foo");
+				operations.opsForValue().append("foo1", "5");
 				operations.opsForList().leftPush("foolist", "6");
 				operations.opsForList().range("foolist", 0l, 1l);
 				operations.opsForSet().add("fooset", "7");
@@ -239,7 +240,7 @@ public List<Object> execute(RedisOperations operations) throws DataAccessExcepti
 		Map<Long, Long> map = new LinkedHashMap<Long, Long>();
 		map.put(10l, 11l);
 		assertThat(results,
-				isEqual(Arrays.asList(new Object[] { 5l, 1l, list, 1l, longSet, true, tupleSet, zSet, true, map })));
+				isEqual(Arrays.asList(new Object[] { 5l, 1L, 1l, list, 1l, longSet, true, tupleSet, zSet, true, map })));
 	}
 
 	@Test