Message ID | 1621222453-2824-1-git-send-email-huangjh.jy@cn.fujitsu.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | xfs: Add the appropriate _require helper | expand |
On Mon, May 17, 2021 at 11:34:13AM +0800, Jinhui Huang wrote: > 1.xfs/162 uses xfs_db -c 'fuzz' but forgets to check if the feature > is supported. This will cause the case to fail on a system without > fuzz support. so we add _require to check if the fuzz is supported. > > 2.xfs/495 use _require_scratch_xfs_fuzz_fields to check the features > required by field fuzzing, but some of the features are not used in > this case like xfs_scrub, this will cause the case to skip on a system > without xfs_scrub support, even if the features being uesd are supported. > So we just need to use _require to check the features being used. > > Signed-off-by: Jinhui Huang <huangjh.jy@cn.fujitsu.com> These targeted fixes ought to be in separate patches, but otherwise they look correct. Reviewed-by: Darrick J. Wong <djwong@kernel.org> --D > --- > tests/xfs/162 | 2 ++ > tests/xfs/495 | 4 +++- > 2 files changed, 5 insertions(+), 1 deletion(-) > > diff --git a/tests/xfs/162 b/tests/xfs/162 > index e3257a3..c7c72d1 100755 > --- a/tests/xfs/162 > +++ b/tests/xfs/162 > @@ -31,6 +31,8 @@ _cleanup() > # real QA test starts here > _supported_fs xfs > _require_scratch_nocheck > +_require_populate_commands > +_require_xfs_db_command "fuzz" > > rm -f $seqres.full > > diff --git a/tests/xfs/495 b/tests/xfs/495 > index 6292b5e..a3b4fb2 100755 > --- a/tests/xfs/495 > +++ b/tests/xfs/495 > @@ -32,7 +32,9 @@ _cleanup() > > # real QA test starts here > _supported_fs xfs > -_require_scratch_xfs_fuzz_fields > +_require_scratch_nocheck > +_require_populate_commands > +_require_xfs_db_command "fuzz" > > echo "Format and populate" > _scratch_populate_cached nofill > $seqres.full 2>&1 > -- > 1.8.3.1 >
diff --git a/tests/xfs/162 b/tests/xfs/162 index e3257a3..c7c72d1 100755 --- a/tests/xfs/162 +++ b/tests/xfs/162 @@ -31,6 +31,8 @@ _cleanup() # real QA test starts here _supported_fs xfs _require_scratch_nocheck +_require_populate_commands +_require_xfs_db_command "fuzz" rm -f $seqres.full diff --git a/tests/xfs/495 b/tests/xfs/495 index 6292b5e..a3b4fb2 100755 --- a/tests/xfs/495 +++ b/tests/xfs/495 @@ -32,7 +32,9 @@ _cleanup() # real QA test starts here _supported_fs xfs -_require_scratch_xfs_fuzz_fields +_require_scratch_nocheck +_require_populate_commands +_require_xfs_db_command "fuzz" echo "Format and populate" _scratch_populate_cached nofill > $seqres.full 2>&1
1.xfs/162 uses xfs_db -c 'fuzz' but forgets to check if the feature is supported. This will cause the case to fail on a system without fuzz support. so we add _require to check if the fuzz is supported. 2.xfs/495 use _require_scratch_xfs_fuzz_fields to check the features required by field fuzzing, but some of the features are not used in this case like xfs_scrub, this will cause the case to skip on a system without xfs_scrub support, even if the features being uesd are supported. So we just need to use _require to check the features being used. Signed-off-by: Jinhui Huang <huangjh.jy@cn.fujitsu.com> --- tests/xfs/162 | 2 ++ tests/xfs/495 | 4 +++- 2 files changed, 5 insertions(+), 1 deletion(-)