[v2,09/10] autotest: update result for skipped test cases
Checks
Commit Message
Fixed in autotest_test_funcs.py to handle test cases
which returns "Skipped" as result.
The issue was skipped test cases got timed out,
causing delay in autotests execution.
Cc: stable@dpdk.org
Signed-off-by: Jananee Parthasarathy <jananeex.m.parthasarathy@intel.com>
Signed-off-by: Reshma Pattan <reshma.pattan@intel.com>
---
test/test/autotest_test_funcs.py | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
Comments
On 13-Jul-18 5:20 PM, Reshma Pattan wrote:
> Fixed in autotest_test_funcs.py to handle test cases
> which returns "Skipped" as result.
> The issue was skipped test cases got timed out,
> causing delay in autotests execution.
>
> Cc: stable@dpdk.org
>
> Signed-off-by: Jananee Parthasarathy <jananeex.m.parthasarathy@intel.com>
> Signed-off-by: Reshma Pattan <reshma.pattan@intel.com>
> ---
Is this patch still applicable? Patch 5 in these series causes skipped
tests to not get executed in the first place, so by the time we reach
this patch it is no longer a problem, i think.
@@ -1,5 +1,5 @@
# SPDX-License-Identifier: BSD-3-Clause
-# Copyright(c) 2010-2014 Intel Corporation
+# Copyright(c) 2010-2018 Intel Corporation
# Test functions
@@ -12,12 +12,14 @@
def default_autotest(child, test_name):
child.sendline(test_name)
result = child.expect(["Test OK", "Test Failed",
- "Command not found", pexpect.TIMEOUT], timeout=900)
+ "Command not found", "Skipped", pexpect.TIMEOUT], timeout=900)
if result == 1:
return -1, "Fail"
elif result == 2:
return -1, "Fail [Not found]"
elif result == 3:
+ return -1, "Fail [Test returns Skipped]"
+ elif result == 4:
return -1, "Fail [Timeout]"
return 0, "Success"