Fix Windows builds and CI

* Feature gate all of the cluster tests behind a libevent requirement
* Minor tweak to client_test.c
* Signed correctness fix wrt isdigit for cygwin.

.github/workflows/build.yml

@@ -154,7 +154,7 @@ jobs:
         run: |
           brew install openssl valkey
 
-      - name: Build hiredis
+      - name: Build library
         run: USE_SSL=1 make
 
       - name: Run tests
@@ -174,15 +174,15 @@ jobs:
           choco install -y ninja memurai-developer
 
       - uses: ilammy/msvc-dev-cmd@v1
-      - name: Build hiredis
+      - name: Build library
         run: |
           mkdir build && cd build
-          cmake .. -G Ninja -DCMAKE_BUILD_TYPE=Release -DENABLE_EXAMPLES=ON
+          cmake .. -G Ninja -DCMAKE_BUILD_TYPE=Release
           ninja -v
 
       - name: Run tests
-        run: |
-          ./build/test.exe
+        working-directory: ${{ github.workspace }}/build
+        run: .\tests\client_test.exe
 
       - name: Install Cygwin Action
         uses: cygwin/cygwin-install-action@v2

src/valkeycluster.c

@@ -2494,15 +2494,15 @@ static int command_pre_fragment(valkeyClusterContext *cc, struct cmd *command,
             uint32_t len = 0;
             char *p;
 
-            for (p = sub_kp->end + 1; !isdigit(*p); p++) {
+            for (p = sub_kp->end + 1; !isdigit((unsigned char)*p); p++) {
             }
 
             p = sub_kp->end + 1;
-            while (!isdigit(*p)) {
+            while (!isdigit((unsigned char)*p)) {
                 p++;
             }
 
-            for (; isdigit(*p); p++) {
+            for (; isdigit((unsigned char)*p); p++) {
                 len = len * 10 + (uint32_t)(*p - '0');
             }