Re: [PATCH] generic: add _require_block_device to more tests

[Eryu Guan <eguan@redhat.com>]

On Fri, Jul 01, 2016 at 11:18:32AM +0800, Eryu Guan wrote:
> generic/067 mounts $SCRATCH_DEV directly in the test, assuming it's
> a block device. generic/299 and generic/300 query the size of
> $SCRATCH_DEV by running 'blockdev --getsz $SCRATCH_DEV'.
> 
> So add the check to make sure $SCRATCH_DEV is a real block device in
> these tests.
> 
> Signed-off-by: Eryu Guan <eguan@redhat.com>

Ping on this as well.

Thanks,
Eryu

> ---
>  tests/generic/067 | 1 +
>  tests/generic/299 | 1 +
>  tests/generic/300 | 1 +
>  3 files changed, 3 insertions(+)
> 
> diff --git a/tests/generic/067 b/tests/generic/067
> index 8a7a55c..8f2baf5 100755
> --- a/tests/generic/067
> +++ b/tests/generic/067
> @@ -56,6 +56,7 @@ _require_test_symlinks
>  _require_test
>  _require_scratch
>  _require_loop
> +_require_block_device $SCRATCH_DEV
>  
>  rm -f $seqres.full
>  
> diff --git a/tests/generic/299 b/tests/generic/299
> index 40265c8..c9ed0e0 100755
> --- a/tests/generic/299
> +++ b/tests/generic/299
> @@ -43,6 +43,7 @@ _supported_fs generic
>  _supported_os Linux
>  _require_scratch
>  _require_odirect
> +_require_block_device $SCRATCH_DEV
>  
>  NUM_JOBS=$((4*LOAD_FACTOR))
>  BLK_DEV_SIZE=`blockdev --getsz $SCRATCH_DEV`
> diff --git a/tests/generic/300 b/tests/generic/300
> index c187f43..181594e 100755
> --- a/tests/generic/300
> +++ b/tests/generic/300
> @@ -43,6 +43,7 @@ _supported_fs generic
>  _supported_os Linux
>  _require_scratch
>  _require_odirect
> +_require_block_device $SCRATCH_DEV
>  
>  # xfs_io is not required for this test, but it's the best way to verify
>  # the test system supports fallocate() for allocation and hole punching
> -- 
> 2.7.4
>