Vinzenz Feenstra has posted comments on this change.
Change subject: Added unit test libvirtvmTests.py:TestLibvirtvm.testBuildCmdLine ......................................................................
Patch Set 3:
I would say it'd be better if we could put the input for the scenarios and expected outputs into external files and load them from there and compare the results then. That would add more flexibility in writing the configurations and expected outputs.
Currently you have basically just 2 large strings containing the data. If you need to modify values in there, you could use some kind of template engine approach and replace the values on load.
-- To view, visit http://gerrit.ovirt.org/14111 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I74b898a6398a72608d7933009644703aa3f8d831 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Maciej Lichon maciej.lichon.wroclaw@gmail.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Maciej Lichon maciej.lichon.wroclaw@gmail.com Gerrit-Reviewer: Michal Skrivanek michal.skrivanek@redhat.com Gerrit-Reviewer: Vinzenz Feenstra vfeenstr@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server
vdsm-patches@lists.fedorahosted.org