Change in vdsm[master]: tests: v2v: refactor tests to make it extensible

danken at redhat.com danken at redhat.com
Tue Nov 3 11:19:59 UTC 2015


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 <fromani at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/47366
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-by: Shahar Havivi <shavivi at redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
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 <fromani at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list