Polishing.

See #1943.

Author: gregturn

spring-data-jpa/src/test/java/org/springframework/data/jpa/domain/SpecificationUnitTests.java

@@ -21,13 +21,13 @@
 import static org.springframework.data.jpa.domain.Specification.not;
 import static org.springframework.util.SerializationUtils.*;
 
-import java.io.Serializable;
-
 import jakarta.persistence.criteria.CriteriaBuilder;
 import jakarta.persistence.criteria.CriteriaQuery;
 import jakarta.persistence.criteria.Predicate;
 import jakarta.persistence.criteria.Root;
 
+import java.io.Serializable;
+
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
@@ -119,7 +119,7 @@ void orConcatenatesNullSpecToSpec() {
 		assertThat(specification.toPredicate(root, query, builder)).isEqualTo(predicate);
 	}
 
-	@Test // DATAJPA-1651
+	@Test // GH-1943
 	public void allOfConcatenatesNull() {
 
 		Specification<Object> specification = Specification.allOf(null, spec, null);
@@ -128,7 +128,7 @@ public void allOfConcatenatesNull() {
 		assertThat(specification.toPredicate(root, query, builder)).isEqualTo(predicate);
 	}
 
-	@Test // DATAJPA-1651
+	@Test // GH-1943
 	public void anyOfConcatenatesNull() {
 
 		Specification<Object> specification = Specification.anyOf(null, spec, null);
@@ -137,7 +137,7 @@ public void anyOfConcatenatesNull() {
 		assertThat(specification.toPredicate(root, query, builder)).isEqualTo(predicate);
 	}
 
-	@Test // DATAJPA-1651
+	@Test // GH-1943
 	public void emptyAllOfReturnsEmptySpecification() {
 
 		Specification<Object> specification = Specification.allOf();
@@ -146,7 +146,7 @@ public void emptyAllOfReturnsEmptySpecification() {
 		assertThat(specification.toPredicate(root, query, builder)).isNull();
 	}
 
-	@Test // DATAJPA-1651
+	@Test // GH-1943
 	public void emptyAnyOfReturnsEmptySpecification() {
 
 		Specification<Object> specification = Specification.anyOf();

spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/UserRepositoryTests.java

@@ -490,7 +490,7 @@ void throwsExceptionForUnderSpecifiedSingleEntitySpecification() {
 				.isThrownBy(() -> repository.findOne(userHasFirstnameLike("e")));
 	}
 
-	@Test // DATAJPA-1651
+	@Test // GH-1943
 	void executesCombinedSpecificationsCorrectly() {
 
 		flushTestUsers();
@@ -516,7 +516,7 @@ void executesNegatingSpecificationCorrectly() {
 		assertThat(repository.findAll(spec)).containsOnly(secondUser);
 	}
 
-	@Test // DATAJPA-1651
+	@Test // GH-1943
 	void executesCombinedSpecificationsWithPageableCorrectly() {
 
 		flushTestUsers();
@@ -623,22 +623,22 @@ void removeDetachedObject() {
 		assertThat(repository.count()).isEqualTo(3L);
 	}
 
-	@Test // DATAJPA-1651
+	@Test
 	void executesPagedSpecificationsCorrectly() {
 
 		Page<User> result = executeSpecWithSort(Sort.unsorted());
 		assertThat(result.getContent()).isSubsetOf(firstUser, thirdUser);
 	}
 
-	@Test // DATAJPA-1651
+	@Test
 	void executesPagedSpecificationsWithSortCorrectly() {
 
 		Page<User> result = executeSpecWithSort(Sort.by(Direction.ASC, "lastname"));
 
 		assertThat(result.getContent()).contains(firstUser).doesNotContain(secondUser, thirdUser);
 	}
 
-	@Test // DATAJPA-1651
+	@Test
 	void executesPagedSpecificationWithSortCorrectly2() {
 
 		Page<User> result = executeSpecWithSort(Sort.by(Direction.DESC, "lastname"));