Dan Kenigsberg has submitted this change and it was merged. Change subject: tests: v2v: refactor tests to make it extensible ......................................................................
tests: v2v: refactor tests to make it extensible Refactor the current tests to make it easier to add new ones reusing existing code. Besides refactoring, the only change is that testGetExternalVMs tests for an handful of VMs, instead of just one. Change-Id: I5a0c2f9575debd328a2c8e0dacdb8e8dc845e340 Signed-off-by: Francesco Romani <[email protected]> Reviewed-on: https://gerrit.ovirt.org/47366 Reviewed-by: Nir Soffer <[email protected]> Reviewed-by: Shahar Havivi <[email protected]> Continuous-Integration: Jenkins CI Reviewed-by: Dan Kenigsberg <[email protected]> --- M tests/v2vTests.py 1 file changed, 69 insertions(+), 32 deletions(-) Approvals: Nir Soffer: Looks good to me, but someone else must approve Shahar Havivi: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved Francesco Romani: Verified -- To view, visit https://gerrit.ovirt.org/47366 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5a0c2f9575debd328a2c8e0dacdb8e8dc845e340 Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Francesco Romani <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Shahar Havivi <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
