[PATCH 0/3] tests: A few fixes for demos

[PATCH 0/3] tests: A few fixes for demos

[Stefano Brivio]

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

This series fixes issues that came with the previous test clean-up
series, or that were made visible by it.

Stefano Brivio (3):
  demo/pasta: Fix bad sleep directive
  tests: Don't count number of test units for demos
  tests: Don't check exit code for every command in demo mode

 test/demo/pasta | 2 +-
 test/lib/term   | 3 +++
 test/lib/test   | 6 ++++--
 3 files changed, 8 insertions(+), 3 deletions(-)

-- 
2.35.1

[PATCH 1/3] demo/pasta: Fix bad sleep directive

[Stefano Brivio]

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

'sleep' always needs an argument, this was meant to introduce
a 2 seconds delay.

Signed-off-by: Stefano Brivio <sbrivio(a)redhat.com>
---
 test/demo/pasta | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/demo/pasta b/test/demo/pasta
index 4bfea80..506d948 100644
--- a/test/demo/pasta
+++ b/test/demo/pasta
@@ -110,7 +110,7 @@ nl
 say	Now the other way around...
 nl
 say	  we can use a loopback address
-sleep
+sleep	2
 hostb	nc -l -p 31337
 sleep	2
 ns	echo "Hello from the namespace" | nc -N 127.0.0.1 31337
-- 
@@ -110,7 +110,7 @@ nl
 say	Now the other way around...
 nl
 say	  we can use a loopback address
-sleep
+sleep	2
 hostb	nc -l -p 31337
 sleep	2
 ns	echo "Hello from the namespace" | nc -N 127.0.0.1 31337
-- 
2.35.1

[PATCH 2/3] tests: Don't count number of test units for demos

[Stefano Brivio]

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

...there are no 'test' directives in demo, and this causes a
script failure.

Signed-off-by: Stefano Brivio <sbrivio(a)redhat.com>
---
 test/lib/test | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/test/lib/test b/test/lib/test
index a5583e3..09e8340 100755
--- a/test/lib/test
+++ b/test/lib/test
@@ -386,8 +386,10 @@ test_one() {
 		return
 	fi
 
-	__ntests="$(grep -c "^test$(printf '\t')" "${__test_file}")"
-	[ ${DEMO} -eq 0 ] && status_file_start "${1}" "${__ntests}"
+	if [ ${DEMO} -eq 0 ]; then
+		__ntests="$(grep -c "^test$(printf '\t')" "${__test_file}")"
+		status_file_start "${1}" "${__ntests}"
+	fi
 
 	[ ${CI} -eq 1 ] && video_link "${1}"
 
-- 
@@ -386,8 +386,10 @@ test_one() {
 		return
 	fi
 
-	__ntests="$(grep -c "^test$(printf '\t')" "${__test_file}")"
-	[ ${DEMO} -eq 0 ] && status_file_start "${1}" "${__ntests}"
+	if [ ${DEMO} -eq 0 ]; then
+		__ntests="$(grep -c "^test$(printf '\t')" "${__test_file}")"
+		status_file_start "${1}" "${__ntests}"
+	fi
 
 	[ ${CI} -eq 1 ] && video_link "${1}"
 
-- 
2.35.1

[PATCH 3/3] tests: Don't check exit code for every command in demo mode

[Stefano Brivio]

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

Having all those 'echo $?' is rather distracting in demos.

Signed-off-by: Sefano Brivio <sbrivio(a)redhat.com>
---
 test/lib/term | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/test/lib/term b/test/lib/term
index 004c2a2..0f581de 100755
--- a/test/lib/term
+++ b/test/lib/term
@@ -232,6 +232,9 @@ pane_parse() {
 # $1:	Pane name
 pane_status() {
 	pane_wait "${1}"
+
+	[ ${DEMO} -eq 1 ] && return 0
+
 	__status="$(pane_parse "${1}")"
 	while ! [ "${__status}" -eq "${__status}" ]; do
 		sleep 1
-- 
@@ -232,6 +232,9 @@ pane_parse() {
 # $1:	Pane name
 pane_status() {
 	pane_wait "${1}"
+
+	[ ${DEMO} -eq 1 ] && return 0
+
 	__status="$(pane_parse "${1}")"
 	while ! [ "${__status}" -eq "${__status}" ]; do
 		sleep 1
-- 
2.35.1

Re: [PATCH 1/3] demo/pasta: Fix bad sleep directive

[David Gibson]

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

On Thu, May 19, 2022 at 01:28:27AM +0200, Stefano Brivio wrote:
> 'sleep' always needs an argument, this was meant to introduce
> a 2 seconds delay.
> 
> Signed-off-by: Stefano Brivio <sbrivio(a)redhat.com>

Reviewed-by: David Gibson <david(a)gibson.dropbear.id.au>

> ---
>  test/demo/pasta | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/test/demo/pasta b/test/demo/pasta
> index 4bfea80..506d948 100644
> --- a/test/demo/pasta
> +++ b/test/demo/pasta
> @@ -110,7 +110,7 @@ nl
>  say	Now the other way around...
>  nl
>  say	  we can use a loopback address
> -sleep
> +sleep	2
>  hostb	nc -l -p 31337
>  sleep	2
>  ns	echo "Hello from the namespace" | nc -N 127.0.0.1 31337

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH 2/3] tests: Don't count number of test units for demos

[David Gibson]

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

On Thu, May 19, 2022 at 01:28:28AM +0200, Stefano Brivio wrote:
> ...there are no 'test' directives in demo, and this causes a
> script failure.
> 
> Signed-off-by: Stefano Brivio <sbrivio(a)redhat.com>

Reviewed-by: David Gibson <david(a)gibson.dropbear.id.au>

> ---
>  test/lib/test | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/test/lib/test b/test/lib/test
> index a5583e3..09e8340 100755
> --- a/test/lib/test
> +++ b/test/lib/test
> @@ -386,8 +386,10 @@ test_one() {
>  		return
>  	fi
>  
> -	__ntests="$(grep -c "^test$(printf '\t')" "${__test_file}")"
> -	[ ${DEMO} -eq 0 ] && status_file_start "${1}" "${__ntests}"
> +	if [ ${DEMO} -eq 0 ]; then
> +		__ntests="$(grep -c "^test$(printf '\t')" "${__test_file}")"
> +		status_file_start "${1}" "${__ntests}"
> +	fi
>  
>  	[ ${CI} -eq 1 ] && video_link "${1}"
>  

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH 3/3] tests: Don't check exit code for every command in demo mode

[David Gibson]

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

On Thu, May 19, 2022 at 01:28:29AM +0200, Stefano Brivio wrote:
> Having all those 'echo $?' is rather distracting in demos.
> 
> Signed-off-by: Sefano Brivio <sbrivio(a)redhat.com>

Reviewed-by: David Gibson <david(a)gibson.dropbear.id.au>

> ---
>  test/lib/term | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/test/lib/term b/test/lib/term
> index 004c2a2..0f581de 100755
> --- a/test/lib/term
> +++ b/test/lib/term
> @@ -232,6 +232,9 @@ pane_parse() {
>  # $1:	Pane name
>  pane_status() {
>  	pane_wait "${1}"
> +
> +	[ ${DEMO} -eq 1 ] && return 0
> +
>  	__status="$(pane_parse "${1}")"
>  	while ! [ "${__status}" -eq "${__status}" ]; do
>  		sleep 1

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]