Yaniv Bronhaim has uploaded a new change for review.

Change subject: Core: Fix for VmMapperTests
......................................................................

Core: Fix for VmMapperTests

Adding mock for OsRepository. The sequence of the tests is
not constant, and the mock wasn't set before the test for some runs that
testMapHostId ran before the other vmMapperTests.

Change-Id: I80a76b5fea41e3a9b914bed2034f14b20fde4117
Signed-off-by: Yaniv Bronhaim <[email protected]>
---
M 
backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/VmMapperTest.java
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/82/16682/1

diff --git 
a/backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/VmMapperTest.java
 
b/backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/VmMapperTest.java
index 46446fe..8451169 100644
--- 
a/backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/VmMapperTest.java
+++ 
b/backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/VmMapperTest.java
@@ -183,6 +183,7 @@
         entity.setStatus(VMStatus.Up);
         Guid guid = Guid.newGuid();
         entity.setRunOnVds(guid);
+        OsTypeMockUtils.mockOsTypes();
         VM model = VmMapper.map(entity, null);
         assertEquals(guid.toString(), model.getHost().getId());
     }


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I80a76b5fea41e3a9b914bed2034f14b20fde4117
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to