[dts] [PATCH V1 5/5] add check case function in run case module
Liu, Yong
yong.liu at intel.com
Fri Aug 7 08:52:40 CEST 2015
One question about skip_case_mode.case_skip.
> -----Original Message-----
> From: dts [mailto:dts-bounces at dpdk.org] On Behalf Of huilong,xu
> Sent: Friday, August 07, 2015 2:03 PM
> To: dts at dpdk.org
> Subject: [dts] [PATCH V1 5/5] add check case function in run case module
>
> From: huilong xu <huilongx.xu at intel.com>
>
>
> Signed-off-by: huilong xu <huilongx.xu at intel.com>
> ---
> framework/dts.py | 21 +++++++++++++++------
> 1 files changed, 15 insertions(+), 6 deletions(-)
>
> diff --git a/framework/dts.py b/framework/dts.py
> index e9513c6..cb8f26e 100644
> --- a/framework/dts.py
> +++ b/framework/dts.py
> @@ -57,7 +57,7 @@ from logger import getLogger
> import logger
> import debugger
> from virt_scene import VirtScene
> -
> +from checkCase import *
> import sys
> reload(sys)
> sys.setdefaultencoding('UTF8')
> @@ -291,6 +291,8 @@ def dts_run_target(crbInst, targets, test_suites, nic,
> scenario):
> """
> Run each target in execution targets.
> """
> + global skip_case_mode
> + skip_case_mode = check_case_skip(dut)
> if scenario != '':
> scene = VirtScene(dut, tester, scenario)
> else:
> @@ -299,7 +301,7 @@ def dts_run_target(crbInst, targets, test_suites, nic,
> scenario):
> if scene:
> scene.load_config()
> scene.create_scene()
> -
> +
> for target in targets:
> log_handler.info("\nTARGET " + target)
> result.target = target
> @@ -402,11 +404,12 @@ def run_all(config_file, pkgName, git, patch,
> skip_setup,
> global Package
> global Patches
> global scenario
> -
> + global skip_case_mode
> # save global variable
> Package = pkgName
> Patches = patch
> -
> + check_case = parse_file()
> + check_case.set_filter_case()
> # prepare the output folder
> if output_dir == '':
> output_dir = FOLDERS['Output']
> @@ -459,7 +462,6 @@ def run_all(config_file, pkgName, git, patch,
> skip_setup,
>
> # verify if the delimiter is good if the lists are vertical
> dutIP, targets, test_suites, nics, scenario =
> dts_parse_config(section)
> -
> log_handler.info("\nDUT " + dutIP)
>
> # look up in crbs - to find the matching IP
> @@ -479,6 +481,7 @@ def run_all(config_file, pkgName, git, patch,
> skip_setup,
> # init dut, tester crb
> dts_crbs_init(crbInst, skip_setup, read_cache, project, base_dir,
> nics, virttype)
>
> + skip_case_mode = check_case_skip(dut)
> # Run DUT prerequisites
> if dts_run_prerequisties(pkgName, patch) is False:
> dts_crbs_exit()
> @@ -576,6 +579,7 @@ def execute_all_test_cases(test_suite):
> """
> if functional_only:
> for test_case in get_functional_test_cases(test_suite):
> + skip_case_mode.case_skip(test_case)
Why call this function here? In function execute_test_case also check whether skip test case.
> execute_test_case(test_suite, test_case)
> if performance_only:
> for test_case in get_performance_test_cases(test_suite):
> @@ -591,8 +595,13 @@ def execute_test_case(test_suite, test_case):
> global debug_case
> global module
> result.test_case = test_case.__name__
> -
> rst.write_title("Test Case: " + test_case.__name__)
> + if skip_case_mode.case_skip(test_case.__name__[len("test_"):]):
> + log_handler.info('Test Case %s Result SKIPED:' %
> test_case.__name__)
> + rst.write_result("N/A")
> + result.test_case_skip(skip_case_mode.comments)
> + save_all_results()
> + return
> if performance_only:
> rst.write_annex_title("Annex: " + test_case.__name__)
> try:
> --
> 1.7.4.4
More information about the dts
mailing list