mirror of
https://github.com/gentoo-mirror/guru.git
synced 2026-11-07 00:00:45 -05:00
dev-python/dockerflow: restore correct redis shutdown in test
After introducing <(redis-server || die) in
commit c5006e42be, the $! value after the exec
command does not match the PID of redis-server and therefore the Redis server
will not get shut down using the kill command later in the test. Since the
test will fail anyway when it does not read the expected output line from
the redis-server command, let's omit the || die here again.
Signed-off-by: Katalin Rebhan <kat@knifepoint.net>
This commit is contained in:
1 file changed
+3
-1
@@ -62,7 +62,8 @@ src_compile() {
|
||||
|
||||
python_test() {
|
||||
# Tests expect a running redis server, so let's provide it
|
||||
exec 3< <(redis-server /dev/null || die "failed to start redis server")
|
||||
einfo 'starting Redis...'
|
||||
exec 3< <(redis-server /dev/null)
|
||||
local redis_pid=$!
|
||||
|
||||
# Wait until it's ready
|
||||
@@ -76,6 +77,7 @@ python_test() {
|
||||
nonfatal epytest
|
||||
local epytest_result=$?
|
||||
|
||||
einfo 'stopping Redis...'
|
||||
kill -INT $redis_pid
|
||||
wait $redis_pid
|
||||
|
||||
|
||||
Reference in new issue
Block a user