ringles commented on a change in pull request #7521:
URL: https://github.com/apache/geode/pull/7521#discussion_r839649525
##########
File path: ci/scripts/execute_redis_tests.sh
##########
@@ -78,7 +78,10 @@ failCount=0
--single unit/type/zset \
--single unit/quit \
--single unit/pubsub \
---single unit/dump
+--single unit/dump \
+--single unit/type/list \
+#--single unit/type/list-2 \ TODO enable when GEODE-10116 merged
+#--single unit/type/list-3 TODO enable when GEODE-10160 is fixed
Review comment:
In terms of a description - all tests have been enabled for the List
commands supported as of now. The only blocking command we support, for
example, is BLPOP so tests for that have been enabled, but not BRPOP,
BRPOPLPUSH, etc. Tests that depend on unsupported command are all disabled, and
a couple of tests that fail due to outstanding bugs are disabled with TODO
comments.
Because LSET returns a slightly different error in one test than native
Redis, the error condition has been modified.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
