Commit 5fec1494 authored by Maciej Szulik's avatar Maciej Szulik

Return information about which int tests failed in the summary

parent f611b322
...@@ -106,7 +106,7 @@ function record_command() { ...@@ -106,7 +106,7 @@ function record_command() {
juLog -output="${output}" -class="test-cmd" -name="${name}" "$@" juLog -output="${output}" -class="test-cmd" -name="${name}" "$@"
if [[ $? -ne 0 ]]; then if [[ $? -ne 0 ]]; then
echo "Error when running ${name}" echo "Error when running ${name}"
foundError="True" foundError="${foundError}""${name}"", "
fi fi
set -o nounset set -o nounset
...@@ -4537,7 +4537,7 @@ run_impersonation_tests() { ...@@ -4537,7 +4537,7 @@ run_impersonation_tests() {
# Requires an env var SUPPORTED_RESOURCES which is a comma separated list of # Requires an env var SUPPORTED_RESOURCES which is a comma separated list of
# resources for which tests should be run. # resources for which tests should be run.
runTests() { runTests() {
foundError="False" foundError=""
if [ -z "${SUPPORTED_RESOURCES:-}" ]; then if [ -z "${SUPPORTED_RESOURCES:-}" ]; then
echo "Need to set SUPPORTED_RESOURCES env var. It is a list of resources that are supported and hence should be tested. Set it to (*) to test all resources" echo "Need to set SUPPORTED_RESOURCES env var. It is a list of resources that are supported and hence should be tested. Set it to (*) to test all resources"
...@@ -5035,8 +5035,8 @@ runTests() { ...@@ -5035,8 +5035,8 @@ runTests() {
kube::test::clear_all kube::test::clear_all
if [ "$foundError" == "True" ]; then if [ ! -z "${foundError}" ]; then
echo "TEST FAILED" echo "FAILED TESTS: ""${foundError}"
exit 1 exit 1
fi fi
} }
......
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