public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Andrew Jones <drjones@redhat.com>
To: kvm@vger.kernel.org
Cc: christoffer.dall@linaro.org, rkrcmar@redhat.com, pbonzini@redhat.com
Subject: [kvm-unit-tests PATCH] run_tests: add run all tests parameter
Date: Wed,  8 Mar 2017 18:00:35 +0100	[thread overview]
Message-ID: <1488992435-15268-1-git-send-email-drjones@redhat.com> (raw)

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
-- 
2.7.4

             reply	other threads:[~2017-03-08 17:00 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-08 17:00 Andrew Jones [this message]
2017-03-10  9:20 ` [kvm-unit-tests PATCH] run_tests: add run all tests parameter Paolo Bonzini
2017-03-15 17:11 ` Radim Krčmář

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1488992435-15268-1-git-send-email-drjones@redhat.com \
    --to=drjones@redhat.com \
    --cc=christoffer.dall@linaro.org \
    --cc=kvm@vger.kernel.org \
    --cc=pbonzini@redhat.com \
    --cc=rkrcmar@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox