DATAKV-137 - Polishing.

Remove whitespaces & unused imports, fix warnings.

Original Pull Request: #20

Author: christophstrobl

src/main/java/org/springframework/data/keyvalue/repository/query/KeyValuePartTreeQuery.java

@@ -144,7 +144,7 @@ protected KeyValueQuery<?> prepareQuery(KeyValueQuery<?> instance, Object[] para
 					parameters);
 			criteria = new SpelCriteria(spelExpression, context);
 		}
-		
+
 		KeyValueQuery<?> query = new KeyValueQuery(criteria);
 		Pageable pageable = accessor.getPageable();
 		Sort sort = accessor.getSort();
@@ -157,12 +157,12 @@ protected KeyValueQuery<?> prepareQuery(KeyValueQuery<?> instance, Object[] para
 	}
 
 	private SpelExpression getSpelExpression(Object criteria) {
-		
-		if(criteria instanceof SpelExpression){
+
+		if (criteria instanceof SpelExpression) {
 			return (SpelExpression) criteria;
 		}
-		
-		if(criteria instanceof SpelCriteria){
+
+		if (criteria instanceof SpelCriteria) {
 			return getSpelExpression(((SpelCriteria) criteria).getExpression());
 		}
 

src/test/java/org/springframework/data/keyvalue/repository/query/CachingKeyValuePartTreeQueryUnitTests.java

@@ -18,7 +18,7 @@
 import static org.hamcrest.core.IsNot.*;
 import static org.hamcrest.core.IsSame.*;
 import static org.junit.Assert.*;
-import static org.mockito.Matchers.any;
+import static org.mockito.Matchers.*;
 import static org.mockito.Mockito.*;
 
 import java.lang.reflect.Method;
@@ -39,7 +39,7 @@
 
 /**
  * Unit tests for {@link CachingKeyValuePartTreeQuery}.
- * 
+ *
  * @author Mark Paluch
  */
 @RunWith(MockitoJUnitRunner.class)
@@ -50,6 +50,7 @@ public class CachingKeyValuePartTreeQueryUnitTests {
 	@Mock ProjectionFactory projectionFactoryMock;
 
 	@Before
+	@SuppressWarnings({ "unchecked", "rawtypes" })
 	public void setUp() throws Exception {
 
 		when(metadataMock.getDomainType()).thenReturn((Class) Person.class);
@@ -60,7 +61,6 @@ public void setUp() throws Exception {
 	 * @see DATAKV-137
 	 */
 	@Test
-	@SuppressWarnings({ "unchecked", "rawtypes" })
 	public void cachedSpelExpressionShouldBeReusedWithNewContext() throws NoSuchMethodException, SecurityException {
 
 		QueryMethod qm = new QueryMethod(Repo.class.getMethod("findByFirstname", String.class), metadataMock,

src/test/java/org/springframework/data/map/AbstractRepositoryUnitTests.java

@@ -203,7 +203,7 @@ public void projectsResultToDynamicInterface() {
 	protected KeyValueRepositoryFactory createKeyValueRepositoryFactory(KeyValueOperations operations) {
 		return new KeyValueRepositoryFactory(operations);
 	}
-	
+
 	protected abstract T getRepository(KeyValueRepositoryFactory factory);
 
 	public static interface PersonRepository extends CrudRepository<Person, String>, KeyValueRepository<Person, String> {

src/test/java/org/springframework/data/map/CachingQuerySimpleKeyValueRepositoryUnitTests.java

@@ -15,31 +15,23 @@
  */
 package org.springframework.data.map;
 
-import static org.hamcrest.Matchers.hasItems;
-import static org.junit.Assert.assertThat;
-
-import org.junit.Test;
 import org.springframework.data.keyvalue.core.KeyValueOperations;
 import org.springframework.data.keyvalue.repository.query.CachingKeyValuePartTreeQuery;
 import org.springframework.data.keyvalue.repository.query.SpelQueryCreator;
 import org.springframework.data.keyvalue.repository.support.KeyValueRepositoryFactory;
 import org.springframework.data.keyvalue.repository.support.SimpleKeyValueRepository;
-import org.springframework.data.map.AbstractRepositoryUnitTests.PersonRepository;
 
 /**
- * Unit tests for {@link SimpleKeyValueRepository} using {@link CachingKeyValuePartTreeQuery} and {@link SpelQueryCreator}.
- * 
+ * Unit tests for {@link SimpleKeyValueRepository} using {@link CachingKeyValuePartTreeQuery} and
+ * {@link SpelQueryCreator}.
+ *
  * @author Mark Paluch
+ * @author Christoph Strobl
  */
-public class CachingQuerySimpleKeyValueRepositoryUnitTests extends AbstractRepositoryUnitTests<PersonRepository> {
+public class CachingQuerySimpleKeyValueRepositoryUnitTests extends SimpleKeyValueRepositoryUnitTests {
 
 	@Override
 	protected KeyValueRepositoryFactory createKeyValueRepositoryFactory(KeyValueOperations operations) {
 		return new KeyValueRepositoryFactory(operations, SpelQueryCreator.class, CachingKeyValuePartTreeQuery.class);
 	}
-
-	@Override
-	protected PersonRepository getRepository(KeyValueRepositoryFactory factory) {
-		return factory.getRepository(PersonRepository.class);
-	}
 }