Improvement (issue #1352): (#1353)

evalsha should throw specialized exception if redis answers with NOSCRIPT message.

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

@@ -12,6 +12,7 @@
 import redis.clients.jedis.exceptions.JedisConnectionException;
 import redis.clients.jedis.exceptions.JedisDataException;
 import redis.clients.jedis.exceptions.JedisMovedDataException;
+import redis.clients.jedis.exceptions.JedisNoScriptException;
 import redis.clients.util.RedisInputStream;
 import redis.clients.util.RedisOutputStream;
 import redis.clients.util.SafeEncoder;
@@ -22,6 +23,7 @@ public final class Protocol {
   private static final String MOVED_RESPONSE = "MOVED";
   private static final String CLUSTERDOWN_RESPONSE = "CLUSTERDOWN";
   private static final String BUSY_RESPONSE = "BUSY";
+  private static final String NOSCRIPT_RESPONSE = "NOSCRIPT";
 
   public static final String DEFAULT_HOST = "localhost";
   public static final int DEFAULT_PORT = 6379;
@@ -120,6 +122,8 @@ private static void processError(final RedisInputStream is) {
       throw new JedisClusterException(message);
     } else if (message.startsWith(BUSY_RESPONSE)) {
       throw new JedisBusyException(message);
+    } else if (message.startsWith(NOSCRIPT_RESPONSE) ) {
+      throw new JedisNoScriptException(message);
     }
     throw new JedisDataException(message);
   }

src/main/java/redis/clients/jedis/exceptions/JedisNoScriptException.java

@@ -0,0 +1,11 @@
+package redis.clients.jedis.exceptions;
+
+public class JedisNoScriptException extends JedisDataException  {
+  private static final long serialVersionUID = 4674378093072060731L;
+
+  public JedisNoScriptException(final String message) { super(message); }
+
+  public JedisNoScriptException(final Throwable cause) { super(cause); }
+
+  public JedisNoScriptException(final String message, final Throwable cause) { super(message, cause); }
+}

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

@@ -18,6 +18,7 @@
 import redis.clients.jedis.Jedis;
 import redis.clients.jedis.exceptions.JedisConnectionException;
 import redis.clients.jedis.exceptions.JedisDataException;
+import redis.clients.jedis.exceptions.JedisNoScriptException;
 import redis.clients.jedis.tests.utils.ClientKillerUtil;
 import redis.clients.util.SafeEncoder;
 
@@ -127,7 +128,7 @@ public void evalsha() {
     assertEquals("bar", result);
   }
 
-  @Test(expected = JedisDataException.class)
+  @Test(expected = JedisNoScriptException.class)
   public void evalshaShaNotFound() {
     jedis.evalsha("ffffffffffffffffffffffffffffffffffffffff");
   }