Polishing.

See #3041

Author: gregturn

.gitignore

@@ -8,3 +8,6 @@ target/
 .sonar4clipse
 *.sonar4clipseExternals
 .DS_Store
+node
+node_modules
+package*.json

spring-data-jpa/src/main/java/org/springframework/data/jpa/provider/PersistenceProvider.java

@@ -196,8 +196,7 @@ public String getCommentHintKey() {
 
 		Class<?> type;
 		try {
-			type = ClassUtils.forName("org.hibernate.query.TypedParameterValue",
-					PersistenceProvider.class.getClassLoader());
+			type = ClassUtils.forName("org.hibernate.query.TypedParameterValue", PersistenceProvider.class.getClassLoader());
 		} catch (ClassNotFoundException e) {
 			type = null;
 		}

spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/query/ParameterBinder.java

@@ -72,6 +72,7 @@ public ParameterBinder(JpaParameters parameters, Iterable<QueryParameterSetter>
 
 	public <T extends Query> T bind(T jpaQuery, QueryParameterSetter.QueryMetadata metadata,
 			JpaParametersParameterAccessor accessor) {
+
 		bind(metadata.withQuery(jpaQuery), accessor, ErrorHandling.STRICT);
 		return jpaQuery;
 	}

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

@@ -105,7 +105,6 @@ void shouldDetectComplexNativeQueriesWithSpelAsNonNative() {
 						+ "AND (n.updatedAt >= ?#{#networkRequest.updatedTime.startDateTime}) AND (n.updatedAt <=?#{#networkRequest.updatedTime.endDateTime})",
 				metadata, SPEL_PARSER, true);
 
-		System.out.println(query.getQueryString());
 		assertThat(query.isNativeQuery()).isFalse();
 	}
 

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

@@ -62,7 +62,8 @@ void exceptionWhenQueryContainNamedParametersAndMethodParametersAreNotNamed() {
 		when(binding.getOrigin()).thenReturn(ParameterOrigin.ofParameter("NamedParameter", 1));
 
 		assertThatExceptionOfType(IllegalStateException.class) //
-				.isThrownBy(() -> setterFactory.create(binding, DeclaredQuery.of("from Employee e where e.name = :NamedParameter", false))) //
+				.isThrownBy(() -> setterFactory.create(binding,
+						DeclaredQuery.of("from Employee e where e.name = :NamedParameter", false))) //
 				.withMessageContaining("Java 8") //
 				.withMessageContaining("@Param") //
 				.withMessageContaining("-parameters");
@@ -80,7 +81,8 @@ void exceptionWhenCriteriaQueryContainsInsufficientAmountOfParameters() {
 		when(binding.getOrigin()).thenReturn(ParameterOrigin.ofParameter(null, 1));
 
 		assertThatExceptionOfType(IllegalArgumentException.class) //
-				.isThrownBy(() -> setterFactory.create(binding, DeclaredQuery.of("from Employee e where e.name = :NamedParameter", false))) //
+				.isThrownBy(() -> setterFactory.create(binding,
+						DeclaredQuery.of("from Employee e where e.name = :NamedParameter", false))) //
 				.withMessage("At least 1 parameter(s) provided but only 0 parameter(s) present in query");
 	}
 

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

@@ -136,7 +136,7 @@ void rewritesNamedLikeToUniqueParametersIfNecessary() {
 				true);
 
 		assertThat(query.hasParameterBindings()).isTrue();
-		assertThat(query.getQueryString())
+		assertThat(query.getQueryString()) //
 				.isEqualTo(
 						"select u from User u where u.firstname like :firstname or u.firstname like :firstname_1 or u.firstname = :firstname_2");
 
@@ -152,6 +152,11 @@ void rewritesNamedLikeToUniqueParametersIfNecessary() {
 		assertThat(binding).isNotNull();
 		assertThat(binding.getName()).isEqualTo("firstname_1");
 		assertThat(binding.getType()).isEqualTo(Type.STARTING_WITH);
+
+		ParameterBinding parameterBinding = bindings.get(2);
+		assertThat(parameterBinding).isNotNull();
+		assertThat(parameterBinding.getName()).isEqualTo("firstname_2");
+		assertThat(((MethodInvocationArgument) parameterBinding.getOrigin()).identifier().getName()).isEqualTo("firstname");
 	}
 
 	@Test // GH-3041
@@ -179,7 +184,6 @@ void reusesNamedLikeBindingsWherePossible() {
 		assertThat(query.getQueryString()).isEqualTo(
 				"select u from User u where u.firstname like :firstname or u.firstname like :firstname_1 or u.firstname like :firstname_1 or u.firstname like :firstname");
 
-
 		query = new StringQuery("select u from User u where u.firstname like %:firstname or u.firstname =:firstname", true);
 
 		assertThat(query.hasParameterBindings()).isTrue();
@@ -582,7 +586,7 @@ void makesUsageOfJdbcStyleParameterAvailable() {
 
 			assertThat(new StringQuery(testQuery, false) //
 					.usesJdbcStyleParameters()) //
-							.describedAs(testQuery)
+							.describedAs(testQuery) //
 							.describedAs(testQuery) //
 							.isFalse();
 		}