On Mon, Oct 19, 2020 at 11:38:45AM +0800, Li Ping wrote:

Signed-off-by: Li Ping <15396232681@163.com> --- tests/test-tools | 37 +++++++++++++++++++++++++++++++++++++ 1 file changed, 37 insertions(+) create mode 100755 tests/test-tools

diff --git a/tests/test-tools b/tests/test-tools new file mode 100755 index 00000000..cee882aa --- /dev/null +++ b/tests/test-tools @@ -0,0 +1,37 @@ +#!/bin/bash +# - testsuite +# - case + +cd $BENCHMARK_ROOT/test-tools/mugen || exit +: "${testsuite:=smoke-testing}" +: "${case:=oe_test_dd_001}" + +set_env() +{ + bash runoet.sh -c + source ~/.bash_profile +} + +test_one_case() +{ + + bash runoet.sh -f $testsuite -r $case -x +} + +test_all_cases() +{ + bash runoet.sh -a +} + +run() +{ + set_env + if [ "${testsuite}" == "all" ] + then

The style if [ "${testsuite}" == "all" ]; then will be better. Thanks, Xueliang

+ test_all_cases + else + test_one_case + fi +} + +run -- 2.23.0

On Mon, Oct 19, 2020 at 11:38:45AM +0800, Li Ping wrote:

Signed-off-by: Li Ping <15396232681@163.com> --- tests/test-tools | 37 +++++++++++++++++++++++++++++++++++++ 1 file changed, 37 insertions(+) create mode 100755 tests/test-tools

diff --git a/tests/test-tools b/tests/test-tools new file mode 100755 index 00000000..cee882aa --- /dev/null +++ b/tests/test-tools @@ -0,0 +1,37 @@ +#!/bin/bash +# - testsuite +# - case + +cd $BENCHMARK_ROOT/test-tools/mugen || exit +: "${testsuite:=smoke-testing}" +: "${case:=oe_test_dd_001}" + +set_env() +{ + bash runoet.sh -c + source ~/.bash_profile +} + +test_one_case() +{ + + bash runoet.sh -f $testsuite -r $case -x

One more blank line

+} + +test_all_cases() +{ + bash runoet.sh -a +} + +run() +{ + set_env

+ if [ "${testsuite}" == "all" ]

You can add a blank line here Thanks sunyukui

+ then + test_all_cases + else + test_one_case + fi +} + +run -- 2.23.0