Skip to content

Commit

Permalink
Ignore testing 'invalid multibulk length' (#2655)
Browse files Browse the repository at this point in the history
Ignoring due to PR redis/redis#9528 is merged in commit redis/redis@93e8534.
  • Loading branch information
sazzad16 committed Dec 12, 2021
1 parent 5183085 commit 5e01cca
Showing 1 changed file with 0 additions and 25 deletions.
25 changes: 0 additions & 25 deletions src/test/java/redis/clients/jedis/tests/ConnectionTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,6 @@

import redis.clients.jedis.Connection;
import redis.clients.jedis.Protocol;
import redis.clients.jedis.Protocol.Command;
import redis.clients.jedis.commands.ProtocolCommand;
import redis.clients.jedis.exceptions.JedisConnectionException;

public class ConnectionTest {
Expand Down Expand Up @@ -62,29 +60,6 @@ public void checkCloseable() {
client.close();
}

@Test
public void getErrorMultibulkLength() throws Exception {
class TestConnection extends Connection {
public TestConnection() {
super("localhost", 6379);
}

@Override
public void sendCommand(ProtocolCommand cmd, byte[]... args) {
super.sendCommand(cmd, args);
}
}

TestConnection conn = new TestConnection();

try {
conn.sendCommand(Command.HMSET, new byte[1024 * 1024 + 1][0]);
fail("Should throw exception");
} catch (JedisConnectionException jce) {
assertEquals("ERR Protocol error: invalid multibulk length", jce.getMessage());
}
}

@Test
public void readWithBrokenConnection() {
class BrokenConnection extends Connection {
Expand Down

0 comments on commit 5e01cca

Please sign in to comment.