Spring/spring-data-commons
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

DATACMNS-937 - Polishing.

Browse Source 
Removed Spring 4.2 guard from QueryExecutionConverter and its unit tests.
This commit is contained in:
Oliver Gierke
2016-11-19 12:46:49 +01:00
parent 1d3af0c128
commit cca9cd8abc
2 changed files with 2 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/main/java/org/springframework/data/repository/util/QueryExecutionConverters.java
View File

@@ -59,10 +59,6 @@ import com.google.common.base.Optional;
*/
public abstract class QueryExecutionConverters {
private static final boolean SPRING_4_2_PRESENT = ClassUtils.isPresent(
"org.springframework.core.annotation.AnnotationConfigurationException",
QueryExecutionConverters.class.getClassLoader());
private static final boolean GUAVA_PRESENT = ClassUtils.isPresent("com.google.common.base.Optional",
QueryExecutionConverters.class.getClassLoader());
private static final boolean JDK_8_PRESENT = ClassUtils.isPresent("java.util.Optional",
@@ -90,7 +86,7 @@ public abstract class QueryExecutionConverters {
UNWRAPPERS.add(Jdk8OptionalUnwrapper.INSTANCE);
}
if (JDK_8_PRESENT && SPRING_4_2_PRESENT) {
if (JDK_8_PRESENT) {
WRAPPER_TYPES.add(NullableWrapperToCompletableFutureConverter.getWrapperType());
}
@@ -480,7 +476,7 @@ public abstract class QueryExecutionConverters {
*
* @author Oliver Gierke
* @author Mark Paluch
* @author 1.13
* @since 1.12
*/
private static enum ScalOptionUnwrapper implements Converter<Object, Object> {

9
src/test/java/org/springframework/data/repository/util/QueryExecutionConvertersUnitTests.java
View File

@@ -17,7 +17,6 @@ package org.springframework.data.repository.util;
import static org.hamcrest.CoreMatchers.*;
import static org.junit.Assert.*;
import static org.junit.Assume.*;
import scala.Option;
@@ -27,9 +26,7 @@ import java.util.concurrent.Future;
import org.junit.Before;
import org.junit.Test;
import org.springframework.core.SpringVersion;
import org.springframework.core.convert.support.DefaultConversionService;
import org.springframework.data.util.Version;
import org.springframework.util.ReflectionUtils;
import org.springframework.util.concurrent.ListenableFuture;
@@ -43,9 +40,6 @@ import com.google.common.base.Optional;
*/
public class QueryExecutionConvertersUnitTests {
private static final Version SPRING_VERSION = Version.parse(SpringVersion.getVersion());
private static final Version FOUR_DOT_TWO = new Version(4, 2);
DefaultConversionService conversionService;
@Before
@@ -74,9 +68,6 @@ public class QueryExecutionConvertersUnitTests {
*/
@Test
public void registersCompletableFutureAsWrapperTypeOnSpring42OrBetter() {
assumeThat(SPRING_VERSION.isGreaterThanOrEqualTo(FOUR_DOT_TWO), is(true));
assertThat(QueryExecutionConverters.supports(CompletableFuture.class), is(true));
}