[PATCH 09/12] tests: Simplify *tools commands using pane_status

[Stefano Brivio <sbrivio@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 2230 bytes --]

From: David Gibson <david(a)gibson.dropbear.id.au>

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 <david(a)gibson.dropbear.id.au>
---
 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%% *}"
-- 
@@ -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