All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel Sangorrin <daniel.sangorrin@toshiba.co.jp>
To: fuego@lists.linuxfoundation.org
Subject: [Fuego] [PATCH 16/19] LTP: fix the buildonly and runonly cases
Date: Fri, 12 May 2017 17:19:57 +0900	[thread overview]
Message-ID: <1494577200-11597-17-git-send-email-daniel.sangorrin@toshiba.co.jp> (raw)
In-Reply-To: <1494577200-11597-1-git-send-email-daniel.sangorrin@toshiba.co.jp>

Previous changes had broken the buildonly/runonly support
so fix it.

Signed-off-by: Daniel Sangorrin <daniel.sangorrin@toshiba.co.jp>
---
 engine/tests/Functional.LTP/fuego_test.sh | 14 +++++++-------
 engine/tests/Functional.LTP/parser.py     |  4 ++--
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/engine/tests/Functional.LTP/fuego_test.sh b/engine/tests/Functional.LTP/fuego_test.sh
index 3d3d360..8ea78c7 100755
--- a/engine/tests/Functional.LTP/fuego_test.sh
+++ b/engine/tests/Functional.LTP/fuego_test.sh
@@ -161,10 +161,13 @@ function test_run {
 }
 
 function test_fetch_results {
-    echo "Fetching LTP results"
-    echo "PWD: $PWD"
-    echo "Triplet: $WORKSPACE/$TRIPLET"
-    get $FUNCTIONAL_LTP_RUNFOLDER/result .
+    if [ "$FUNCTIONAL_LTP_BUILDONLY" = "false" ]; then
+        echo "Fetching LTP results"
+        rm -rf result/
+        get $FUNCTIONAL_LTP_RUNFOLDER/result .
+    else
+        echo "Skip fetching LTP results"
+    fi
 }
 
 function test_processing {
@@ -178,11 +181,8 @@ function test_processing {
         [ -e results.xlsx ] && cp results.xlsx ${LOGDIR}/results.xlsx
         [ -e rt.log ] && cp rt.log $FUEGO_RW/logs/$JOB_NAME/
         cd ..
-        rm -rf result/
     else
         echo "Processing LTP build log"
         log_compare "$TESTDIR" "1836" "compile PASSED$" "p"
     fi
 }
-
-
diff --git a/engine/tests/Functional.LTP/parser.py b/engine/tests/Functional.LTP/parser.py
index 6f0638c..feab36a 100755
--- a/engine/tests/Functional.LTP/parser.py
+++ b/engine/tests/Functional.LTP/parser.py
@@ -77,8 +77,8 @@ def read_output (test_category, test_case):
 try:
     os.chdir('./result')
 except:
-    print "ERROR: no result directory"
-    sys.exit(1)
+    print "WARNING: no result directory (probably a build only test)."
+    sys.exit(3)
 
 tests = os.listdir('.')
 tests.sort()
-- 
2.7.4



  parent reply	other threads:[~2017-05-12  8:19 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-12  8:19 [Fuego] New batch of patches Daniel Sangorrin
2017-05-12  8:19 ` [Fuego] [PATCH 01/19] fail_regexp: rename FAIL_PATTERN to FAIL_REGEXP for coherency Daniel Sangorrin
2017-05-12 22:56   ` Bird, Timothy
2017-05-12  8:19 ` [Fuego] [PATCH 02/19] functions:fail_check_cases: add message when no fail cases available Daniel Sangorrin
2017-05-12  8:19 ` [Fuego] [PATCH 03/19] ft2demos: fix test and add to docker Daniel Sangorrin
2017-05-12  8:19 ` [Fuego] [PATCH 04/19] unpack: remove nostrip option Daniel Sangorrin
2017-05-12 22:57   ` Bird, Timothy
2017-05-12  8:19 ` [Fuego] [PATCH 05/19] unpack: move code related to spec-defined tarballs to unpack Daniel Sangorrin
2017-05-12 23:05   ` Bird, Timothy
2017-05-12  8:19 ` [Fuego] [PATCH 06/19] bc: test that specs can override the tarball variable Daniel Sangorrin
2017-05-12  8:19 ` [Fuego] [PATCH 07/19] unpack: handle possible errors Daniel Sangorrin
2017-05-12 23:09   ` Bird, Timothy
2017-05-12  8:19 ` [Fuego] [PATCH 08/19] main: remove ReBuild check Daniel Sangorrin
2017-05-12  8:19 ` [Fuego] [PATCH 09/19] rebuild: make it simpler for test developers Daniel Sangorrin
2017-05-12 23:02   ` Bird, Timothy
2017-05-16  2:35     ` Daniel Sangorrin
2017-05-16  4:35       ` Daniel Sangorrin
2017-05-16  4:52         ` Bird, Timothy
2017-05-16  5:14           ` Daniel Sangorrin
2017-05-12  8:19 ` [Fuego] [PATCH 10/19] call_if_present: tests do not need to implement dummy functions Daniel Sangorrin
2017-05-12  8:19 ` [Fuego] [PATCH 11/19] expat: remove unneeded semicolons Daniel Sangorrin
2017-05-12  8:19 ` [Fuego] [PATCH 12/19] tarball: remove unneeded tarball definitions Daniel Sangorrin
2017-05-12  8:19 ` [Fuego] [PATCH 13/19] style: fix trailing spaces and indentation Daniel Sangorrin
2017-05-12  8:19 ` [Fuego] [PATCH 14/19] parser: allow parser.py to skip processing Daniel Sangorrin
2017-05-12  8:19 ` [Fuego] [PATCH 15/19] LTP: copy target_bin before modifying it Daniel Sangorrin
2017-05-12  8:19 ` Daniel Sangorrin [this message]
2017-05-12  8:19 ` [Fuego] [PATCH 17/19] LTP: add support for skipping certain test cases Daniel Sangorrin
2017-05-12 22:57   ` Bird, Timothy
2017-05-12  8:19 ` [Fuego] [PATCH 18/19] LTP: add a spec for docker Daniel Sangorrin
2017-05-12  8:20 ` [Fuego] [PATCH 19/19] LTP: add a fixthis for the -t flag Daniel Sangorrin
2017-05-12 22:52 ` [Fuego] New batch of patches Bird, Timothy

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=1494577200-11597-17-git-send-email-daniel.sangorrin@toshiba.co.jp \
    --to=daniel.sangorrin@toshiba.co.jp \
    --cc=fuego@lists.linuxfoundation.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.