Let's switch back again to set -e.

Signed-off-by: Petr "Stone" Hracek <[email protected]>

Author: phracek

test.sh

@@ -26,6 +26,7 @@ failed_version() {
 
 analyze_logs_by_logdetective() {
   # logdetective should not break the test functionality
+  # Therefore `set +e` is setup
   set +e
   local log_file_name="$1"
   echo "Sending failed log by fpaste command to paste bin."
@@ -46,21 +47,25 @@ analyze_logs_by_logdetective() {
     echo "ERROR: Failed to analyze log file by logdetective server."
     cat "${logdetective_test_file}"
     echo "-------- LOGDETECTIVE TEST LOG ANALYSIS FAILED --------"
+    # Let's switch it back
     set -e
     return
   fi
-  set -e
   jq -rC '.explanation.text' < "${logdetective_test_file}"
   # This part of code is from https://github.com/teemtee/tmt/blob/main/tmt/steps/scripts/tmt-file-submit
   if [ -z "$TMT_TEST_PIDFILE" ]; then
     echo "File submit to data dir can be used only in the context of a running test."
+    # Let's switch it back
+    set -e
     return
   fi
   # This variable is set by tmt
   [ -d "$TMT_TEST_DATA" ] || mkdir -p "$TMT_TEST_DATA"
   cp -f "${logdetective_test_file}" "$TMT_TEST_DATA"
   echo "File '${logdetective_test_file}' stored to '$TMT_TEST_DATA'."
   echo "-------- LOGDETECTIVE TEST LOG ANALYSIS FINISHED --------"
+  # Let's switch it back
+  set -e
 }
 
 # This adds backwards compatibility if only single version needs to be testing