ci: skip valgrind on version matrix

Author: aviggiano

.github/workflows/ci.yml

@@ -41,6 +41,8 @@ jobs:
       run: sudo apt-get install valgrind
     - name: Test
       run: ./test.sh
+      env:
+        USE_VALGRIND: "0"
 
   coverage:
     name: Generate and upload coverage

test.sh

@@ -5,7 +5,7 @@ set -eu
 . ./tests/helper.sh
 
 function unit() {
-  if [[ "$OSTYPE" == "darwin"* ]]; then
+  if [[ "${USE_VALGRIND:-1}" != "1" ]] || [[ "$OSTYPE" == "darwin"* ]]; then
     ./build/unit
   else
     valgrind --leak-check=full --error-exitcode=1 ./build/unit
@@ -17,26 +17,42 @@ function unit() {
 function integration_1() {
   stop_redis
   rm dump.rdb 2>/dev/null || true
-  start_redis --valgrind
+  if [[ "${USE_VALGRIND:-1}" == "1" ]]; then
+    start_redis --valgrind
+  else
+    start_redis
+  fi
   ./tests/integration_1.sh
   stop_redis
   echo "All integration (1) tests passed"
 }
 
 function integration_2() {
   stop_redis
-  start_redis --valgrind --aof
+  if [[ "${USE_VALGRIND:-1}" == "1" ]]; then
+    start_redis --valgrind --aof
+  else
+    start_redis --aof
+  fi
   ./tests/integration_1.sh
   stop_redis
 
   # Test RDB load
-  start_redis --valgrind
+  if [[ "${USE_VALGRIND:-1}" == "1" ]]; then
+    start_redis --valgrind
+  else
+    start_redis
+  fi
   ./tests/integration_2.sh
   stop_redis
   rm dump.rdb 2>/dev/null || true
 
   # Test AOF load
-  start_redis --valgrind --aof
+  if [[ "${USE_VALGRIND:-1}" == "1" ]]; then
+    start_redis --valgrind --aof
+  else
+    start_redis --aof
+  fi
   ./tests/integration_2.sh
   stop_redis
   rm appendonly.aof 2>/dev/null || true
@@ -47,7 +63,11 @@ function integration_2() {
 function integration_3() {
   stop_redis
   rm dump.rdb 2>/dev/null || true
-  start_redis --valgrind
+  if [[ "${USE_VALGRIND:-1}" == "1" ]]; then
+    start_redis --valgrind
+  else
+    start_redis
+  fi
   ./tests/integration_3.sh
   stop_redis
   echo "All integration (3) tests passed"