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

Unified Diff: src/scripts/run_remote_tests.sh

Issue 1794004: Added --test_args parameter that passes its value as -a flag to autoserv. (Closed) Base URL: ssh://git@chromiumos-git/chromeos
Patch Set: Parse --args with optparser in autotest Created 10 years, 8 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/autotest ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/scripts/run_remote_tests.sh
diff --git a/src/scripts/run_remote_tests.sh b/src/scripts/run_remote_tests.sh
index 35013d63d063de36892598c02cfa434eb10b1fa0..48b497d9fc80b78518f6f50f3ed0c51e540e3a21 100755
--- a/src/scripts/run_remote_tests.sh
+++ b/src/scripts/run_remote_tests.sh
@@ -14,6 +14,7 @@
get_default_board
+DEFINE_string args "" "Command line arguments for test, separated with comma" a
DEFINE_string board "$DEFAULT_BOARD" \
"The board for which you are building autotest"
DEFINE_string chroot "${DEFAULT_CHROOT_DIR}" "alternate chroot location" c
@@ -242,6 +243,12 @@ function main() {
if [[ ${FLAGS_verbose} -eq $FLAGS_TRUE ]]; then
verbose="--verbose"
fi
+ local args=()
+ if [[ -n "${FLAGS_args}" ]]; then
+ local with_spaces=""
+ with_spaces=${FLAGS_args//,/ }
+ args=("-a" "${with_spaces}")
+ fi
RAN_ANY_TESTS=${FLAGS_TRUE}
@@ -253,7 +260,8 @@ function main() {
fi
${enter_chroot} ${autotest} --board "${FLAGS_board}" -m "${FLAGS_remote}" \
- "${option}" "${control_file}" -r "${results_dir}" ${verbose}
+ "${option}" "${control_file}" -r "${results_dir}" ${verbose} \
+ "${args[@]}"
results_dir="${TMP}/${results_dir_name}"
local test_status="${results_dir}/status.log"
« no previous file with comments | « src/scripts/autotest ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698