From 6d0d8bb3291883e07adf7e7b5f65b2ca1018bcd7 Mon Sep 17 00:00:00 2001 From: =?utf8?q?=EC=9D=B4=ED=95=9C=EC=A2=85/On-Device=20Lab=28SR=29/Enginee?= =?utf8?q?r/=EC=82=BC=EC=84=B1=EC=A0=84=EC=9E=90?= Date: Fri, 23 Aug 2019 08:16:56 +0900 Subject: [PATCH] [neurun] Check cmd args in run_frameworktest.sh (#6813) * [neurun] Check cmd args in run_frameworktest.sh Check command line arguments in `run_frameworktest.sh`. If a required argument is not given, it will show `Usage` then stop. Signed-off-by: Hanjoung Lee * FRAMEWORKTEST_LIST_FILE is not a required argument Signed-off-by: Hanjoung Lee --- tests/scripts/run_frameworktest.sh | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/tests/scripts/run_frameworktest.sh b/tests/scripts/run_frameworktest.sh index 491e9ef..bf36d0c 100755 --- a/tests/scripts/run_frameworktest.sh +++ b/tests/scripts/run_frameworktest.sh @@ -32,6 +32,8 @@ function Usage() echo " --tapname=framework_test.tap \\ # Tap file name" echo " --logname=framework_test.log \\ # Log file name" echo " --testname=Frameworktest # Name of the test just a label of tests" + + exit 1 } for i in "$@" @@ -39,7 +41,6 @@ do case $i in -h|--help|help) Usage - exit 1 ;; --runtestsh=*) FWTEST_RUN_TEST_SH=${i#*=} @@ -66,7 +67,12 @@ do shift done -# TODO: handle exceptions for params +[ ! -z "$FWTEST_RUN_TEST_SH" ] || Usage +[ ! -z "$FWTEST_DRIVER_BIN" ] || Usage +[ ! -z "$FWTEST_REPORT_DIR" ] || Usage +[ ! -z "$FWTEST_TAP_NAME" ] || Usage +[ ! -z "$FWTEST_LOG_NAME" ] || Usage +[ ! -z "$FWTEST_TEST_NAME" ] || Usage if [ ! -e "$FWTEST_REPORT_DIR" ]; then mkdir -p $FWTEST_REPORT_DIR -- 2.7.4