Martin Polednik has uploaded a new change for review.

Change subject: vmtests: use assertXMLEqual in assertBuildCmdline
......................................................................

vmtests: use assertXMLEqual in assertBuildCmdline

Test method assertBuildCmdline worked around whitespace in it's own
way. We have AssertXMLEqual for that, leading to a more consistent code.

Change-Id: If38342d837374c51a862814288803915e4ac3f07
Signed-off-by: Martin Polednik <[email protected]>
---
M tests/vmTests.py
1 file changed, 1 insertion(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/73/56973/1

diff --git a/tests/vmTests.py b/tests/vmTests.py
index 1e45fda..6a0c555 100644
--- a/tests/vmTests.py
+++ b/tests/vmTests.py
@@ -117,9 +117,7 @@
 
                     output = testVm._buildDomainXML()
 
-                    self.assertEqual(
-                        re.sub('\n\s*', ' ', output.strip(' ')),
-                        re.sub('\n\s*', ' ', expectedXML.strip(' ')))
+                    self.assertXMLEqual(output, expectedXML)
 
     def testDomXML(self):
         expectedXML = """


-- 
To view, visit https://gerrit.ovirt.org/56973
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If38342d837374c51a862814288803915e4ac3f07
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to