comment ping changes + update redissuite skip test list/comments

Author: sjpotter

src/commands.c

@@ -72,9 +72,8 @@ static const CommandSpec commands[] = {
 
  /* Admin commands - bypassed */
     {"auth",                        CMD_SPEC_DONT_INTERCEPT                      },
-    /* queued by multi, need to intercept */
-//    {"ping",                        CMD_SPEC_DONT_INTERCEPT                      },
-    {"ping",                       CMD_SPEC_READONLY                             },
+    /* queued by multi, need to intercept in multi, but can't do that right now with redis */
+    {"ping",                        CMD_SPEC_DONT_INTERCEPT                      },
     {"hello",                       CMD_SPEC_DONT_INTERCEPT                      },
     {"module",                      CMD_SPEC_DONT_INTERCEPT                      },
     {"config",                      CMD_SPEC_DONT_INTERCEPT                      },

tests/redis-suite/skip.txt

@@ -41,7 +41,6 @@ UNLINK can reclaim memory in background
 -- ACL test fails because we prepend "raft" string to the command
 Script ACL check
 
-
 -- MULTI/EXEC is currently read-write in RedisRaft 
 EXEC with only read commands should not be rejected when OOM
 
@@ -52,10 +51,14 @@ EXEC with only read commands should not be rejected when OOM
 MULTI with SAVE
 MULTI with SHUTDOWN
 MULTI with config error
-
--- pubsub can't allow ping to be intercepted, but needs to be intercepted for multi tests
--- where part of the multi response
-/.*Sub PING on.*
+-- same as above, but these all relate to PING usage inside of MULTI
+MULTI / EXEC basics
+EXEC works on WATCHed key not modified
+After successful EXEC key is no longer watched
+After failed EXEC key is no longer watched
+It is possible to UNWATCH
+FLUSHALL does not touch non affected keys
+FLUSHDB does not touch non affected keys
 
 -- After fixing this: https://github.com/RedisLabs/redisraft/issues/367
 -- We don't need to skip this test as it doesn't actually configure a replica.