Change in vdsm[master]: vmTests: move device tests out of vmTests

danken at redhat.com danken at redhat.com
Wed Feb 18 22:34:52 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: vmTests: move device tests out of vmTests
......................................................................


vmTests: move device tests out of vmTests

vmTests.py, like vm.py, has grown in size a lot. Since some of the
devices are already isolated in their own class, we can easily move
this out to different module. Also some of the devices were in the
TestVM class, where they don't belong and are therefore also moved out.

Change-Id: I3acc047765dd9cd709844253d5d22b4c1c7b62ba
Signed-off-by: Martin Polednik <mpolednik at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/37818
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M tests/Makefile.am
A tests/deviceTests.py
M tests/vmTests.py
3 files changed, 461 insertions(+), 434 deletions(-)

Approvals:
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Looks good to me, but someone else must approve
  Martin Polednik: Verified



-- 
To view, visit http://gerrit.ovirt.org/37818
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3acc047765dd9cd709844253d5d22b4c1c7b62ba
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list