Support rename-command (#1817)

Makefile

@@ -91,10 +91,14 @@ define REDIS8_CONF
 daemonize yes
 protected-mode no
 port 6386
+requirepass foobared
 pidfile /tmp/redis8.pid
 logfile /tmp/redis8.log
 save ""
 appendonly no
+rename-command SET NEWSET
+rename-command GET NEWGET
+rename-command DEL ""
 endef
 
 # SENTINELS

pom.xml

@@ -45,7 +45,7 @@
   </scm>
 
 	<properties>
-		<redis-hosts>localhost:6379,localhost:6380,localhost:6381,localhost:6382,localhost:6383,localhost:6384,localhost:6385</redis-hosts>
+		<redis-hosts>localhost:6379,localhost:6380,localhost:6381,localhost:6382,localhost:6383,localhost:6384,localhost:6385,localhost:6386</redis-hosts>
 		<sentinel-hosts>localhost:26379,localhost:26380,localhost:26381</sentinel-hosts>
 		<cluster-hosts>localhost:7379,localhost:7380,localhost:7381,localhost:7382,localhost:7383,localhost:7384,localhost:7385</cluster-hosts>
     	<github.global.server>github</github.global.server>

src/main/java/redis/clients/jedis/Protocol.java

@@ -261,16 +261,20 @@ public static enum Command implements ProtocolCommand {
     GEORADIUSBYMEMBER, GEORADIUSBYMEMBER_RO, MODULE, BITFIELD, HSTRLEN, TOUCH, SWAPDB, MEMORY,
     XADD, XLEN, XDEL, XTRIM, XRANGE, XREVRANGE, XREAD, XACK, XGROUP, XREADGROUP, XPENDING, XCLAIM;
 
-    private final byte[] raw;
+    private byte[] raw;
 
     Command() {
-      raw = SafeEncoder.encode(this.name());
+      rename(this.name());
     }
 
     @Override
     public byte[] getRaw() {
       return raw;
     }
+
+    public void rename(final String newName) {
+      raw = SafeEncoder.encode(newName);
+    }
   }
 
   public static enum Keyword {

src/test/java/redis/clients/jedis/tests/commands/RenameProtocolCommandsTest.java

@@ -0,0 +1,59 @@
+package redis.clients.jedis.tests.commands;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.fail;
+
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Test;
+
+import redis.clients.jedis.HostAndPort;
+import redis.clients.jedis.Jedis;
+import redis.clients.jedis.Protocol.Command;
+import redis.clients.jedis.exceptions.JedisDataException;
+import redis.clients.jedis.tests.HostAndPortUtil;
+
+public class RenameProtocolCommandsTest {
+
+  private Jedis jedis;
+
+  @Before
+  public void setUp() throws Exception {
+    HostAndPort hnp = HostAndPortUtil.getRedisServers().get(7);
+    jedis = new Jedis(hnp.getHost(), hnp.getPort(), 500);
+    jedis.connect();
+    jedis.auth("foobared");
+    jedis.flushAll();
+
+    Command.SET.rename("NEWSET");
+    Command.GET.rename("NEWGET");
+  }
+
+  @After
+  public void tearDown() {
+    Command.SET.rename("SET");
+    Command.GET.rename("GET");
+
+    jedis.disconnect();
+  }
+
+  @Test
+  public void renameCommand() {
+    jedis.set("mykey", "hello world");
+    String value = jedis.get("mykey");
+    assertEquals("hello world", value);
+  }
+
+  @Test
+  public void disabledCommand() {
+    jedis.set("mykey", "hello world2");
+    try {
+      jedis.del("mykey");
+      fail("'DEL' command should be unknown");
+    } catch (JedisDataException expected) {
+      assertEquals("ERR unknown command 'DEL'", expected.getMessage());
+    }
+    String value = jedis.get("mykey");
+    assertEquals("hello world2", value);
+  }
+}