fix string deserialize exception (#74)

* fix deserialize string exception

* add wrongtype tests

Author: tuhuynh27

core/src/main/java/dev/keva/core/command/mapping/CommandMapper.java

@@ -19,6 +19,7 @@
 import lombok.Getter;
 import lombok.extern.slf4j.Slf4j;
 import lombok.val;
+import org.apache.commons.lang3.SerializationException;
 import org.reflections.Reflections;
 
 import java.lang.reflect.InvocationTargetException;
@@ -120,7 +121,7 @@ public void init() {
                         } catch (Exception e) {
                             log.error(e.getMessage(), e);
                             if (e instanceof InvocationTargetException) {
-                                if (e.getCause() instanceof ClassCastException) {
+                                if (e.getCause() instanceof SerializationException || e.getCause() instanceof ClassCastException) {
                                     return new ErrorReply("ERR WRONGTYPE Operation against a key holding the wrong kind of value");
                                 }
                                 return new ErrorReply("ERR " + e.getCause().getMessage());

core/src/test/java/dev/keva/core/server/AbstractServerTest.java

@@ -106,6 +106,39 @@ void del() {
         }
     }
 
+    @Test
+    void wrongTypeSet() {
+        try {
+            val setAbc = jedis.set("abc", "123");
+            assertEquals("OK", setAbc);
+            jedis.lpush("abc", "123");
+        } catch (Exception e) {
+            assertEquals(JedisDataException.class, e.getClass());
+        }
+    }
+
+    @Test
+    void wrongTypeSet2() {
+        try {
+            val setAbc = jedis.lpush("abc", "123");
+            assertEquals(1, setAbc);
+            jedis.hset("abc", "key", "val");
+        } catch (Exception e) {
+            assertEquals(JedisDataException.class, e.getClass());
+        }
+    }
+
+    @Test
+    void wrongTypeSet3() {
+        try {
+            val setAbc = jedis.hset("abc", "key", "val");
+            assertEquals(1, setAbc);
+            jedis.lpush("abc", "123");
+        } catch (Exception e) {
+            assertEquals(JedisDataException.class, e.getClass());
+        }
+    }
+
     @Test
     void transaction() {
         try {