Unverified Commit 5d68ad2f authored by Michael Grunder's avatar Michael Grunder Committed by GitHub
Browse files

Merge pull request #1072 from michael-grunder/fix-redis7-unit-tests

Fix tests so they work for Redis 7.0
parents 95a0c128 f4b6ed28
...@@ -152,7 +152,8 @@ jobs: ...@@ -152,7 +152,8 @@ jobs:
- name: Install dependencies - name: Install dependencies
run: | run: |
brew install openssl redis brew install openssl redis@6.2
brew link redis@6.2 --force
- name: Build hiredis - name: Build hiredis
run: USE_SSL=1 make run: USE_SSL=1 make
......
...@@ -5,9 +5,16 @@ REDIS_PORT=${REDIS_PORT:-56379} ...@@ -5,9 +5,16 @@ REDIS_PORT=${REDIS_PORT:-56379}
REDIS_SSL_PORT=${REDIS_SSL_PORT:-56443} REDIS_SSL_PORT=${REDIS_SSL_PORT:-56443}
TEST_SSL=${TEST_SSL:-0} TEST_SSL=${TEST_SSL:-0}
SKIPS_AS_FAILS=${SKIPS_AS_FAILS-:0} SKIPS_AS_FAILS=${SKIPS_AS_FAILS-:0}
ENABLE_DEBUG_CMD=
SSL_TEST_ARGS= SSL_TEST_ARGS=
SKIPS_ARG= SKIPS_ARG=
# We need to enable the DEBUG command for redis-server >= 7.0.0
REDIS_MAJOR_VERSION="$(redis-server --version|awk -F'[^0-9]+' '{ print $2 }')"
if [ "$REDIS_MAJOR_VERSION" -gt "6" ]; then
ENABLE_DEBUG_CMD="enable-debug-command local"
fi
tmpdir=$(mktemp -d) tmpdir=$(mktemp -d)
PID_FILE=${tmpdir}/hiredis-test-redis.pid PID_FILE=${tmpdir}/hiredis-test-redis.pid
SOCK_FILE=${tmpdir}/hiredis-test-redis.sock SOCK_FILE=${tmpdir}/hiredis-test-redis.sock
...@@ -49,8 +56,10 @@ cleanup() { ...@@ -49,8 +56,10 @@ cleanup() {
} }
trap cleanup INT TERM EXIT trap cleanup INT TERM EXIT
cat > ${tmpdir}/redis.conf <<EOF cat > ${tmpdir}/redis.conf <<EOF
daemonize yes daemonize yes
${ENABLE_DEBUG_CMD}
pidfile ${PID_FILE} pidfile ${PID_FILE}
port ${REDIS_PORT} port ${REDIS_PORT}
bind 127.0.0.1 bind 127.0.0.1
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment