Kanagaraj M has uploaded a new change for review.

Change subject: gluster: fix testcase failure in VolumeDao
......................................................................

gluster: fix testcase failure in VolumeDao

Empty GUID is assigned to taskId of Volume while reading from database.
But this assignment was missing in the DaoTest, added it now

Change-Id: I104f59d9dc170a1b151046e16de6ad5c35279e55
Signed-off-by: Kanagaraj M <[email protected]>
---
M 
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/gluster/GlusterVolumeDaoTest.java
1 file changed, 5 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/60/21260/1

diff --git 
a/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/gluster/GlusterVolumeDaoTest.java
 
b/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/gluster/GlusterVolumeDaoTest.java
index 26ce9b3..db8e285 100644
--- 
a/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/gluster/GlusterVolumeDaoTest.java
+++ 
b/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/gluster/GlusterVolumeDaoTest.java
@@ -12,6 +12,7 @@
 import java.util.Set;
 
 import org.junit.Test;
+import org.ovirt.engine.core.common.asynctasks.gluster.GlusterAsyncTask;
 import org.ovirt.engine.core.common.asynctasks.gluster.GlusterTaskType;
 import org.ovirt.engine.core.common.businessentities.VdsStatic;
 import org.ovirt.engine.core.common.businessentities.gluster.AccessProtocol;
@@ -402,6 +403,10 @@
         volume.addAccessProtocol(AccessProtocol.GLUSTER);
         volume.addAccessProtocol(AccessProtocol.NFS);
 
+        GlusterAsyncTask asyncTask = new GlusterAsyncTask();
+        asyncTask.setTaskId(Guid.Empty);
+        volume.setAsyncTask(asyncTask);
+
         GlusterBrickEntity brick = new GlusterBrickEntity();
         brick.setVolumeId(volumeId);
         brick.setServerId(server.getId());


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

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

Reply via email to