Skip to content

Commit e536712

Browse files
authored
Migration from deprecated Long constructor (#3289)
* feat: migration from old new Long constructor * feat: migrating remaining deprecated constructors --------- Co-authored-by: Anuragkillswitch <[email protected]>
1 parent c4506e4 commit e536712

File tree

4 files changed

+12
-12
lines changed

4 files changed

+12
-12
lines changed

src/test/java/redis/clients/jedis/PipeliningTest.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -279,8 +279,8 @@ public void piplineWithError() {
279279
// Response<List<Object>> r3 = p.exec();
280280
// List<Object> result = p.syncAndReturnAll();
281281
//
282-
// assertEquals(new Long(-1), r1.get());
283-
// assertEquals(new Long(-3), r2.get());
282+
// assertEquals(Long.valueOf(-1), r1.get());
283+
// assertEquals(Long.valueOf(-3), r2.get());
284284
//
285285
// assertEquals(4, result.size());
286286
//
@@ -291,11 +291,11 @@ public void piplineWithError() {
291291
// // 4th result is a list with the results from the multi
292292
// @SuppressWarnings("unchecked")
293293
// List<Object> multiResult = (List<Object>) result.get(3);
294-
// assertEquals(new Long(-1), multiResult.get(0));
295-
// assertEquals(new Long(-3), multiResult.get(1));
294+
// assertEquals(Long.valueOf(-1), multiResult.get(0));
295+
// assertEquals(Long.valueOf(-3), multiResult.get(1));
296296
//
297-
// assertEquals(new Long(-1), r3.get().get(0));
298-
// assertEquals(new Long(-3), r3.get().get(1));
297+
// assertEquals(Long.valueOf(-1), r3.get().get(0));
298+
// assertEquals(Long.valueOf(-3), r3.get().get(1));
299299
//
300300
// }
301301
//

src/test/java/redis/clients/jedis/commands/jedis/ObjectCommandsTest.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -48,11 +48,11 @@ public void tearDown() throws Exception {
4848
public void objectRefcount() {
4949
jedis.lpush(key, "hello world");
5050
Long refcount = jedis.objectRefcount(key);
51-
assertEquals(new Long(1), refcount);
51+
assertEquals(Long.valueOf(1), refcount);
5252

5353
// Binary
5454
refcount = jedis.objectRefcount(binaryKey);
55-
assertEquals(new Long(1), refcount);
55+
assertEquals(Long.valueOf(1), refcount);
5656

5757
}
5858

@@ -79,11 +79,11 @@ public void objectIdletime() throws InterruptedException {
7979
jedis.lpush(key, "hello world");
8080

8181
Long time = jedis.objectIdletime(key);
82-
assertEquals(new Long(0), time);
82+
assertEquals(Long.valueOf(0), time);
8383

8484
// Binary
8585
time = jedis.objectIdletime(binaryKey);
86-
assertEquals(new Long(0), time);
86+
assertEquals(Long.valueOf(0), time);
8787
}
8888

8989
@Test

src/test/java/redis/clients/jedis/commands/jedis/PublishSubscribeCommandsTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,7 @@ public void onPSubscribe(String pattern, int subscribedChannels) {
185185
if (count == 3) {
186186
Jedis otherJedis = createJedis();
187187
Long numPatterns = otherJedis.pubsubNumPat();
188-
assertEquals(new Long(2l), numPatterns);
188+
assertEquals(Long.valueOf(2l), numPatterns);
189189
punsubscribe();
190190
}
191191
}

src/test/java/redis/clients/jedis/commands/jedis/ScriptingCommandsTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ public void evalInt() {
111111

112112
Long response = (Long) jedis.eval(script, keys, new ArrayList<String>());
113113

114-
assertEquals(new Long(2), response);
114+
assertEquals(Long.valueOf(2), response);
115115
}
116116

117117
@Test

0 commit comments

Comments
 (0)