[Fuego] [PATCH v2] Add test case of service openhpid.

Tim.Bird at sony.com Tim.Bird at sony.com
Wed Mar 13 22:14:33 UTC 2019


I have the same comments for this test as for the ipmi test.
But still - Applied.  Thanks,
 -- Tim


> -----Original Message-----
> From: fuego-bounces at lists.linuxfoundation.org [mailto:fuego-
> bounces at lists.linuxfoundation.org] On Behalf Of Wang Mingyu
> Sent: Wednesday, March 06, 2019 2:08 AM
> To: fuego at lists.linuxfoundation.org
> Subject: [Fuego] [PATCH v2] Add test case of service openhpid.
> 
> The openhpid is HPI instance to which multiple clients can connect.
> This test set is used to check if the ps of service can be generated
> successfully.
> 
> Signed-off-by: Wang Mingyu <wangmy at cn.fujitsu.com>
> ---
>  tests/Functional.openhpid/fuego_test.sh        | 18 ++++++++++++++++++
>  tests/Functional.openhpid/openhpid_test.sh     |  4 ++++
>  tests/Functional.openhpid/parser.py            | 22 ++++++++++++++++++++++
>  tests/Functional.openhpid/spec.json            |  7 +++++++
>  tests/Functional.openhpid/tests/openhpid_ps.sh | 26
> ++++++++++++++++++++++++++
>  5 files changed, 77 insertions(+)
>  create mode 100644 tests/Functional.openhpid/fuego_test.sh
>  create mode 100755 tests/Functional.openhpid/openhpid_test.sh
>  create mode 100644 tests/Functional.openhpid/parser.py
>  create mode 100644 tests/Functional.openhpid/spec.json
>  create mode 100644 tests/Functional.openhpid/tests/openhpid_ps.sh
> 
> diff --git a/tests/Functional.openhpid/fuego_test.sh
> b/tests/Functional.openhpid/fuego_test.sh
> new file mode 100644
> index 0000000..f90897a
> --- /dev/null
> +++ b/tests/Functional.openhpid/fuego_test.sh
> @@ -0,0 +1,18 @@
> +function test_pre_check {
> +    assert_has_program openhpid
> +}
> +
> +function test_deploy {
> +    put $TEST_HOME/openhpid_test.sh $BOARD_TESTDIR/fuego.$TESTDIR/
> +    put $FUEGO_CORE/scripts/fuego_board_function_lib.sh
> $BOARD_TESTDIR/fuego.$TESTDIR
> +    put -r $TEST_HOME/tests $BOARD_TESTDIR/fuego.$TESTDIR/
> +}
> +
> +function test_run {
> +    report "cd $BOARD_TESTDIR/fuego.$TESTDIR;\
> +    ./openhpid_test.sh"
> +}
> +
> +function test_processing {
> +    log_compare "$TESTDIR" "0" "TEST-FAIL" "n"
> +}
> diff --git a/tests/Functional.openhpid/openhpid_test.sh
> b/tests/Functional.openhpid/openhpid_test.sh
> new file mode 100755
> index 0000000..dd5ce37
> --- /dev/null
> +++ b/tests/Functional.openhpid/openhpid_test.sh
> @@ -0,0 +1,4 @@
> +#!/bin/sh
> +for i in tests/*.sh; do
> +    sh $i
> +done
> diff --git a/tests/Functional.openhpid/parser.py
> b/tests/Functional.openhpid/parser.py
> new file mode 100644
> index 0000000..d85abd7
> --- /dev/null
> +++ b/tests/Functional.openhpid/parser.py
> @@ -0,0 +1,22 @@
> +#!/usr/bin/python
> +# See common.py for description of command-line arguments
> +
> +import os, sys, collections
> +
> +sys.path.insert(0, os.environ['FUEGO_CORE'] + '/engine/scripts/parser')
> +import common as plib
> +
> +measurements = {}
> +measurements = collections.OrderedDict()
> +
> +regex_string = '^ -> (.*): TEST-(.*)$'
> +matches = plib.parse_log(regex_string)
> +
> +if matches:
> +    for m in matches:
> +        measurements['default.' + m[0]] = 'PASS' if m[1] == 'PASS' else 'FAIL'
> +
> +# split the output for each testcase
> +plib.split_output_per_testcase(regex_string, measurements)
> +
> +sys.exit(plib.process(measurements))
> diff --git a/tests/Functional.openhpid/spec.json
> b/tests/Functional.openhpid/spec.json
> new file mode 100644
> index 0000000..9004c0d
> --- /dev/null
> +++ b/tests/Functional.openhpid/spec.json
> @@ -0,0 +1,7 @@
> +{
> +    "testName": "Functional.openhpid",
> +    "specs": {
> +        "default": {}
> +    }
> +}
> +
> diff --git a/tests/Functional.openhpid/tests/openhpid_ps.sh
> b/tests/Functional.openhpid/tests/openhpid_ps.sh
> new file mode 100644
> index 0000000..f812827
> --- /dev/null
> +++ b/tests/Functional.openhpid/tests/openhpid_ps.sh
> @@ -0,0 +1,26 @@
> +#!/bin/sh
> +
> +#  In the target start openhpid, and confirm the process condition by
> command ps.
> +
> +test="ps"
> +
> +. ./fuego_board_function_lib.sh
> +
> +set_init_manager
> +
> +service_status=$(get_service_status openhpid)
> +
> +exec_service_on_target openhpid stop
> +
> +if exec_service_on_target openhpid start
> +then
> +    echo " -> $test: TEST-PASS"
> +else
> +    echo " -> start of openhpid failed."
> +    echo " -> $test: TEST-FAIL"
> +fi
> +
> +if [ "$service_status" = "inactive" ]
> +then
> +    exec_service_on_target openhpid stop
> +fi
> --
> 1.8.3.1
> 
> 
> 
> _______________________________________________
> Fuego mailing list
> Fuego at lists.linuxfoundation.org
> https://lists.linuxfoundation.org/mailman/listinfo/fuego


More information about the Fuego mailing list