tests: declare and assign variables separately in test_exit_status

Fixes these shellcheck errors:

    In test_exit_status line 34:
            local actual_stdout=$(mktemp -t test_cli_exit_status_stdout_actual.XXXXXX)
                  ^-----------^ SC2155: Declare and assign separately to avoid masking return values.

    In test_exit_status line 35:
            local actual_stderr=$(mktemp -t test_cli_exit_status_stderr_actual.XXXXXX)
                  ^-----------^ SC2155: Declare and assign separately to avoid masking return values.

    In test_exit_status line 53:
            local actual_stdout=$(mktemp -t test_cli_exit_status_stdout_actual.XXXXXX)
                  ^-----------^ SC2155: Declare and assign separately to avoid masking return values.

    In test_exit_status line 54:
            local actual_stderr=$(mktemp -t test_cli_exit_status_stderr_actual.XXXXXX)
                  ^-----------^ SC2155: Declare and assign separately to avoid masking return values.

    In test_exit_status line 72:
            local actual_stdout=$(mktemp -t test_cli_exit_status_stdout_actual.XXXXXX)
                  ^-----------^ SC2155: Declare and assign separately to avoid masking return values.

    In test_exit_status line 73:
            local actual_stderr=$(mktemp -t test_cli_exit_status_stderr_actual.XXXXXX)
                  ^-----------^ SC2155: Declare and assign separately to avoid masking return values.

Reported-by: shellcheck
Change-Id: Iddb1c4e8b0a3bc1c37cc08a37b1103d97b8560e7
Signed-off-by: Simon Marchi <simon.marchi@efficios.com>
Reviewed-on: https://review.lttng.org/c/babeltrace/+/2416
Tested-by: jenkins <jenkins@lttng.org>

diff --git a/tests/cli/test_exit_status b/tests/cli/test_exit_status
index 3e6f5029217e4225a5ea7ffa1d4749632a59e479..01b0632e428c795e7dfa67aa64002e70a376be6b 100755 (executable)
--- a/tests/cli/test_exit_status
+++ b/tests/cli/test_exit_status
@@ -31,8 +31,11 @@ source_name="src.test_exit_status.StatusSrc"
 
 test_interrupted_graph() {
        local cli_args=("--plugin-path=$data_dir" "-c" "$source_name" "-p" "case=\"INTERRUPTED\"")
-       local actual_stdout=$(mktemp -t test_cli_exit_status_stdout_actual.XXXXXX)
-       local actual_stderr=$(mktemp -t test_cli_exit_status_stderr_actual.XXXXXX)
+       local actual_stdout
+       local actual_stderr
+
+       actual_stdout=$(mktemp -t test_cli_exit_status_stdout_actual.XXXXXX)
+       actual_stderr=$(mktemp -t test_cli_exit_status_stderr_actual.XXXXXX)
 
        bt_cli "$actual_stdout" "$actual_stderr" "${cli_args[@]}"
 
@@ -50,8 +53,11 @@ test_interrupted_graph() {
 
 test_error_graph() {
        local cli_args=("--plugin-path=$data_dir" "-c" "$source_name" "-p" "case=\"ERROR\"")
-       local actual_stdout=$(mktemp -t test_cli_exit_status_stdout_actual.XXXXXX)
-       local actual_stderr=$(mktemp -t test_cli_exit_status_stderr_actual.XXXXXX)
+       local actual_stdout
+       local actual_stderr
+
+       actual_stdout=$(mktemp -t test_cli_exit_status_stdout_actual.XXXXXX)
+       actual_stderr=$(mktemp -t test_cli_exit_status_stderr_actual.XXXXXX)
 
        bt_cli "$actual_stdout" "$actual_stderr" "${cli_args[@]}"
 
@@ -69,8 +75,11 @@ test_error_graph() {
 
 test_stop_graph() {
        local cli_args=("--plugin-path=$data_dir" "-c" "$source_name" "-p" "case=\"STOP\"")
-       local actual_stdout=$(mktemp -t test_cli_exit_status_stdout_actual.XXXXXX)
-       local actual_stderr=$(mktemp -t test_cli_exit_status_stderr_actual.XXXXXX)
+       local actual_stdout
+       local actual_stderr
+
+       actual_stdout=$(mktemp -t test_cli_exit_status_stdout_actual.XXXXXX)
+       actual_stderr=$(mktemp -t test_cli_exit_status_stderr_actual.XXXXXX)
 
        bt_cli "$actual_stdout" "$actual_stderr" "${cli_args[@]}"