Align with KeyValue KeySpaceResolver changes.

We now use the KeyValue KeySpaceResolver to primarily detect the keyspace to be used instead of a fallback.

See #2457
See #2452

Author: mp911de

src/main/java/org/springframework/data/redis/core/mapping/BasicRedisPersistentEntity.java

@@ -41,12 +41,12 @@ public class BasicRedisPersistentEntity<T> extends BasicKeyValuePersistentEntity
 	 * Creates new {@link BasicRedisPersistentEntity}.
 	 *
 	 * @param information must not be {@literal null}.
-	 * @param fallbackKeySpaceResolver can be {@literal null}.
+	 * @param keySpaceResolver can be {@literal null}.
 	 * @param timeToLiveAccessor can be {@literal null}.
 	 */
-	public BasicRedisPersistentEntity(TypeInformation<T> information, @Nullable KeySpaceResolver fallbackKeySpaceResolver,
+	public BasicRedisPersistentEntity(TypeInformation<T> information, @Nullable KeySpaceResolver keySpaceResolver,
 			TimeToLiveAccessor timeToLiveAccessor) {
-		super(information, fallbackKeySpaceResolver);
+		super(information, keySpaceResolver);
 
 		Assert.notNull(timeToLiveAccessor, "TimeToLiveAccessor must not be null");
 		this.timeToLiveAccessor = timeToLiveAccessor;

src/main/java/org/springframework/data/redis/core/mapping/RedisMappingContext.java

@@ -133,7 +133,7 @@ public String resolveKeySpace(Class<?> type) {
 				}
 			}
 
-			return ClassNameKeySpaceResolver.INSTANCE.resolveKeySpace(type);
+			return null;
 		}
 	}
 

src/test/java/org/springframework/data/redis/core/mapping/ConfigAwareKeySpaceResolverUnitTests.java

@@ -19,17 +19,18 @@
 
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
-
 import org.springframework.data.redis.core.convert.KeyspaceConfiguration;
 import org.springframework.data.redis.core.convert.KeyspaceConfiguration.KeyspaceSettings;
 import org.springframework.data.redis.core.mapping.RedisMappingContext.ConfigAwareKeySpaceResolver;
 
 /**
+ * Unit tests for {@link ConfigAwareKeySpaceResolver}.
+ *
  * @author Christoph Strobl
+ * @author Mark Paluch
  */
 class ConfigAwareKeySpaceResolverUnitTests {
 
-	static final String CUSTOM_KEYSPACE = "car'a'carn";
 	private KeyspaceConfiguration config = new KeyspaceConfiguration();
 	private ConfigAwareKeySpaceResolver resolver;
 
@@ -43,10 +44,9 @@ void resolveShouldThrowExceptionWhenTypeIsNull() {
 		assertThatIllegalArgumentException().isThrownBy(() -> resolver.resolveKeySpace(null));
 	}
 
-	@Test // DATAREDIS-425
-	void resolveShouldUseClassNameAsDefaultKeyspace() {
-		assertThat(resolver.resolveKeySpace(TypeWithoutAnySettings.class))
-				.isEqualTo(TypeWithoutAnySettings.class.getName());
+	@Test // DATAREDIS-425, GH-2457
+	void resolveShouldReturnNullAsDefaultKeyspace() {
+		assertThat(resolver.resolveKeySpace(TypeWithoutAnySettings.class)).isNull();
 	}
 
 	@Test // DATAREDIS-425