Polishing.

Add author tags and issue number comment for new tests.

See #3250

Signed-off-by: Chris Bono <chris.bono@broadcom.com>

Author: onobc

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

@@ -1289,7 +1289,7 @@ default Mono<Long> bitPos(ByteBuffer key, boolean bit, Range<Long> range) {
 	}
 
 	/**
-	 * Emmit the position of the first bit set to given {@code bit} in a string. Get the length of the value stored at
+	 * Emit the position of the first bit set to given {@code bit} in a string. Get the length of the value stored at
 	 * {@literal key}.
 	 *
 	 * @param commands must not be {@literal null}.

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

@@ -32,6 +32,7 @@
  * @author Christoph Strobl
  * @author Mark Paluch
  * @author Marcin Grzejszczak
+ * @author Viktoriya Kutsarova
  * @see RedisCommands
  */
 @NullUnmarked

src/main/java/org/springframework/data/redis/connection/jedis/JedisConverters.java

@@ -109,6 +109,7 @@
  * @author Guy Korland
  * @author dengliming
  * @author John Blum
+ * @author Viktoriya Kutsarova
  */
 @SuppressWarnings("ConstantConditions")
 abstract class JedisConverters extends Converters {

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

@@ -50,6 +50,7 @@
  * @author Michele Mancioppi
  * @author John Blum
  * @author Marcin Grzejszczak
+ * @author Viktoriya Kutsarova
  * @since 2.0
  */
 class LettuceReactiveStringCommands implements ReactiveStringCommands {

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

@@ -40,6 +40,7 @@
  * @author dengliming
  * @author John Blum
  * @author Marcin Grzejszczak
+ * @author Viktoriya Kutsarova
  * @since 2.0
  */
 @NullUnmarked

src/test/java/org/springframework/data/redis/connection/AbstractConnectionIntegrationTests.java

@@ -117,6 +117,7 @@
  * @author Roman Osadchuk
  * @author Tihomir Mateev
  * @author Jeonggyu Choi
+ * @author Viktoriya Kutsarova
  */
 public abstract class AbstractConnectionIntegrationTests {
 
@@ -571,7 +572,7 @@ void testBitOpNotMultipleSources() {
 				.isThrownBy(() -> connection.bitOp(BitOperation.NOT, "key3", "key1", "key2"));
 	}
 
-	@Test
+	@Test // GH-3250
 	void testBitOpDiff() {
 
 		actual.add(connection.set("key1", "foobar"));
@@ -580,7 +581,7 @@ void testBitOpDiff() {
 		verifyResults(Arrays.asList(Boolean.TRUE, Boolean.TRUE, 6L));
 	}
 
-	@Test
+	@Test // GH-3250
 	void testBitOpDiff1() {
 
 		actual.add(connection.set("key1", "foobar"));
@@ -589,7 +590,7 @@ void testBitOpDiff1() {
 		verifyResults(Arrays.asList(Boolean.TRUE, Boolean.TRUE, 6L));
 	}
 
-	@Test
+	@Test // GH-3250
 	void testBitOpAndor() {
 
 		actual.add(connection.set("key1", "foo"));
@@ -598,7 +599,7 @@ void testBitOpAndor() {
 		verifyResults(Arrays.asList(Boolean.TRUE, Boolean.TRUE, 3L));
 	}
 
-	@Test
+	@Test // GH-3250
 	void testBitOpOne() {
 
 		actual.add(connection.set("key1", "foo"));

src/test/java/org/springframework/data/redis/connection/DefaultStringRedisConnectionPipelineTests.java

@@ -38,6 +38,7 @@
  * @author Ninad Divadkar
  * @author Mark Paluch
  * @author dengliming
+ * @author Viktoriya Kutsarova
  */
 public class DefaultStringRedisConnectionPipelineTests extends DefaultStringRedisConnectionTests {
 
@@ -1010,85 +1011,85 @@ public void testBitOp() {
 		super.testBitOp();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpOrBytes() {
 		doReturn(Collections.singletonList(5L)).when(nativeConnection).closePipeline();
 		super.testBitOpOrBytes();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpOr() {
 		doReturn(Collections.singletonList(5L)).when(nativeConnection).closePipeline();
 		super.testBitOpOr();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpXorBytes() {
 		doReturn(Collections.singletonList(5L)).when(nativeConnection).closePipeline();
 		super.testBitOpXorBytes();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpXor() {
 		doReturn(Collections.singletonList(5L)).when(nativeConnection).closePipeline();
 		super.testBitOpXor();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpNotBytes() {
 		doReturn(Collections.singletonList(5L)).when(nativeConnection).closePipeline();
 		super.testBitOpNotBytes();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpNot() {
 		doReturn(Collections.singletonList(5L)).when(nativeConnection).closePipeline();
 		super.testBitOpNot();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpDiffBytes() {
 		doReturn(Collections.singletonList(5L)).when(nativeConnection).closePipeline();
 		super.testBitOpDiffBytes();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpDiff() {
 		doReturn(Collections.singletonList(5L)).when(nativeConnection).closePipeline();
 		super.testBitOpDiff();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpDiff1Bytes() {
 		doReturn(Collections.singletonList(5L)).when(nativeConnection).closePipeline();
 		super.testBitOpDiff1Bytes();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpDiff1() {
 		doReturn(Collections.singletonList(5L)).when(nativeConnection).closePipeline();
 		super.testBitOpDiff1();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpAndorBytes() {
 		doReturn(Collections.singletonList(5L)).when(nativeConnection).closePipeline();
 		super.testBitOpAndorBytes();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpAndor() {
 		doReturn(Collections.singletonList(5L)).when(nativeConnection).closePipeline();
 		super.testBitOpAndor();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpOneBytes() {
 		doReturn(Collections.singletonList(5L)).when(nativeConnection).closePipeline();
 		super.testBitOpOneBytes();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpOne() {
 		doReturn(Collections.singletonList(5L)).when(nativeConnection).closePipeline();
 		super.testBitOpOne();

src/test/java/org/springframework/data/redis/connection/DefaultStringRedisConnectionPipelineTxTests.java

@@ -37,6 +37,7 @@
  * @author Ninad Divadkar
  * @author Mark Paluch
  * @author dengliming
+ * @author Viktoriya Kutsarova
  */
 public class DefaultStringRedisConnectionPipelineTxTests extends DefaultStringRedisConnectionTxTests {
 
@@ -1025,85 +1026,85 @@ public void testBitOp() {
 		super.testBitOp();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpOrBytes() {
 		doReturn(Collections.singletonList(Collections.singletonList(5L))).when(nativeConnection).closePipeline();
 		super.testBitOpOrBytes();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpOr() {
 		doReturn(Collections.singletonList(Collections.singletonList(5L))).when(nativeConnection).closePipeline();
 		super.testBitOpOr();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpXorBytes() {
 		doReturn(Collections.singletonList(Collections.singletonList(5L))).when(nativeConnection).closePipeline();
 		super.testBitOpXorBytes();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpXor() {
 		doReturn(Collections.singletonList(Collections.singletonList(5L))).when(nativeConnection).closePipeline();
 		super.testBitOpXor();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpNotBytes() {
 		doReturn(Collections.singletonList(Collections.singletonList(5L))).when(nativeConnection).closePipeline();
 		super.testBitOpNotBytes();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpNot() {
 		doReturn(Collections.singletonList(Collections.singletonList(5L))).when(nativeConnection).closePipeline();
 		super.testBitOpNot();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpDiffBytes() {
 		doReturn(Collections.singletonList(Collections.singletonList(5L))).when(nativeConnection).closePipeline();
 		super.testBitOpDiffBytes();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpDiff() {
 		doReturn(Collections.singletonList(Collections.singletonList(5L))).when(nativeConnection).closePipeline();
 		super.testBitOpDiff();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpDiff1Bytes() {
 		doReturn(Collections.singletonList(Collections.singletonList(5L))).when(nativeConnection).closePipeline();
 		super.testBitOpDiff1Bytes();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpDiff1() {
 		doReturn(Collections.singletonList(Collections.singletonList(5L))).when(nativeConnection).closePipeline();
 		super.testBitOpDiff1();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpAndorBytes() {
 		doReturn(Collections.singletonList(Collections.singletonList(5L))).when(nativeConnection).closePipeline();
 		super.testBitOpAndorBytes();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpAndor() {
 		doReturn(Collections.singletonList(Collections.singletonList(5L))).when(nativeConnection).closePipeline();
 		super.testBitOpAndor();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpOneBytes() {
 		doReturn(Collections.singletonList(Collections.singletonList(5L))).when(nativeConnection).closePipeline();
 		super.testBitOpOneBytes();
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpOne() {
 		doReturn(Collections.singletonList(Collections.singletonList(5L))).when(nativeConnection).closePipeline();
 		super.testBitOpOne();

src/test/java/org/springframework/data/redis/connection/DefaultStringRedisConnectionTests.java

@@ -78,6 +78,7 @@
  * @author Mark Paluch
  * @author dengliming
  * @author ihaohong
+ * @author Viktoriya Kutsarova
  */
 @ExtendWith(MockitoExtension.class)
 @MockitoSettings(strictness = Strictness.LENIENT)
@@ -1308,56 +1309,56 @@ public void testBitOpNot() {
 		verifyResults(Collections.singletonList(5L));
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpDiffBytes() {
 		doReturn(5L).when(nativeConnection).bitOp(BitOperation.DIFF, fooBytes, barBytes);
 		actual.add(connection.bitOp(BitOperation.DIFF, fooBytes, barBytes));
 		verifyResults(Collections.singletonList(5L));
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpDiff() {
 		doReturn(5L).when(nativeConnection).bitOp(BitOperation.DIFF, fooBytes, barBytes);
 		actual.add(connection.bitOp(BitOperation.DIFF, foo, bar));
 		verifyResults(Collections.singletonList(5L));
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpDiff1Bytes() {
 		doReturn(5L).when(nativeConnection).bitOp(BitOperation.DIFF1, fooBytes, barBytes);
 		actual.add(connection.bitOp(BitOperation.DIFF1, fooBytes, barBytes));
 		verifyResults(Collections.singletonList(5L));
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpDiff1() {
 		doReturn(5L).when(nativeConnection).bitOp(BitOperation.DIFF1, fooBytes, barBytes);
 		actual.add(connection.bitOp(BitOperation.DIFF1, foo, bar));
 		verifyResults(Collections.singletonList(5L));
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpAndorBytes() {
 		doReturn(5L).when(nativeConnection).bitOp(BitOperation.ANDOR, fooBytes, barBytes);
 		actual.add(connection.bitOp(BitOperation.ANDOR, fooBytes, barBytes));
 		verifyResults(Collections.singletonList(5L));
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpAndor() {
 		doReturn(5L).when(nativeConnection).bitOp(BitOperation.ANDOR, fooBytes, barBytes);
 		actual.add(connection.bitOp(BitOperation.ANDOR, foo, bar));
 		verifyResults(Collections.singletonList(5L));
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpOneBytes() {
 		doReturn(5L).when(nativeConnection).bitOp(BitOperation.ONE, fooBytes, barBytes);
 		actual.add(connection.bitOp(BitOperation.ONE, fooBytes, barBytes));
 		verifyResults(Collections.singletonList(5L));
 	}
 
-	@Test
+	@Test // GH-3250
 	public void testBitOpOne() {
 		doReturn(5L).when(nativeConnection).bitOp(BitOperation.ONE, fooBytes, barBytes);
 		actual.add(connection.bitOp(BitOperation.ONE, foo, bar));