Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1147)

Unified Diff: src/scripts/run_tests.sh

Issue 614005: Adds changes to unittest scripts to use ebuild if board is specified (Closed)
Patch Set: Fix quotes Created 10 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/scripts/build_tests.sh ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/scripts/run_tests.sh
diff --git a/src/scripts/run_tests.sh b/src/scripts/run_tests.sh
index 7965ba162ed92a8fec8c97d4d0ce0f13d6d6d708..734f7c8f534f498a0f95c889fa9752a637c03e50 100755
--- a/src/scripts/run_tests.sh
+++ b/src/scripts/run_tests.sh
@@ -11,6 +11,8 @@
# Flags
DEFINE_string build_root "$DEFAULT_BUILD_ROOT" \
"Root of build output"
+DEFINE_string board "" \
+ "Target board of which tests were built"
# Parse command line
FLAGS "$@" || exit 1
@@ -20,11 +22,21 @@ eval set -- "${FLAGS_ARGV}"
set -ex
# Run tests
-TESTS_DIR="$FLAGS_build_root/x86/tests"
-cd "$TESTS_DIR"
-
-# TODO: standardize test names - should all end in "_test"
-for i in *_test *_tests *_unittests; do ./${i}; done
-
-cd -
-echo "All tests passed."
+if [ -n "$FLAGS_board" ]
+then
+ TESTS_DIR="/build/${FLAGS_board}/tests"
+ echo "Not implemented" >&2
+ exit 1
+
+ # TODO(sosa@chromium.org) - Call autotest job to run tests from TESTS_DIR
+ # using run_remote_tests
+else
+ TESTS_DIR="$FLAGS_build_root/x86/tests"
+ cd "$TESTS_DIR"
+
+ # TODO: standardize test names - should all end in "_test"
+ for i in *_test *_tests *_unittests; do ! ./${i}; done
+
+ cd -
+ echo "All tests passed."
+fi
« no previous file with comments | « src/scripts/build_tests.sh ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698