tests: cleanup cli/convert/test_convert_args

A few cleanups:

- Use bt_cli instead of running the babeltrace binary by hand.
- Give a base name to the temporary files.
- Split CLI arguments into an array before passing them to bt_cli (this
  avoids a shellcheck warning).

bt_cli prints the executed command line, so it becomes unnecessary to
print it manually in this test, which allows simplifying things a bit.

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

diff --git a/tests/cli/convert/test_convert_args b/tests/cli/convert/test_convert_args
index f7fbeb1ca0a14ea659f839cccd65607782966e09..5b20d46b01d9a4acce59a01f7a83bff3f9f4fb39 100755 (executable)
--- a/tests/cli/convert/test_convert_args
+++ b/tests/cli/convert/test_convert_args
@@ -26,30 +26,28 @@ fi
 # shellcheck source=../../utils/utils.sh
 source "$UTILSSH"
 
-test_head_comment=0
-tmp_stderr=$(mktemp)
+tmp_stdout=$(mktemp -t test_convert_args_stdout.XXXXXX)
+tmp_stderr=$(mktemp -t test_convert_args_stderr.XXXXXX)
 
 test_bt_convert_run_args() {
        local what="$1"
        local convert_args="$2"
        local expected_run_args="$3"
 
-       # execute convert command
-       local run_args="$("$BT_TESTS_BT2_BIN" convert --run-args $convert_args)"
+       local run_args
 
-       # check result
-       if [ "$test_head_comment" = 1 ]; then
-               comment "convert args: $convert_args"
-       fi
+       # Split argument string into array.
+       IFS=' ' read -ra convert_args_array <<< "$convert_args"
 
-       if [ "$run_args" = "$expected_run_args" ]; then
-               pass "ARGS: $what"
-       else
-               fail "ARGS: $what"
-               diag "ARGS:     $convert_args"
-               diag "EXPECTED: $expected_run_args"
-               diag "GOT:      $run_args"
-       fi
+       # Execute convert command.
+       bt_cli "${tmp_stdout}" "${tmp_stderr}" convert --run-args "${convert_args_array[@]}"
+       ok $? "${what}: success exit status"
+
+       run_args=$(cat "${tmp_stdout}")
+
+       # Verify output run args.
+       [ "$run_args" = "$expected_run_args" ]
+       ok $? "${what}: run arguments"
 }
 
 test_bt_convert_fails() {
@@ -57,22 +55,16 @@ test_bt_convert_fails() {
        local convert_args="$2"
        local expected_error_str="${3:-}"
 
-       # execute convert command
-       "$BT_TESTS_BT2_BIN" convert --run-args $convert_args >/dev/null 2>"${tmp_stderr}"
+       # Split argument string into array.
+       IFS=' ' read -ra convert_args_array <<< "$convert_args"
 
-       local status=$?
+       # Execute convert command.
+       bt_cli "${tmp_stdout}" "${tmp_stderr}" convert --run-args "${convert_args_array[@]}"
+       isnt "$?" 0 "failure exit status"
 
-       # check result
-       if [ "$test_head_comment" = 1 ]; then
-               comment "convert args: $convert_args"
-       fi
-
-       if [ "$status" = 0 ]; then
-               fail "SUCCEEDS (should fail): $what"
-               diag "ARGS: $convert_args"
-       else
-               pass "FAILS: $what"
-       fi
+       # Nothing should be printed on stdout.
+       bt_diff /dev/null "${tmp_stdout}"
+       ok $? "$what: nothing is printed on stdout"
 
        if [ -n "${expected_error_str}" ]; then
                grep --quiet "$expected_error_str" "$tmp_stderr"
@@ -85,10 +77,6 @@ test_bt_convert_fails() {
        fi
 }
 
-comment() {
-       echo "### $1 ###"
-}
-
 path_to_trace="${BT_CTF_TRACES_PATH}/succeed/succeed1"
 path_to_trace2="${BT_CTF_TRACES_PATH}/succeed/succeed2"
 output_path="/output/path"
@@ -102,7 +90,7 @@ if [ "$BT_OS_TYPE" = "mingw" ]; then
        output_path=$(cygpath -m "$output_path")
 fi
 
-plan_tests 65
+plan_tests 130
 
 test_bt_convert_run_args 'path non-option arg' "$path_to_trace" "--component auto-disc-source-ctf-fs:source.ctf.fs --params 'inputs=[\"$path_to_trace\"]' --component pretty:sink.text.pretty --component muxer:filter.utils.muxer --connect auto-disc-source-ctf-fs:muxer --connect muxer:pretty"
 test_bt_convert_run_args 'path non-option args' "$path_to_trace $path_to_trace2" "--component auto-disc-source-ctf-fs:source.ctf.fs --params 'inputs=[\"$path_to_trace\", \"${path_to_trace2}\"]' --component pretty:sink.text.pretty --component muxer:filter.utils.muxer --connect auto-disc-source-ctf-fs:muxer --connect muxer:pretty"
@@ -171,4 +159,5 @@ test_bt_convert_fails '--stream-intersection' "$path_to_trace --stream-intersect
 test_bt_convert_fails 'two sinks with -o dummy + --clock-seconds' "$path_to_trace -o dummy --clock-seconds"
 test_bt_convert_fails 'path non-option arg + user sink + -o text' "$path_to_trace --component=sink.abc.def -o text"
 
+rm -f "${tmp_stdout}"
 rm -f "${tmp_stderr}"