DATAREDIS-531 - Polishing.

Use quit method for close verification in tests. Refine static imports. Fix spelling.

Original pull request: #218.

Author: mp911de

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

@@ -225,7 +225,7 @@ public void scanShouldKeepTheConnectionOpen() {
 
 			connection.scan();
 
-			verify(jedisSpy, never()).close();
+			verify(jedisSpy, never()).quit();
 		}
 
 		/**
@@ -254,7 +254,7 @@ public void sScanShouldKeepTheConnectionOpen() {
 
 			connection.sScan("foo".getBytes(), ScanOptions.NONE);
 
-			verify(jedisSpy, never()).close();
+			verify(jedisSpy, never()).quit();
 		}
 
 		/**
@@ -283,7 +283,7 @@ public void zScanShouldKeepTheConnectionOpen() {
 
 			connection.zScan("foo".getBytes(), ScanOptions.NONE);
 
-			verify(jedisSpy, never()).close();
+			verify(jedisSpy, never()).quit();
 		}
 
 		/**
@@ -312,7 +312,7 @@ public void hScanShouldKeepTheConnectionOpen() {
 
 			connection.hScan("foo".getBytes(), ScanOptions.NONE);
 
-			verify(jedisSpy, never()).close();
+			verify(jedisSpy, never()).quit();
 		}
 
 		/**
@@ -467,6 +467,5 @@ public MockedClientJedis(String host, Client client) {
 			super(host);
 			this.client = client;
 		}
-
 	}
 }

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

@@ -17,13 +17,13 @@
 
 import static org.hamcrest.core.Is.*;
 import static org.hamcrest.core.IsNull.*;
+import static org.hamcrest.core.IsSame.*;
 import static org.junit.Assert.*;
 import static org.mockito.Matchers.*;
 import static org.mockito.Mockito.*;
 
 import java.io.Serializable;
 
-import org.hamcrest.core.IsSame;
 import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
@@ -89,7 +89,7 @@ public void templateShouldPassOnAndUseResoureLoaderClassLoaderToDefaultJdkSerial
 		template.afterPropertiesSet();
 
 		when(redisConnectionMock.get(any(byte[].class)))
-				.thenReturn(new JdkSerializationRedisSerializer().serialize(new SomeArbitrarySeriaizableObject()));
+				.thenReturn(new JdkSerializationRedisSerializer().serialize(new SomeArbitrarySerializableObject()));
 
 		Object deserialized = template.opsForValue().get("spring");
 		assertThat(deserialized, notNullValue());
@@ -105,11 +105,11 @@ public void executeWithStickyConnectionShouldNotCloseConnectionWhenDone() {
 		CapturingCallback callback = new CapturingCallback();
 		template.executeWithStickyConnection(callback);
 
-		assertThat(callback.getConnection(), IsSame.sameInstance(redisConnectionMock));
+		assertThat(callback.getConnection(), sameInstance(redisConnectionMock));
 		verify(redisConnectionMock, never()).close();
 	}
 
-	static class SomeArbitrarySeriaizableObject implements Serializable {
+	static class SomeArbitrarySerializableObject implements Serializable {
 		private static final long serialVersionUID = -5973659324040506423L;
 	}
 
@@ -126,7 +126,5 @@ public Cursor<Object> doInRedis(RedisConnection connection) throws DataAccessExc
 		public RedisConnection getConnection() {
 			return connection;
 		}
-
 	}
-
 }