fixup: handle review comments and other findings.

Let connect error logs be printed to stdout since this step is part
of the testcase.

tests/clusterclient.c

@@ -54,24 +54,24 @@ void eventCallback(const redisClusterContext *cc, int event, void *privdata) {
 }
 
 int main(int argc, char **argv) {
-    int use_cluster_slots = 1;
     int show_events = 0;
+    int use_cluster_slots = 1;
     int send_to_all = 0;
 
     int argindex;
     for (argindex = 1; argindex < argc && argv[argindex][0] == '-';
          argindex++) {
-        if (strcmp(argv[argindex], "--use-cluster-nodes") == 0) {
-            use_cluster_slots = 0;
-        } else if (strcmp(argv[argindex], "--events") == 0) {
+        if (strcmp(argv[argindex], "--events") == 0) {
             show_events = 1;
+        } else if (strcmp(argv[argindex], "--use-cluster-nodes") == 0) {
+            use_cluster_slots = 0;
         } else {
             fprintf(stderr, "Unknown argument: '%s'\n", argv[argindex]);
         }
     }
 
     if (argindex >= argc) {
-        fprintf(stderr, "Usage: clusterclient [--events] HOST:PORT\n");
+        fprintf(stderr, "Usage: clusterclient [--events] [--use-cluster-nodes] HOST:PORT\n");
         exit(1);
     }
     const char *initnode = argv[argindex];
@@ -89,8 +89,8 @@ int main(int argc, char **argv) {
     }
 
     if (redisClusterConnect2(cc) != REDIS_OK) {
-        fprintf(stderr, "Connect error: %s\n", cc->errstr);
-        exit(2);
+        printf("Connect error: %s\n", cc->errstr);
+        exit(1);
     }
 
     char command[256];

tests/scripts/connect-error-using-cluster-nodes-test.sh

@@ -31,7 +31,7 @@ server=$!
 wait $syncpid;
 
 # Run client and use CLUSTER NODES to get topology
-timeout 3s "$clientprog" --use-cluster-nodes 127.0.0.1:7400 > "$testname.out" 2>&1
+timeout 3s "$clientprog" --use-cluster-nodes 127.0.0.1:7400 > "$testname.out"
 clientexit=$?
 
 # Wait for server to exit
@@ -44,7 +44,7 @@ if [ $serverexit -ne 0 ]; then
 fi
 
 # Check exit status on client, which SHOULD fail.
-if [ $clientexit -ne 2 ]; then
+if [ $clientexit -ne 1 ]; then
     echo "$clientprog exited with status $clientexit"
     exit $clientexit
 fi