Change in ovirt-engine[master]: engine: fix RunVmCommandTest

lhornyak at redhat.com lhornyak at redhat.com
Wed Aug 28 07:09:23 UTC 2013


Laszlo Hornyak has submitted this change and it was merged.

Change subject: engine: fix RunVmCommandTest
......................................................................


engine: fix RunVmCommandTest

OsRepository mocking was missing from the test, while the code was built
on the assumption that it is created and configured already. This test
failed in unlucky (but not impossible) situations when it ran before any
other tests that could have initialized the OsRepository.

Change-Id: I6c7b8935273b91da8d1c06eb9c9867212ca022c1
Signed-off-by: Laszlo Hornyak <lhornyak at redhat.com>
---
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/RunVmCommandTest.java
1 file changed, 9 insertions(+), 0 deletions(-)

Approvals:
  Laszlo Hornyak: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6c7b8935273b91da8d1c06eb9c9867212ca022c1
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Laszlo Hornyak <lhornyak at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme at redhat.com>
Gerrit-Reviewer: Laszlo Hornyak <lhornyak at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list