[libvirt] [PATCH 4/8] tests: Introduce tests for storage pool xml to argv checks
Michal Privoznik
mprivozn at redhat.com
Wed Dec 12 14:04:07 UTC 2018
On 12/4/18 5:47 PM, John Ferlan wrote:
> Similar to qemuxml2argv and storagevolxml2argv, let's create some
> tests to ensure that the XML generates a consistent command line.
>
> Using the same list of pools as storagepoolxml2xmltest, start with
> the file system tests (fs, netfs, netfs-cifs, netfs-gluster).
>
> Signed-off-by: John Ferlan <jferlan at redhat.com>
> ---
> tests/Makefile.am | 12 ++
> tests/storagepoolxml2argvdata/pool-fs.argv | 1 +
> .../pool-netfs-cifs.argv | 1 +
> .../pool-netfs-gluster.argv | 1 +
> tests/storagepoolxml2argvdata/pool-netfs.argv | 1 +
> tests/storagepoolxml2argvtest.c | 171 ++++++++++++++++++
> 6 files changed, 187 insertions(+)
> create mode 100644 tests/storagepoolxml2argvdata/pool-fs.argv
> create mode 100644 tests/storagepoolxml2argvdata/pool-netfs-cifs.argv
> create mode 100644 tests/storagepoolxml2argvdata/pool-netfs-gluster.argv
> create mode 100644 tests/storagepoolxml2argvdata/pool-netfs.argv
> create mode 100644 tests/storagepoolxml2argvtest.c
>
> diff --git a/tests/Makefile.am b/tests/Makefile.am
> index d7ec7e3a6f..bec0930c11 100644
> --- a/tests/Makefile.am
> +++ b/tests/Makefile.am
> @@ -140,6 +140,7 @@ EXTRA_DIST = \
> storagepoolschemadata \
> storagepoolxml2xmlin \
> storagepoolxml2xmlout \
> + storagepoolxml2argvdata \
> storagevolschemadata \
> storagevolxml2argvdata \
> storagevolxml2xmlin \
> @@ -363,6 +364,7 @@ endif WITH_NWFILTER
>
> if WITH_STORAGE
> test_programs += storagevolxml2argvtest
> +test_programs += storagepoolxml2argvtest
> test_programs += virstorageutiltest
> endif WITH_STORAGE
>
> @@ -901,6 +903,16 @@ storagevolxml2argvtest_LDADD = \
> ../src/libvirt_util.la \
> $(LDADDS)
>
> +storagepoolxml2argvtest_SOURCES = \
> + storagepoolxml2argvtest.c \
> + testutils.c testutils.h
> +storagepoolxml2argvtest_LDADD = \
> + $(LIBXML_LIBS) \
> + ../src/libvirt_driver_storage_impl.la \
> + ../src/libvirt_conf.la \
> + ../src/libvirt_util.la \
> + $(LDADDS)
> +
> else ! WITH_STORAGE
> EXTRA_DIST += storagevolxml2argvtest.c
> EXTRA_DIST += virstorageutiltest.c
> diff --git a/tests/storagepoolxml2argvdata/pool-fs.argv b/tests/storagepoolxml2argvdata/pool-fs.argv
> new file mode 100644
> index 0000000000..4a94148114
> --- /dev/null
> +++ b/tests/storagepoolxml2argvdata/pool-fs.argv
> @@ -0,0 +1 @@
> +/usr/bin/mount -t ext3 /dev/sda6 /mnt
Problem is, on my system (Gentoo), it is /usr/mount so this is failing
for me.
I wonder if we should have a placeholder here, say "MOUNT" and then
replace it with actual location of 'mount' (taken from config.h) at runtime.
Michal
More information about the libvir-list
mailing list