From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Brivio To: passt-dev@passt.top Subject: [PATCH 09/12] tests: Simplify *tools commands using pane_status Date: Tue, 17 May 2022 14:36:11 +0200 Message-ID: <20220517123614.1892316-10-sbrivio@redhat.com> In-Reply-To: <20220517123614.1892316-1-sbrivio@redhat.com> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============8032644932019351220==" --===============8032644932019351220== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit From: David Gibson Now that we have pane_status to check the success of commands issued to panes, we can more easily check for the success of the 'which' commands used to check tool availability, rather than constructing, then parsing special "skip" output. Signed-off-by: David Gibson --- test/lib/test | 25 ++++++++++--------------- 1 file changed, 10 insertions(+), 15 deletions(-) diff --git a/test/lib/test b/test/lib/test index 4262343..a5583e3 100755 --- a/test/lib/test +++ b/test/lib/test @@ -151,9 +151,8 @@ test_one_line() { tmux send-keys -t ${PANE_HOST} "C-c" ;; "htools") - pane_run HOST 'which '"${__arg}"' >/dev/null || echo skip' - pane_wait HOST - [ "$(pane_parse HOST)" = "skip" ] && TEST_ONE_skip=1 || true + pane_run HOST 'which '"${__arg}"' >/dev/null' + pane_status HOST || TEST_ONE_skip=1 ;; "passt") pane_run PASST "${__arg}" @@ -192,19 +191,16 @@ test_one_line() { pane_status GUEST_1 || TEST_ONE_nok=1 ;; "gtools") - pane_run GUEST 'which '"${__arg}"' >/dev/null || echo skip' - pane_wait GUEST - [ "$(pane_parse GUEST)" = "skip" ] && TEST_ONE_skip=1 || true + pane_run GUEST 'which '"${__arg}"' >/dev/null' + pane_status GUEST || TEST_ONE_skip=1 ;; "g1tools") - pane_run GUEST_1 'which '"${__arg}"' >/dev/null || echo skip' - pane_wait GUEST_1 - [ "$(pane_parse GUEST_1)" = "skip" ] && TEST_ONE_skip=1 || true + pane_run GUEST_1 'which '"${__arg}"' >/dev/null' + pane_status GUEST_1 || TEST_ONE_skip=1 ;; "g2tools") - pane_run GUEST_2 'which '"${__arg}"' >/dev/null || echo skip' - pane_wait GUEST_2 - [ "$(pane_parse GUEST_2)" = "skip" ] && TEST_ONE_skip=1 || true + pane_run GUEST_2 'which '"${__arg}"' >/dev/null' + pane_status GUEST_2 || TEST_ONE_skip=1 ;; "guest2") pane_run GUEST_2 "${__arg}" @@ -247,9 +243,8 @@ test_one_line() { pane_status NS2 || TEST_ONE_nok=1 ;; "nstools") - pane_run NS 'which '"${__arg}"' >/dev/null || echo skip' - pane_wait NS - [ "$(pane_parse NS)" = "skip" ] && TEST_ONE_skip=1 || true + pane_run NS 'which '"${__arg}"' >/dev/null' + pane_status NS || TEST_ONE_skip=1 ;; "gout") __varname="${__arg%% *}" -- 2.35.1 --===============8032644932019351220==--