Add nodedevxml2xml test

Add a test to check node device XML parsing by first parsing the XML, then re-formatting as XML and finally comparing the resulting XML to the original XML.

Add nodedevxml2xml test
Add a test to check node device XML parsing by first parsing the XML, then re-formatting as XML and finally comparing the resulting XML to the original XML.
da61daa2 · Mark McLoughlin · c4c5e9db · da61daa2 · da61daa2 · da61daa2
16 changed file
--- a/ChangeLog
+++ b/ChangeLog
+Tue Feb 24 14:55:28 GMT 2009 Mark McLoughlin <markmc@redhat.com>
+
+	* tests/nodedevxml2xmltest.c: Add a test to check node
+	device XML parsing by first parsing the XML, then
+	re-formatting as XML and finally comparing the resulting
+	XML to the original XML.
+
+	* tests/nodedevschemadata/*.xml: strip trailing newlines.
+
+	* tests/Makefile.am, tests/.*ignore: adminstrivia
+
 Tue Feb 24 14:54:05 GMT 2009 Mark McLoughlin <markmc@redhat.com>

 	*  src/libvirt_private.syms, src/node_device_conf.[ch]:

--- a/tests/.cvsignore
+++ b/tests/.cvsignore
@@ -11,6 +11,7 @@ xmconfigtest
 xencapstest
 qemuxml2xmltest
 qemuxml2argvtest
+nodedevxml2xmltest
 nodeinfotest
 statstest
 qparamtest

--- a/tests/.gitignore
+++ b/tests/.gitignore
@@ -11,6 +11,7 @@ xmconfigtest
 xencapstest
 qemuxml2xmltest
 qemuxml2argvtest
+nodedevxml2xmltest
 nodeinfotest
 statstest
 qparamtest

--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -64,6 +64,8 @@ if WITH_QEMU
 noinst_PROGRAMS += qemuxml2argvtest qemuxml2xmltest
 endif

+noinst_PROGRAMS += nodedevxml2xmltest
+
 test_scripts = \
 	capabilityschematest \
 	networkschematest \
@@ -110,6 +112,8 @@ if WITH_QEMU
 TESTS += qemuxml2argvtest qemuxml2xmltest
 endif

+TESTS += nodedevxml2xmltest
+
 path_add = $$abs_top_builddir/src$(PATH_SEPARATOR)$$abs_top_builddir/qemud

 # NB, automake < 1.10 does not provide the real
@@ -175,6 +179,11 @@ else
 EXTRA_DIST += qemuxml2argvtest.c qemuxml2xmltest.c testutilsqemu.c testutilsqemu.h
 endif

+nodedevxml2xmltest_SOURCES = \
+	nodedevxml2xmltest.c \
+	testutils.c testutils.h
+nodedevxml2xmltest_LDADD = ../src/libvirt_driver_qemu.la $(LDADDS)
+
 virshtest_SOURCES = \
 	virshtest.c \
 	testutils.c testutils.h

--- a/tests/nodedevschemadata/DVD_GCC_4247N.xml
+++ b/tests/nodedevschemadata/DVD_GCC_4247N.xml
@@ -13,5 +13,3 @@
    </capability>
  </capability>
 </device>
-
-
--- a/tests/nodedevschemadata/computer.xml
+++ b/tests/nodedevschemadata/computer.xml
@@ -14,5 +14,3 @@
    </firmware>
  </capability>
 </device>
-
-
--- a/tests/nodedevschemadata/net_00_13_02_b9_f9_d3.xml
+++ b/tests/nodedevschemadata/net_00_13_02_b9_f9_d3.xml
@@ -7,5 +7,3 @@
    <capability type='80211'/>
  </capability>
 </device>
-
-
--- a/tests/nodedevschemadata/net_00_15_58_2f_e9_55.xml
+++ b/tests/nodedevschemadata/net_00_15_58_2f_e9_55.xml
@@ -7,5 +7,3 @@
    <capability type='80203'/>
  </capability>
 </device>
-
-
--- a/tests/nodedevschemadata/pci_1002_71c4.xml
+++ b/tests/nodedevschemadata/pci_1002_71c4.xml
@@ -10,5 +10,3 @@
    <vendor id='0x1002'>ATI Technologies Inc</vendor>
  </capability>
 </device>
-
-
--- a/tests/nodedevschemadata/pci_8086_27c5_scsi_host.xml
+++ b/tests/nodedevschemadata/pci_8086_27c5_scsi_host.xml
@@ -5,5 +5,3 @@
    <host>0</host>
  </capability>
 </device>
-
-
--- a/tests/nodedevschemadata/pci_8086_27c5_scsi_host_0.xml
+++ b/tests/nodedevschemadata/pci_8086_27c5_scsi_host_0.xml
@@ -5,5 +5,3 @@
    <host>1</host>
  </capability>
 </device>
-
-
--- a/tests/nodedevschemadata/pci_8086_27c5_scsi_host_scsi_device_lun0.xml
+++ b/tests/nodedevschemadata/pci_8086_27c5_scsi_host_scsi_device_lun0.xml
@@ -9,5 +9,3 @@
    <type>disk</type>
  </capability>
 </device>
-
-
--- a/tests/nodedevschemadata/pci_8086_27c5_scsi_host_scsi_host.xml
+++ b/tests/nodedevschemadata/pci_8086_27c5_scsi_host_scsi_host.xml
@@ -5,5 +5,3 @@
    <host>0</host>
  </capability>
 </device>
-
-
--- a/tests/nodedevschemadata/storage_serial_SATA_HTS721010G9SA00_MPCZ12Y0GNGWSE.xml
+++ b/tests/nodedevschemadata/storage_serial_SATA_HTS721010G9SA00_MPCZ12Y0GNGWSE.xml
@@ -10,5 +10,3 @@
    <size>100030242816</size>
  </capability>
 </device>
-
-
--- a/tests/nodedevschemadata/usb_device_1d6b_1_0000_00_1d_0.xml
+++ b/tests/nodedevschemadata/usb_device_1d6b_1_0000_00_1d_0.xml
@@ -8,5 +8,3 @@
    <vendor id='0x1d6b'>Linux Foundation</vendor>
  </capability>
 </device>
-
-
--- a/tests/nodedevschemadata/usb_device_1d6b_1_0000_00_1d_0_if0.xml
+++ b/tests/nodedevschemadata/usb_device_1d6b_1_0000_00_1d_0_if0.xml
@@ -8,5 +8,3 @@
    <protocol>0</protocol>
  </capability>
 </device>
-
-