DATAREDIS-1031 - Fix reactive pExpire/pExpireAt to invoke correct command.

We now invoke the correct PEXPIRE(AT) command. Previously, we invoked EXPIRE/EXPIREAT commands resulting in the wrong command being called.

Original Pull Request: #474

Author: mp911de

src/main/java/org/springframework/data/redis/connection/ReactiveKeyCommands.java

@@ -131,7 +131,7 @@ default Mono<List<ByteBuffer>> keys(ByteBuffer pattern) {
 	 * Find all keys matching the given {@literal pattern}.<br />
 	 * It is recommended to use {@link #scan(ScanOptions)} to iterate over the keyspace as {@link #keys(Publisher)} is a
 	 * non-interruptible and expensive Redis operation.
-	 * 
+	 *
 	 * @param patterns must not be {@literal null}.
 	 * @return
 	 * @see <a href="https://redis.io/commands/keys">Redis Documentation: KEYS</a>
@@ -462,7 +462,7 @@ default Mono<Boolean> pExpire(ByteBuffer key, Duration timeout) {
 		Assert.notNull(key, "Key must not be null!");
 		Assert.notNull(timeout, "Timeout must not be null!");
 
-		return expire(Mono.just(new ExpireCommand(key, timeout))).next().map(BooleanResponse::getOutput);
+		return pExpire(Mono.just(new ExpireCommand(key, timeout))).next().map(BooleanResponse::getOutput);
 	}
 
 	/**

src/main/java/org/springframework/data/redis/connection/lettuce/LettuceReactiveKeyCommands.java

@@ -262,7 +262,7 @@ public Flux<BooleanResponse<ExpireCommand>> pExpire(Publisher<ExpireCommand> com
 			Assert.notNull(command.getKey(), "Key must not be null!");
 			Assert.notNull(command.getTimeout(), "Timeout must not be null!");
 
-			return cmd.pexpire(command.getKey(), command.getTimeout().getSeconds())
+			return cmd.pexpire(command.getKey(), command.getTimeout().toMillis())
 					.map(value -> new BooleanResponse<>(command, value));
 		}));
 	}
@@ -294,7 +294,7 @@ public Flux<BooleanResponse<ExpireAtCommand>> pExpireAt(Publisher<ExpireAtComman
 			Assert.notNull(command.getKey(), "Key must not be null!");
 			Assert.notNull(command.getExpireAt(), "Expire at must not be null!");
 
-			return cmd.expireat(command.getKey(), command.getExpireAt().toEpochMilli())
+			return cmd.pexpireat(command.getKey(), command.getExpireAt().toEpochMilli())
 					.map(value -> new BooleanResponse<>(command, value));
 		}));
 	}

src/test/java/org/springframework/data/redis/connection/lettuce/LettuceReactiveKeyCommandsTests.java

@@ -278,7 +278,7 @@ public void shouldExpireKeysCorrectly() {
 		assertThat(nativeCommands.ttl(KEY_1), is(greaterThan(8L)));
 	}
 
-	@Test // DATAREDIS-602
+	@Test // DATAREDIS-602, DATAREDIS-1031
 	public void shouldPreciseExpireKeysCorrectly() {
 
 		nativeCommands.set(KEY_1, VALUE_1);
@@ -289,9 +289,10 @@ public void shouldPreciseExpireKeysCorrectly() {
 				.verify();
 
 		assertThat(nativeCommands.ttl(KEY_1), is(greaterThan(8L)));
+		assertThat(nativeCommands.ttl(KEY_1), is(lessThan(11L)));
 	}
 
-	@Test // DATAREDIS-602
+	@Test // DATAREDIS-602, DATAREDIS-1031
 	public void shouldExpireAtKeysCorrectly() {
 
 		nativeCommands.set(KEY_1, VALUE_1);
@@ -303,9 +304,10 @@ public void shouldExpireAtKeysCorrectly() {
 				.verify();
 
 		assertThat(nativeCommands.ttl(KEY_1), is(greaterThan(8L)));
+		assertThat(nativeCommands.ttl(KEY_1), is(lessThan(11L)));
 	}
 
-	@Test // DATAREDIS-602
+	@Test // DATAREDIS-602, DATAREDIS-1031
 	public void shouldPreciseExpireAtKeysCorrectly() {
 
 		nativeCommands.set(KEY_1, VALUE_1);
@@ -317,6 +319,7 @@ public void shouldPreciseExpireAtKeysCorrectly() {
 				.verify();
 
 		assertThat(nativeCommands.ttl(KEY_1), is(greaterThan(8L)));
+		assertThat(nativeCommands.ttl(KEY_1), is(lessThan(11L)));
 	}
 
 	@Test // DATAREDIS-602