Remove punctuation in Exception messages.

jxblum · jxblum · commit efaeeebe39c3 · 2022-06-08T16:59:58.000-07:00
Closes #442.
diff --git a/src/main/java/org/springframework/data/keyvalue/core/DefaultIdentifierGenerator.java b/src/main/java/org/springframework/data/keyvalue/core/DefaultIdentifierGenerator.java
@@ -56,7 +56,7 @@ public <T> T generateIdentifierOfType(TypeInformation<T> identifierType) {
 		}
 
 		throw new InvalidDataAccessApiUsageException(
-				String.format("Identifier cannot be generated for %s. Supported types are: UUID, String, Integer, and Long.",
+				String.format("Identifier cannot be generated for %s; Supported types are: UUID, String, Integer, and Long",
 						identifierType.getType().getName()));
 	}
 
@@ -77,7 +77,7 @@ private SecureRandom getSecureRandom() {
 
 		if (secureRandom == null) {
 			throw new InvalidDataAccessApiUsageException(
-					String.format("Could not create SecureRandom instance for one of the algorithms '%s'.",
+					String.format("Could not create SecureRandom instance for one of the algorithms '%s'",
 							StringUtils.collectionToCommaDelimitedString(OsTools.secureRandomAlgorithmNames())));
 		}
 
diff --git a/src/main/java/org/springframework/data/keyvalue/core/ForwardingCloseableIterator.java b/src/main/java/org/springframework/data/keyvalue/core/ForwardingCloseableIterator.java
@@ -52,7 +52,7 @@ public ForwardingCloseableIterator(Iterator<? extends T> delegate) {
 	 */
 	public ForwardingCloseableIterator(Iterator<? extends T> delegate, @Nullable Runnable closeHandler) {
 
-		Assert.notNull(delegate, "Delegate iterator must not be null!");
+		Assert.notNull(delegate, "Delegate iterator must not be null");
 
 		this.delegate = delegate;
 		this.closeHandler = closeHandler;
diff --git a/src/main/java/org/springframework/data/keyvalue/core/GeneratingIdAccessor.java b/src/main/java/org/springframework/data/keyvalue/core/GeneratingIdAccessor.java
@@ -45,9 +45,9 @@ class GeneratingIdAccessor implements IdentifierAccessor {
 	GeneratingIdAccessor(PersistentPropertyAccessor<?> accessor, PersistentProperty<?> identifierProperty,
 			IdentifierGenerator generator) {
 
-		Assert.notNull(accessor, "PersistentPropertyAccessor must not be null!");
-		Assert.notNull(identifierProperty, "Identifier property must not be null!");
-		Assert.notNull(generator, "IdentifierGenerator must not be null!");
+		Assert.notNull(accessor, "PersistentPropertyAccessor must not be null");
+		Assert.notNull(identifierProperty, "Identifier property must not be null");
+		Assert.notNull(generator, "IdentifierGenerator must not be null");
 
 		this.accessor = accessor;
 		this.identifierProperty = identifierProperty;
diff --git a/src/main/java/org/springframework/data/keyvalue/core/KeyValuePersistenceExceptionTranslator.java b/src/main/java/org/springframework/data/keyvalue/core/KeyValuePersistenceExceptionTranslator.java
@@ -36,7 +36,7 @@ public class KeyValuePersistenceExceptionTranslator implements PersistenceExcept
 	@Override
 	public DataAccessException translateExceptionIfPossible(RuntimeException exception) {
 
-		Assert.notNull(exception, "Exception must not be null!");
+		Assert.notNull(exception, "Exception must not be null");
 
 		if (exception instanceof DataAccessException) {
 			return (DataAccessException) exception;
diff --git a/src/main/java/org/springframework/data/keyvalue/core/KeyValueTemplate.java b/src/main/java/org/springframework/data/keyvalue/core/KeyValueTemplate.java
@@ -96,9 +96,9 @@ public KeyValueTemplate(KeyValueAdapter adapter,
 			MappingContext<? extends KeyValuePersistentEntity<?, ?>, ? extends KeyValuePersistentProperty<?>> mappingContext,
 			IdentifierGenerator identifierGenerator) {
 
-		Assert.notNull(adapter, "Adapter must not be null!");
-		Assert.notNull(mappingContext, "MappingContext must not be null!");
-		Assert.notNull(identifierGenerator, "IdentifierGenerator must not be null!");
+		Assert.notNull(adapter, "Adapter must not be null");
+		Assert.notNull(mappingContext, "MappingContext must not be null");
+		Assert.notNull(identifierGenerator, "IdentifierGenerator must not be null");
 
 		this.adapter = adapter;
 		this.mappingContext = mappingContext;
@@ -112,7 +112,7 @@ public KeyValueTemplate(KeyValueAdapter adapter,
 	 */
 	public void setExceptionTranslator(PersistenceExceptionTranslator exceptionTranslator) {
 
-		Assert.notNull(exceptionTranslator, "ExceptionTranslator must not be null!");
+		Assert.notNull(exceptionTranslator, "ExceptionTranslator must not be null");
 		this.exceptionTranslator = exceptionTranslator;
 	}
 
@@ -152,8 +152,8 @@ public <T> T insert(T objectToInsert) {
 	@Override
 	public <T> T insert(Object id, T objectToInsert) {
 
-		Assert.notNull(id, "Id for object to be inserted must not be null!");
-		Assert.notNull(objectToInsert, "Object to be inserted must not be null!");
+		Assert.notNull(id, "Id for object to be inserted must not be null");
+		Assert.notNull(objectToInsert, "Object to be inserted must not be null");
 
 		String keyspace = resolveKeySpace(objectToInsert.getClass());
 
@@ -163,7 +163,7 @@ public <T> T insert(Object id, T objectToInsert) {
 
 			if (adapter.contains(id, keyspace)) {
 				throw new DuplicateKeyException(
-						String.format("Cannot insert existing object with id %s!. Please use update.", id));
+						String.format("Cannot insert existing object with id %s; Please use update", id));
 			}
 
 			adapter.put(id, objectToInsert, keyspace);
@@ -191,8 +191,8 @@ public <T> T update(T objectToUpdate) {
 	@Override
 	public <T> T update(Object id, T objectToUpdate) {
 
-		Assert.notNull(id, "Id for object to be inserted must not be null!");
-		Assert.notNull(objectToUpdate, "Object to be updated must not be null!");
+		Assert.notNull(id, "Id for object to be inserted must not be null");
+		Assert.notNull(objectToUpdate, "Object to be updated must not be null");
 
 		String keyspace = resolveKeySpace(objectToUpdate.getClass());
 
@@ -209,7 +209,7 @@ public <T> T update(Object id, T objectToUpdate) {
 	@Override
 	public <T> Iterable<T> findAll(Class<T> type) {
 
-		Assert.notNull(type, "Type to fetch must not be null!");
+		Assert.notNull(type, "Type to fetch must not be null");
 
 		return executeRequired(adapter -> {
 
@@ -229,8 +229,8 @@ public <T> Iterable<T> findAll(Class<T> type) {
 	@Override
 	public <T> Optional<T> findById(Object id, Class<T> type) {
 
-		Assert.notNull(id, "Id for object to be found must not be null!");
-		Assert.notNull(type, "Type to fetch must not be null!");
+		Assert.notNull(id, "Id for object to be found must not be null");
+		Assert.notNull(type, "Type to fetch must not be null");
 
 		String keyspace = resolveKeySpace(type);
 
@@ -255,7 +255,7 @@ public <T> Optional<T> findById(Object id, Class<T> type) {
 	@Override
 	public void delete(Class<?> type) {
 
-		Assert.notNull(type, "Type to delete must not be null!");
+		Assert.notNull(type, "Type to delete must not be null");
 
 		String keyspace = resolveKeySpace(type);
 
@@ -283,8 +283,8 @@ public <T> T delete(T objectToDelete) {
 	@Override
 	public <T> T delete(Object id, Class<T> type) {
 
-		Assert.notNull(id, "Id for object to be deleted must not be null!");
-		Assert.notNull(type, "Type to delete must not be null!");
+		Assert.notNull(id, "Id for object to be deleted must not be null");
+		Assert.notNull(type, "Type to delete must not be null");
 
 		String keyspace = resolveKeySpace(type);
 
@@ -300,15 +300,15 @@ public <T> T delete(Object id, Class<T> type) {
 	@Override
 	public long count(Class<?> type) {
 
-		Assert.notNull(type, "Type for count must not be null!");
+		Assert.notNull(type, "Type for count must not be null");
 		return adapter.count(resolveKeySpace(type));
 	}
 
 	@Nullable
 	@Override
 	public <T> T execute(KeyValueCallback<T> action) {
 
-		Assert.notNull(action, "KeyValueCallback must not be null!");
+		Assert.notNull(action, "KeyValueCallback must not be null");
 
 		try {
 			return action.doInKeyValue(this.adapter);
@@ -332,7 +332,7 @@ protected <T> T executeRequired(KeyValueCallback<T> action) {
 			return result;
 		}
 
-		throw new IllegalStateException(String.format("KeyValueCallback %s returned null value!", action));
+		throw new IllegalStateException(String.format("KeyValueCallback %s returned null value", action));
 	}
 
 	@Override
diff --git a/src/main/java/org/springframework/data/keyvalue/core/QueryEngine.java b/src/main/java/org/springframework/data/keyvalue/core/QueryEngine.java
@@ -144,7 +144,7 @@ protected ADAPTER getRequiredAdapter() {
 			return adapter;
 		}
 
-		throw new IllegalStateException("Required KeyValueAdapter is not set!");
+		throw new IllegalStateException("Required KeyValueAdapter is not set");
 	}
 
 	/**
@@ -156,7 +156,7 @@ public void registerAdapter(KeyValueAdapter adapter) {
 		if (this.adapter == null) {
 			this.adapter = (ADAPTER) adapter;
 		} else {
-			throw new IllegalArgumentException("Cannot register more than one adapter for this QueryEngine.");
+			throw new IllegalArgumentException("Cannot register more than one adapter for this QueryEngine");
 		}
 	}
 }
diff --git a/src/main/java/org/springframework/data/keyvalue/core/SpelCriteria.java b/src/main/java/org/springframework/data/keyvalue/core/SpelCriteria.java
@@ -49,8 +49,8 @@ public SpelCriteria(SpelExpression expression) {
 	 */
 	public SpelCriteria(SpelExpression expression, EvaluationContext context) {
 
-		Assert.notNull(expression, "SpEL expression must not be null!");
-		Assert.notNull(context, "EvaluationContext must not be null!");
+		Assert.notNull(expression, "SpEL expression must not be null");
+		Assert.notNull(context, "EvaluationContext must not be null");
 
 		this.expression = expression;
 		this.context = context;
diff --git a/src/main/java/org/springframework/data/keyvalue/core/SpelCriteriaAccessor.java b/src/main/java/org/springframework/data/keyvalue/core/SpelCriteriaAccessor.java
@@ -37,7 +37,7 @@ class SpelCriteriaAccessor implements CriteriaAccessor<SpelCriteria> {
 	 */
 	public SpelCriteriaAccessor(SpelExpressionParser parser) {
 
-		Assert.notNull(parser, "SpelExpressionParser must not be null!");
+		Assert.notNull(parser, "SpelExpressionParser must not be null");
 
 		this.parser = parser;
 	}
diff --git a/src/main/java/org/springframework/data/keyvalue/core/SpelPropertyComparator.java b/src/main/java/org/springframework/data/keyvalue/core/SpelPropertyComparator.java
@@ -47,8 +47,8 @@ public class SpelPropertyComparator<T> implements Comparator<T> {
 	 */
 	public SpelPropertyComparator(String path, SpelExpressionParser parser) {
 
-		Assert.hasText(path, "Path must not be null or empty!");
-		Assert.notNull(parser, "SpelExpressionParser must not be null!");
+		Assert.hasText(path, "Path must not be null or empty");
+		Assert.notNull(parser, "SpelExpressionParser must not be null");
 
 		this.path = path;
 		this.parser = parser;
diff --git a/src/main/java/org/springframework/data/keyvalue/core/SpelSortAccessor.java b/src/main/java/org/springframework/data/keyvalue/core/SpelSortAccessor.java
@@ -43,7 +43,7 @@ public class SpelSortAccessor implements SortAccessor<Comparator<?>> {
 	 */
 	public SpelSortAccessor(SpelExpressionParser parser) {
 
-		Assert.notNull(parser, "SpelExpressionParser must not be null!");
+		Assert.notNull(parser, "SpelExpressionParser must not be null");
 		this.parser = parser;
 	}
 
diff --git a/src/main/java/org/springframework/data/keyvalue/core/mapping/AnnotationBasedKeySpaceResolver.java b/src/main/java/org/springframework/data/keyvalue/core/mapping/AnnotationBasedKeySpaceResolver.java
@@ -39,7 +39,7 @@ enum AnnotationBasedKeySpaceResolver implements KeySpaceResolver {
 	@Nullable
 	public String resolveKeySpace(Class<?> type) {
 
-		Assert.notNull(type, "Type for keyspace for null!");
+		Assert.notNull(type, "Type for keyspace for null");
 
 		Class<?> userClass = ClassUtils.getUserClass(type);
 		Object keySpace = getKeySpace(userClass);
diff --git a/src/main/java/org/springframework/data/keyvalue/core/mapping/ClassNameKeySpaceResolver.java b/src/main/java/org/springframework/data/keyvalue/core/mapping/ClassNameKeySpaceResolver.java
@@ -31,7 +31,7 @@ enum ClassNameKeySpaceResolver implements KeySpaceResolver {
 	@Override
 	public String resolveKeySpace(Class<?> type) {
 
-		Assert.notNull(type, "Type must not be null!");
+		Assert.notNull(type, "Type must not be null");
 		return ClassUtils.getUserClass(type).getName();
 	}
 }
diff --git a/src/main/java/org/springframework/data/keyvalue/core/query/KeyValueQuery.java b/src/main/java/org/springframework/data/keyvalue/core/query/KeyValueQuery.java
@@ -133,7 +133,7 @@ public void setRows(int rows) {
 	 */
 	public void setSort(Sort sort) {
 
-		Assert.notNull(sort, "Sort must not be null!");
+		Assert.notNull(sort, "Sort must not be null");
 
 		this.sort = sort;
 	}
@@ -146,7 +146,7 @@ public void setSort(Sort sort) {
 	 */
 	public KeyValueQuery<T> orderBy(Sort sort) {
 
-		Assert.notNull(sort, "Sort must not be null!");
+		Assert.notNull(sort, "Sort must not be null");
 
 		if (this.sort.isSorted()) {
 			this.sort = this.sort.and(sort);
diff --git a/src/main/java/org/springframework/data/keyvalue/repository/query/KeyValuePartTreeQuery.java b/src/main/java/org/springframework/data/keyvalue/repository/query/KeyValuePartTreeQuery.java
@@ -88,10 +88,10 @@ public KeyValuePartTreeQuery(QueryMethod queryMethod, QueryMethodEvaluationConte
 			KeyValueOperations keyValueOperations,
 			QueryCreatorFactory<AbstractQueryCreator<KeyValueQuery<?>, ?>> queryCreatorFactory) {
 
-		Assert.notNull(queryMethod, "Query method must not be null!");
-		Assert.notNull(evaluationContextProvider, "EvaluationContextprovider must not be null!");
-		Assert.notNull(keyValueOperations, "KeyValueOperations must not be null!");
-		Assert.notNull(queryCreatorFactory, "QueryCreatorFactory type must not be null!");
+		Assert.notNull(queryMethod, "Query method must not be null");
+		Assert.notNull(evaluationContextProvider, "EvaluationContextprovider must not be null");
+		Assert.notNull(keyValueOperations, "KeyValueOperations must not be null");
+		Assert.notNull(queryCreatorFactory, "QueryCreatorFactory type must not be null");
 
 		this.partTree = Lazy
 				.of(() -> new PartTree(queryMethod.getName(), queryMethod.getEntityInformation().getJavaType()));
@@ -142,7 +142,7 @@ protected Object doExecute(Object[] parameters, KeyValueQuery<?> query) {
 			return keyValueOperations.exists(query, queryMethod.getEntityInformation().getJavaType());
 		}
 
-		throw new UnsupportedOperationException("Query method not supported.");
+		throw new UnsupportedOperationException("Query method not supported");
 	}
 
 	protected KeyValueQuery<?> prepareQuery(Object[] parameters) {
@@ -258,7 +258,7 @@ private static class ConstructorCachingQueryCreatorFactory
 		public AbstractQueryCreator<KeyValueQuery<?>, ?> queryCreatorFor(PartTree partTree, ParameterAccessor accessor) {
 
 			Assert.state(constructor != null,
-					() -> String.format("No constructor (PartTree, ParameterAccessor) could be found on type %s!", type));
+					() -> String.format("No constructor (PartTree, ParameterAccessor) could be found on type %s", type));
 			return (AbstractQueryCreator<KeyValueQuery<?>, ?>) BeanUtils.instantiateClass(constructor, partTree, accessor);
 		}
 	}
diff --git a/src/main/java/org/springframework/data/keyvalue/repository/query/SpelQueryCreator.java b/src/main/java/org/springframework/data/keyvalue/repository/query/SpelQueryCreator.java
@@ -107,7 +107,7 @@ protected SpelExpression toPredicateExpression(PartTree tree) {
 
 				// TODO: check if we can have caseinsensitive search
 				if (!part.shouldIgnoreCase().equals(IgnoreCaseType.NEVER)) {
-					throw new InvalidDataAccessApiUsageException("Ignore case not supported!");
+					throw new InvalidDataAccessApiUsageException("Ignore case not supported");
 				}
 
 				switch (part.getType()) {
diff --git a/src/main/java/org/springframework/data/keyvalue/repository/support/KeyValueQuerydslUtils.java b/src/main/java/org/springframework/data/keyvalue/repository/support/KeyValueQuerydslUtils.java
@@ -54,8 +54,8 @@ private KeyValueQuerydslUtils() {
 	 */
 	static OrderSpecifier<?>[] toOrderSpecifier(Sort sort, PathBuilder<?> builder) {
 
-		Assert.notNull(sort, "Sort must not be null!");
-		Assert.notNull(builder, "Builder must not be null!");
+		Assert.notNull(sort, "Sort must not be null");
+		Assert.notNull(builder, "Builder must not be null");
 
 		List<OrderSpecifier<?>> specifiers = null;
 
@@ -88,8 +88,8 @@ private static OrderSpecifier<?> toOrderSpecifier(Order order, PathBuilder<?> bu
 	 */
 	private static Expression<?> buildOrderPropertyPathFrom(Order order, PathBuilder<?> builder) {
 
-		Assert.notNull(order, "Order must not be null!");
-		Assert.notNull(builder, "Builder must not be null!");
+		Assert.notNull(order, "Order must not be null");
+		Assert.notNull(builder, "Builder must not be null");
 
 		PropertyPath path = PropertyPath.from(order.getProperty(), builder.getType());
 		Expression<?> sortPropertyExpression = builder;
@@ -118,7 +118,7 @@ private static Expression<?> buildOrderPropertyPathFrom(Order order, PathBuilder
 	 */
 	private static NullHandling toQueryDslNullHandling(org.springframework.data.domain.Sort.NullHandling nullHandling) {
 
-		Assert.notNull(nullHandling, "NullHandling must not be null!");
+		Assert.notNull(nullHandling, "NullHandling must not be null");
 
 		switch (nullHandling) {
 
diff --git a/src/main/java/org/springframework/data/keyvalue/repository/support/KeyValueRepositoryFactory.java b/src/main/java/org/springframework/data/keyvalue/repository/support/KeyValueRepositoryFactory.java
@@ -99,9 +99,9 @@ public KeyValueRepositoryFactory(KeyValueOperations keyValueOperations,
 	public KeyValueRepositoryFactory(KeyValueOperations keyValueOperations,
 			Class<? extends AbstractQueryCreator<?, ?>> queryCreator, Class<? extends RepositoryQuery> repositoryQueryType) {
 
-		Assert.notNull(keyValueOperations, "KeyValueOperations must not be null!");
-		Assert.notNull(queryCreator, "Query creator type must not be null!");
-		Assert.notNull(repositoryQueryType, "RepositoryQueryType type must not be null!");
+		Assert.notNull(keyValueOperations, "KeyValueOperations must not be null");
+		Assert.notNull(queryCreator, "Query creator type must not be null");
+		Assert.notNull(repositoryQueryType, "RepositoryQueryType type must not be null");
 
 		this.queryCreator = queryCreator;
 		this.keyValueOperations = keyValueOperations;
@@ -204,10 +204,10 @@ public KeyValueQueryLookupStrategy(@Nullable Key key,
 				Class<? extends AbstractQueryCreator<?, ?>> queryCreator,
 				Class<? extends RepositoryQuery> repositoryQueryType) {
 
-			Assert.notNull(evaluationContextProvider, "EvaluationContextProvider must not be null!");
-			Assert.notNull(keyValueOperations, "KeyValueOperations must not be null!");
-			Assert.notNull(queryCreator, "Query creator type must not be null!");
-			Assert.notNull(repositoryQueryType, "RepositoryQueryType type must not be null!");
+			Assert.notNull(evaluationContextProvider, "EvaluationContextProvider must not be null");
+			Assert.notNull(keyValueOperations, "KeyValueOperations must not be null");
+			Assert.notNull(queryCreator, "Query creator type must not be null");
+			Assert.notNull(repositoryQueryType, "RepositoryQueryType type must not be null");
 
 			this.evaluationContextProvider = evaluationContextProvider;
 			this.keyValueOperations = keyValueOperations;
@@ -228,7 +228,7 @@ public RepositoryQuery resolveQuery(Method method, RepositoryMetadata metadata,
 
 			Assert.state(constructor != null,
 					String.format(
-							"Constructor %s(QueryMethod, EvaluationContextProvider, KeyValueOperations, Class) not available!",
+							"Constructor %s(QueryMethod, EvaluationContextProvider, KeyValueOperations, Class) not available",
 							ClassUtils.getShortName(this.repositoryQueryType)));
 
 			return BeanUtils.instantiateClass(constructor, queryMethod, evaluationContextProvider, this.keyValueOperations,
diff --git a/src/main/java/org/springframework/data/keyvalue/repository/support/KeyValueRepositoryFactoryBean.java b/src/main/java/org/springframework/data/keyvalue/repository/support/KeyValueRepositoryFactoryBean.java
diff --git a/src/main/java/org/springframework/data/keyvalue/repository/support/QuerydslKeyValuePredicateExecutor.java b/src/main/java/org/springframework/data/keyvalue/repository/support/QuerydslKeyValuePredicateExecutor.java
diff --git a/src/main/java/org/springframework/data/keyvalue/repository/support/QuerydslKeyValueRepository.java b/src/main/java/org/springframework/data/keyvalue/repository/support/QuerydslKeyValueRepository.java
diff --git a/src/main/java/org/springframework/data/keyvalue/repository/support/SimpleKeyValueRepository.java b/src/main/java/org/springframework/data/keyvalue/repository/support/SimpleKeyValueRepository.java
diff --git a/src/main/java/org/springframework/data/map/MapKeyValueAdapter.java b/src/main/java/org/springframework/data/map/MapKeyValueAdapter.java
diff --git a/src/test/java/org/springframework/data/map/repository/config/MapRepositoriesConfigurationExtensionIntegrationTests.java b/src/test/java/org/springframework/data/map/repository/config/MapRepositoriesConfigurationExtensionIntegrationTests.java

Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@ public <T> T generateIdentifierOfType(TypeInformation<T> identifierType) {`
`56`	`56`	`}`
`57`	`57`
`58`	`58`	`throw new InvalidDataAccessApiUsageException(`
`59`		`- String.format("Identifier cannot be generated for %s. Supported types are: UUID, String, Integer, and Long.",`
	`59`	`+ String.format("Identifier cannot be generated for %s; Supported types are: UUID, String, Integer, and Long",`
`60`	`60`	`identifierType.getType().getName()));`
`61`	`61`	`}`
`62`	`62`
`@@ -77,7 +77,7 @@ private SecureRandom getSecureRandom() {`
`77`	`77`
`78`	`78`	`if (secureRandom == null) {`
`79`	`79`	`throw new InvalidDataAccessApiUsageException(`
`80`		`- String.format("Could not create SecureRandom instance for one of the algorithms '%s'.",`
	`80`	`+ String.format("Could not create SecureRandom instance for one of the algorithms '%s'",`
`81`	`81`	`StringUtils.collectionToCommaDelimitedString(OsTools.secureRandomAlgorithmNames())));`
`82`	`82`	`}`
`83`	`83`
Original file line number	Diff line number	Diff line change
`@@ -144,7 +144,7 @@ protected ADAPTER getRequiredAdapter() {`
`144`	`144`	`return adapter;`
`145`	`145`	`}`
`146`	`146`
`147`		`- throw new IllegalStateException("Required KeyValueAdapter is not set!");`
	`147`	`+ throw new IllegalStateException("Required KeyValueAdapter is not set");`
`148`	`148`	`}`
`149`	`149`
`150`	`150`	`/**`
`@@ -156,7 +156,7 @@ public void registerAdapter(KeyValueAdapter adapter) {`
`156`	`156`	`if (this.adapter == null) {`
`157`	`157`	`this.adapter = (ADAPTER) adapter;`
`158`	`158`	`} else {`
`159`		`- throw new IllegalArgumentException("Cannot register more than one adapter for this QueryEngine.");`
	`159`	`+ throw new IllegalArgumentException("Cannot register more than one adapter for this QueryEngine");`
`160`	`160`	`}`
`161`	`161`	`}`
`162`	`162`	`}`
Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ class SpelCriteriaAccessor implements CriteriaAccessor<SpelCriteria> {`
`37`	`37`	`*/`
`38`	`38`	`public SpelCriteriaAccessor(SpelExpressionParser parser) {`
`39`	`39`
`40`		`- Assert.notNull(parser, "SpelExpressionParser must not be null!");`
	`40`	`+ Assert.notNull(parser, "SpelExpressionParser must not be null");`
`41`	`41`
`42`	`42`	`this.parser = parser;`
`43`	`43`	`}`
Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ public class SpelSortAccessor implements SortAccessor<Comparator<?>> {`
`43`	`43`	`*/`
`44`	`44`	`public SpelSortAccessor(SpelExpressionParser parser) {`
`45`	`45`
`46`		`- Assert.notNull(parser, "SpelExpressionParser must not be null!");`
	`46`	`+ Assert.notNull(parser, "SpelExpressionParser must not be null");`
`47`	`47`	`this.parser = parser;`
`48`	`48`	`}`
`49`	`49`