Change in vdsm[master]: tests: negative tests for updateDevice - graphics

danken at redhat.com danken at redhat.com
Tue May 26 10:12:41 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: tests: negative tests for updateDevice - graphics
......................................................................


tests: negative tests for updateDevice - graphics

Failures are more cumbersome to tests, due to need
to inject errors or to cause them.
Add tests for Vm.updateDevice for Graphics devices,
to do basic sanity tests, and to help verification of
the upcoming response.error() transition patches.

Change-Id: Icf3d03f125f9c53339d9ad7fef8c31482bab0bfb
Signed-off-by: Francesco Romani <fromani at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/38665
Continuous-Integration: Jenkins CI
Reviewed-by: Martin Polednik <mpolednik at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M tests/vmTests.py
1 file changed, 49 insertions(+), 16 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Icf3d03f125f9c53339d9ad7fef8c31482bab0bfb
Gerrit-PatchSet: 12
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: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list