diff mbox

[kvm-unit-tests] run_tests: add run all tests parameter

Message ID 1488992435-15268-1-git-send-email-drjones@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Andrew Jones March 8, 2017, 5 p.m. UTC
Tests can be flagged as 'nodefault' in unittests.cfg files. Allow
running all tests, including all nodefault tests by just calling
run_tests.sh with a new -a parameter. (Also cleanup the help text.)

Suggested-by: Christoffer Dall <christoffer.dall@linaro.org>
Signed-off-by: Andrew Jones <drjones@redhat.com>
---
 run_tests.sh         | 14 +++++++++-----
 scripts/runtime.bash |  1 +
 2 files changed, 10 insertions(+), 5 deletions(-)

Comments

Paolo Bonzini March 10, 2017, 9:20 a.m. UTC | #1
On 08/03/2017 18:00, Andrew Jones wrote:
> Tests can be flagged as 'nodefault' in unittests.cfg files. Allow
> running all tests, including all nodefault tests by just calling
> run_tests.sh with a new -a parameter. (Also cleanup the help text.)
> 
> Suggested-by: Christoffer Dall <christoffer.dall@linaro.org>
> Signed-off-by: Andrew Jones <drjones@redhat.com>
> ---
>  run_tests.sh         | 14 +++++++++-----
>  scripts/runtime.bash |  1 +
>  2 files changed, 10 insertions(+), 5 deletions(-)
> 
> diff --git a/run_tests.sh b/run_tests.sh
> index 09cd057b7f41d..c35a058023467 100755
> --- a/run_tests.sh
> +++ b/run_tests.sh
> @@ -1,6 +1,7 @@
>  #!/bin/bash
>  
>  verbose="no"
> +run_all_tests="no" # don't run nodefault tests
>  
>  if [ ! -f config.mak ]; then
>      echo "run ./configure && make first. See ./configure -h"
> @@ -13,13 +14,13 @@ function usage()
>  {
>  cat <<EOF
>  
> -Usage: $0 [-g group] [-h] [-v]
> -Usage: $0 [-g group] [-h] [-v] [-j NUM-TASKS]
> +Usage: $0 [-h] [-v] [-a] [-g group] [-j NUM-TASKS]
>  
> -    -g: Only execute tests in the given group
>      -h: Output this help text
> -    -j: Execute tests in parallel
>      -v: Enables verbose mode
> +    -a: Run all tests, including those flagged as 'nodefault'
> +    -g: Only execute tests in the given group
> +    -j: Execute tests in parallel
>  
>  Set the environment variable QEMU=/path/to/qemu-system-ARCH to
>  specify the appropriate qemu binary for ARCH-run.
> @@ -30,8 +31,11 @@ EOF
>  RUNTIME_arch_run="./$TEST_DIR/run"
>  source scripts/runtime.bash
>  
> -while getopts "g:hj:v" opt; do
> +while getopts "ag:hj:v" opt; do
>      case $opt in
> +        a)
> +            run_all_tests="yes"
> +            ;;
>          g)
>              only_group=$OPTARG
>              ;;
> diff --git a/scripts/runtime.bash b/scripts/runtime.bash
> index 9c1bc3b3b3fe7..98f18357e537a 100644
> --- a/scripts/runtime.bash
> +++ b/scripts/runtime.bash
> @@ -34,6 +34,7 @@ get_cmdline()
>  
>  skip_nodefault()
>  {
> +    [ "$run_all_tests" = "yes" ] && return 1
>      [ "$STANDALONE" != "yes" ] && return 0
>  
>      while true; do
> 

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Radim Krčmář March 15, 2017, 5:11 p.m. UTC | #2
2017-03-08 18:00+0100, Andrew Jones:
> Tests can be flagged as 'nodefault' in unittests.cfg files. Allow
> running all tests, including all nodefault tests by just calling
> run_tests.sh with a new -a parameter. (Also cleanup the help text.)
> 
> Suggested-by: Christoffer Dall <christoffer.dall@linaro.org>
> Signed-off-by: Andrew Jones <drjones@redhat.com>
> ---

Applied, thanks.

>  run_tests.sh         | 14 +++++++++-----
>  scripts/runtime.bash |  1 +
>  2 files changed, 10 insertions(+), 5 deletions(-)
> 
> diff --git a/run_tests.sh b/run_tests.sh
> index 09cd057b7f41d..c35a058023467 100755
> --- a/run_tests.sh
> +++ b/run_tests.sh
> @@ -1,6 +1,7 @@
>  #!/bin/bash
>  
>  verbose="no"
> +run_all_tests="no" # don't run nodefault tests
>  
>  if [ ! -f config.mak ]; then
>      echo "run ./configure && make first. See ./configure -h"
> @@ -13,13 +14,13 @@ function usage()
>  {
>  cat <<EOF
>  
> -Usage: $0 [-g group] [-h] [-v]
> -Usage: $0 [-g group] [-h] [-v] [-j NUM-TASKS]
> +Usage: $0 [-h] [-v] [-a] [-g group] [-j NUM-TASKS]
>  
> -    -g: Only execute tests in the given group
>      -h: Output this help text
> -    -j: Execute tests in parallel
>      -v: Enables verbose mode
> +    -a: Run all tests, including those flagged as 'nodefault'
> +    -g: Only execute tests in the given group
> +    -j: Execute tests in parallel
>  
>  Set the environment variable QEMU=/path/to/qemu-system-ARCH to
>  specify the appropriate qemu binary for ARCH-run.
> @@ -30,8 +31,11 @@ EOF
>  RUNTIME_arch_run="./$TEST_DIR/run"
>  source scripts/runtime.bash
>  
> -while getopts "g:hj:v" opt; do
> +while getopts "ag:hj:v" opt; do
>      case $opt in
> +        a)
> +            run_all_tests="yes"
> +            ;;
>          g)
>              only_group=$OPTARG
>              ;;
> diff --git a/scripts/runtime.bash b/scripts/runtime.bash
> index 9c1bc3b3b3fe7..98f18357e537a 100644
> --- a/scripts/runtime.bash
> +++ b/scripts/runtime.bash
> @@ -34,6 +34,7 @@ get_cmdline()
>  
>  skip_nodefault()
>  {
> +    [ "$run_all_tests" = "yes" ] && return 1
>      [ "$STANDALONE" != "yes" ] && return 0
>  
>      while true; do
> -- 
> 2.7.4
>
diff mbox

Patch

diff --git a/run_tests.sh b/run_tests.sh
index 09cd057b7f41d..c35a058023467 100755
--- a/run_tests.sh
+++ b/run_tests.sh
@@ -1,6 +1,7 @@ 
 #!/bin/bash
 
 verbose="no"
+run_all_tests="no" # don't run nodefault tests
 
 if [ ! -f config.mak ]; then
     echo "run ./configure && make first. See ./configure -h"
@@ -13,13 +14,13 @@  function usage()
 {
 cat <<EOF
 
-Usage: $0 [-g group] [-h] [-v]
-Usage: $0 [-g group] [-h] [-v] [-j NUM-TASKS]
+Usage: $0 [-h] [-v] [-a] [-g group] [-j NUM-TASKS]
 
-    -g: Only execute tests in the given group
     -h: Output this help text
-    -j: Execute tests in parallel
     -v: Enables verbose mode
+    -a: Run all tests, including those flagged as 'nodefault'
+    -g: Only execute tests in the given group
+    -j: Execute tests in parallel
 
 Set the environment variable QEMU=/path/to/qemu-system-ARCH to
 specify the appropriate qemu binary for ARCH-run.
@@ -30,8 +31,11 @@  EOF
 RUNTIME_arch_run="./$TEST_DIR/run"
 source scripts/runtime.bash
 
-while getopts "g:hj:v" opt; do
+while getopts "ag:hj:v" opt; do
     case $opt in
+        a)
+            run_all_tests="yes"
+            ;;
         g)
             only_group=$OPTARG
             ;;
diff --git a/scripts/runtime.bash b/scripts/runtime.bash
index 9c1bc3b3b3fe7..98f18357e537a 100644
--- a/scripts/runtime.bash
+++ b/scripts/runtime.bash
@@ -34,6 +34,7 @@  get_cmdline()
 
 skip_nodefault()
 {
+    [ "$run_all_tests" = "yes" ] && return 1
     [ "$STANDALONE" != "yes" ] && return 0
 
     while true; do