Hi! > Now we verify the exit code of the second test case. We did not > do this previously, and it was a bug. > > Signed-off-by: Stanislav Kholmanskikh <stanislav.kholmansk...@oracle.com> > --- > testcases/kernel/fs/acls/acl_test01 | 3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-) > > diff --git a/testcases/kernel/fs/acls/acl_test01 > b/testcases/kernel/fs/acls/acl_test01 > index be88aa8..0f050b9 100755 > --- a/testcases/kernel/fs/acls/acl_test01 > +++ b/testcases/kernel/fs/acls/acl_test01 > @@ -228,7 +228,8 @@ else > chown $TEST_USER1 $FILE_ACL > chown $TEST_USER1 $FILE_ACL_LINK > > - su $TEST_USER1 -c "$0" > + su $TEST_USER1 -c "$0" & > + tst_record_childstatus $!
We should probably wrap the su into a script that makes sure that the return value is compatible with LTP. Because it may set bogus test return status if su reported anything else than 0 or 1. Or we may add just another function into test.sh that would work similariry to tst_record_childstatus but that will normalize the return value. I.e. 0 == TPASS, anything else TFAIL. -- Cyril Hrubis chru...@suse.cz ------------------------------------------------------------------------------ _______________________________________________ Ltp-list mailing list Ltp-list@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ltp-list