Skip to content

Commit 78f0d6f

Browse files
authored
Report an error if the valgrind leak check cannot be executed (#189)
1 parent 6c9bb2e commit 78f0d6f

File tree

1 file changed

+12
-7
lines changed

1 file changed

+12
-7
lines changed

test/test.sh

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -32,16 +32,21 @@ fi
3232
printf "\n${GREEN}Running memory leak checks...${NC}\n"
3333
valgrind --leak-check=full --error-exitcode=200 --log-fd=3 \
3434
bin/tests -f "test/queries/queries-good.sql" -f "test/queries/queries-bad.sql" \
35-
3>&1 >/dev/null 2>/dev/null
35+
3>&1>/dev/null;
36+
3637
MEM_LEAK_RET=$?
38+
RET=1
3739

38-
if [ $MEM_LEAK_RET -ne 200 ]; then
39-
printf "${GREEN}Memory leak check succeeded!${NC}\n"
40-
MEM_LEAK_RET=0
40+
if [ $MEM_LEAK_RET -eq 0 ]; then
41+
printf "${GREEN}Memory leak check succeeded!${NC}\n"
42+
MEM_LEAK_RET=0
43+
RET=0
44+
elif [ $MEM_LEAK_RET -eq 200 ]; then
45+
printf "${RED}Memory leak check failed!${NC}\n"
46+
elif [ $MEM_LEAK_RET -eq 127 ]; then
47+
printf "${RED}Memory leak check failed: command 'valgrind' not found!${NC}\n"
4148
else
42-
MEM_LEAK_RET=1
43-
RET=1
44-
printf "${RED}Memory leak check failed!${NC}\n"
49+
printf "${RED}Memory leak check failed: error code ${MEM_LEAK_RET}!${NC}\n"
4550
fi
4651

4752
#################################################

0 commit comments

Comments
 (0)