Details
-
Task
-
Resolution: Won't Fix
-
Minor - P4
-
None
-
None
Description
The disable_unit_tests conditional test in "run tests" is hard to follow as there is no indentation and the entire function is ignored if it's set to true.
Suggest changing the code block:
if [ ${disable_unit_tests|false} = "false" ]; then
|
|
|
# activate the virtualenv if it has been set up
|
${activate_virtualenv}
|
|
|
if [ -f /proc/self/coredump_filter ]; then
|
# Set the shell process (and its children processes) to dump ELF headers (bit 4),
|
...
|
fi # end if ${disable_unit_tests}
|
Should be refactored to
if [ ${disable_unit_tests|false} = "true" ]; then
|
exit 0
|
fi
|
|
|
# activate the virtualenv if it has been set up
|
${activate_virtualenv}
|
|
|
if [ -f /proc/self/coredump_filter ]; then
|
...
|