[Libguestfs] [PATCH 2/2] tests: fix podcheck tests

Richard W.M. Jones rjones at redhat.com
Thu Dec 12 08:05:03 UTC 2019


On Wed, Dec 11, 2019 at 02:26:33PM +0100, Pino Toscano wrote:
> Pass to the various podcheck invocations the paths with POD files that
> are included.
> 
> Followup of commit 46e59e9535c2fcd1c188464b5249a249f22af1a0.
> ---
>  cat/test-docs.sh                        | 13 +++++++++----
>  customize/test-virt-customize-docs.sh   |  1 +
>  diff/test-virt-diff-docs.sh             |  1 +
>  edit/test-virt-edit-docs.sh             |  3 ++-
>  fish/test-docs.sh                       |  4 +---
>  fuse/test-docs.sh                       |  3 ++-
>  get-kernel/test-virt-get-kernel-docs.sh |  3 ++-
>  inspector/test-virt-inspector-docs.sh   |  3 ++-
>  sparsify/test-virt-sparsify-docs.sh     |  1 +
>  sysprep/test-virt-sysprep-docs.sh       |  1 +
>  10 files changed, 22 insertions(+), 11 deletions(-)
> 
> diff --git a/cat/test-docs.sh b/cat/test-docs.sh
> index 30cc1eeb4..0d6d5461a 100755
> --- a/cat/test-docs.sh
> +++ b/cat/test-docs.sh
> @@ -21,9 +21,14 @@ set -e
>  $TEST_FUNCTIONS
>  skip_if_skipped
>  
> -$top_srcdir/podcheck.pl virt-cat.pod virt-cat
> -$top_srcdir/podcheck.pl virt-filesystems.pod virt-filesystems
> -$top_srcdir/podcheck.pl virt-log.pod virt-log
> +$top_srcdir/podcheck.pl virt-cat.pod virt-cat \
> +                        --path $top_srcdir/common/options
> +$top_srcdir/podcheck.pl virt-filesystems.pod virt-filesystems \
> +                        --path $top_srcdir/common/options
> +$top_srcdir/podcheck.pl virt-log.pod virt-log \
> +                        --path $top_srcdir/common/options
>  $top_srcdir/podcheck.pl virt-ls.pod virt-ls \
> +                        --path $top_srcdir/common/options \
>                          --ignore=--checksums,--extra-stat,--time,--uid
> -$top_srcdir/podcheck.pl virt-tail.pod virt-tail
> +$top_srcdir/podcheck.pl virt-tail.pod virt-tail \
> +                        --path $top_srcdir/common/options
> diff --git a/customize/test-virt-customize-docs.sh b/customize/test-virt-customize-docs.sh
> index 99e67cb10..9a8da5180 100755
> --- a/customize/test-virt-customize-docs.sh
> +++ b/customize/test-virt-customize-docs.sh
> @@ -21,6 +21,7 @@ set -e
>  $TEST_FUNCTIONS
>  
>  $top_srcdir/podcheck.pl virt-customize.pod virt-customize \
> +  --path $top_srcdir/common/options \
>    --insert $top_srcdir/common/mlcustomize/customize-synopsis.pod:__CUSTOMIZE_SYNOPSIS__ \
>    --insert $top_srcdir/common/mlcustomize/customize-options.pod:__CUSTOMIZE_OPTIONS__ \
>    --ignore=--dryrun
> diff --git a/diff/test-virt-diff-docs.sh b/diff/test-virt-diff-docs.sh
> index cf6ce728e..d0d93cfd2 100755
> --- a/diff/test-virt-diff-docs.sh
> +++ b/diff/test-virt-diff-docs.sh
> @@ -21,4 +21,5 @@ set -e
>  $TEST_FUNCTIONS
>  
>  $top_srcdir/podcheck.pl virt-diff.pod virt-diff \
> +  --path $top_srcdir/common/options \
>    --ignore=--checksums,--dir-link,--dir-nlink,--dir-nlinks,--dir-time,--extra-stat,--time,--uid,--xattr
> diff --git a/edit/test-virt-edit-docs.sh b/edit/test-virt-edit-docs.sh
> index d824db9d4..7805d7e1d 100755
> --- a/edit/test-virt-edit-docs.sh
> +++ b/edit/test-virt-edit-docs.sh
> @@ -20,4 +20,5 @@ set -e
>  
>  $TEST_FUNCTIONS
>  
> -$top_srcdir/podcheck.pl virt-edit.pod virt-edit
> +$top_srcdir/podcheck.pl virt-edit.pod virt-edit \
> +  --path $top_srcdir/common/options
> diff --git a/fish/test-docs.sh b/fish/test-docs.sh
> index 6630da43e..7e9f9928e 100755
> --- a/fish/test-docs.sh
> +++ b/fish/test-docs.sh
> @@ -22,7 +22,5 @@ $TEST_FUNCTIONS
>  skip_if_skipped
>  
>  $top_srcdir/podcheck.pl guestfish.pod guestfish \
> -    --insert $srcdir/guestfish-actions.pod:__ACTIONS__ \
> -    --insert $srcdir/guestfish-commands.pod:__FISH_COMMANDS__ \
> -    --insert $srcdir/guestfish-prepopts.pod:__PREPOPTS__ \
> +    --path $srcdir --path $top_srcdir/common/options \
>      --ignore=-D
> diff --git a/fuse/test-docs.sh b/fuse/test-docs.sh
> index ba2c9c57a..0edc25658 100755
> --- a/fuse/test-docs.sh
> +++ b/fuse/test-docs.sh
> @@ -21,7 +21,8 @@ set -e
>  $TEST_FUNCTIONS
>  skip_if_skipped
>  
> -$top_srcdir/podcheck.pl guestmount.pod guestmount
> +$top_srcdir/podcheck.pl guestmount.pod guestmount \
> +  --path $top_srcdir/common/options
>  
>  # guestunmount doesn't implement bash completion, so we cannot
>  # test it at the moment. XXX
> diff --git a/get-kernel/test-virt-get-kernel-docs.sh b/get-kernel/test-virt-get-kernel-docs.sh
> index 7c454777e..40bb7334d 100755
> --- a/get-kernel/test-virt-get-kernel-docs.sh
> +++ b/get-kernel/test-virt-get-kernel-docs.sh
> @@ -21,4 +21,5 @@ set -e
>  $TEST_FUNCTIONS
>  skip_if_skipped
>  
> -$top_srcdir/podcheck.pl virt-get-kernel.pod virt-get-kernel
> +$top_srcdir/podcheck.pl virt-get-kernel.pod virt-get-kernel \
> +  --path $top_srcdir/common/options
> diff --git a/inspector/test-virt-inspector-docs.sh b/inspector/test-virt-inspector-docs.sh
> index c9ad00b6b..11ee6518f 100755
> --- a/inspector/test-virt-inspector-docs.sh
> +++ b/inspector/test-virt-inspector-docs.sh
> @@ -21,4 +21,5 @@ set -e
>  $TEST_FUNCTIONS
>  skip_if_skipped
>  
> -$top_srcdir/podcheck.pl virt-inspector.pod virt-inspector
> +$top_srcdir/podcheck.pl virt-inspector.pod virt-inspector \
> +  --path $top_srcdir/common/options
> diff --git a/sparsify/test-virt-sparsify-docs.sh b/sparsify/test-virt-sparsify-docs.sh
> index 2afca3b12..6c2bd8f89 100755
> --- a/sparsify/test-virt-sparsify-docs.sh
> +++ b/sparsify/test-virt-sparsify-docs.sh
> @@ -22,4 +22,5 @@ $TEST_FUNCTIONS
>  skip_if_skipped
>  
>  $top_srcdir/podcheck.pl virt-sparsify.pod virt-sparsify \
> +                        --path $top_srcdir/common/options \
>                          --ignore=--inplace
> diff --git a/sysprep/test-virt-sysprep-docs.sh b/sysprep/test-virt-sysprep-docs.sh
> index e950410d7..b31e0f097 100755
> --- a/sysprep/test-virt-sysprep-docs.sh
> +++ b/sysprep/test-virt-sysprep-docs.sh
> @@ -22,6 +22,7 @@ $TEST_FUNCTIONS
>  skip_if_skipped
>  
>  $top_srcdir/podcheck.pl virt-sysprep.pod virt-sysprep \
> +    --path $top_srcdir/common/options \
>      --insert sysprep-extra-options.pod:__EXTRA_OPTIONS__ \
>      --insert sysprep-operations.pod:__OPERATIONS__ \
>      --ignore=--dryrun,--dump-pod,--dump-pod-options,--no-selinux-relabel
> -- 
> 2.23.0
> 

Sorry I missed this, thanks for fixing it.

ACK series.

Rich.

-- 
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
virt-df lists disk usage of guests without needing to install any
software inside the virtual machine.  Supports Linux and Windows.
http://people.redhat.com/~rjones/virt-df/




More information about the Libguestfs mailing list