We are missing a review for this patch, please.
02/03/2020 21:41, Thinh Tran: > - When running the dedault configuration of autotest of the make test > it'd take 900 seconds (15 minutes) for the script TIMEOUT and marks > Failed for a specific device test that is not supported on the > system under test. > - Adding the checking for those tests, print out as "Skipped [Not Run]" > quickly return and continue for next test > > Signed-off-by: Thinh Tran <thin...@linux.vnet.ibm.com> > --- > --- a/app/test/autotest_test_funcs.py > +++ b/app/test/autotest_test_funcs.py > @@ -12,13 +12,16 @@ > 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", pexpect.TIMEOUT, > + "Test Skipped"], timeout=900) > if result == 1: > return -1, "Fail" > elif result == 2: > return -1, "Fail [Not found]" > elif result == 3: > return -1, "Fail [Timeout]" > + elif result == 4: > + return 0, "Skipped [Not Run]" > return 0, "Success"
