tests: fix shellcheck warnings in plugins/src.ctf.fs/query/test_query_metadata_info
authorSimon Marchi <simon.marchi@efficios.com>
Tue, 25 Feb 2020 19:08:47 +0000 (14:08 -0500)
committerSimon Marchi <simon.marchi@efficios.com>
Wed, 26 Feb 2020 14:10:00 +0000 (09:10 -0500)
Fix:

    In test_query_metadata_info line 32:
    this_dir_build="$BT_TESTS_BUILDDIR/$this_dir_relative"
    ^------------^ SC2034: this_dir_build appears unused. Verify use (or export if used externally).

    In test_query_metadata_info line 41:
    local temp_stdout_output_file="$(mktemp -t actual_stdout.XXXXXX)"
  ^---------------------^ SC2155: Declare and assign separately to avoid masking return values.

    In test_query_metadata_info line 42:
    local temp_stderr_output_file="$(mktemp -t actual_stderr.XXXXXX)"
  ^---------------------^ SC2155: Declare and assign separately to avoid masking return values.

Change-Id: I747d8092bf4625a748afb313b094e6782824c407
Signed-off-by: Simon Marchi <simon.marchi@efficios.com>
Reported-by: shellcheck
Reviewed-on: https://review.lttng.org/c/babeltrace/+/3138
Tested-by: jenkins <jenkins@lttng.org>
tests/plugins/src.ctf.fs/query/test_query_metadata_info

index e52e7fa072d96a383f0f8952b2b9f2fbe153bc7b..3d1e79b16354187db173a31e3900475cfb3493a8 100755 (executable)
@@ -29,7 +29,6 @@ fi
 source "$UTILSSH"
 
 this_dir_relative="plugins/src.ctf.fs/query"
-this_dir_build="$BT_TESTS_BUILDDIR/$this_dir_relative"
 succeed_trace_dir="$BT_CTF_TRACES_PATH/succeed"
 expect_dir="$BT_TESTS_DATADIR/$this_dir_relative"
 
@@ -38,10 +37,13 @@ test_query_metadata_info() {
        local ret=0
        local trace_path="$succeed_trace_dir/$name"
        local expected_stdout="$expect_dir/metadata-info-$name.expect"
-       local temp_stdout_output_file="$(mktemp -t actual_stdout.XXXXXX)"
-       local temp_stderr_output_file="$(mktemp -t actual_stderr.XXXXXX)"
+       local temp_stdout_output_file
+       local temp_stderr_output_file
        local query=("query" "src.ctf.fs" "metadata-info" "--params" "path=\"$trace_path\"")
 
+       temp_stdout_output_file="$(mktemp -t actual_stdout.XXXXXX)"
+       temp_stderr_output_file="$(mktemp -t actual_stderr.XXXXXX)"
+
        bt_cli "$temp_stdout_output_file" "$temp_stderr_output_file" \
                "${query[@]}"
 
This page took 0.027343 seconds and 4 git commands to generate.