Omer Frenkel has submitted this change and it was merged. Change subject: engine: Fetching cluster Id code moved from constructor to canDoAction method. ...................................................................... engine: Fetching cluster Id code moved from constructor to canDoAction method. It is difficult to write test case without static mocking, because the code uses DbFacade from constructor. Hence it is moved to canDoAction. Change-Id: I187f4fa678389dbbd271d487bbbf2b81e5c0080c Signed-off-by: Selvasundaram <sesubram(a)redhat.com&gt; Signed-off-by: Allon Mureinik <amureini(a)redhat.com&gt; --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterVolumeCommandBase.java 1 file changed, 6 insertions(+), 7 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, but someone else must approve Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6365 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I187f4fa678389dbbd271d487bbbf2b81e5c0080c Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Selvasundaram <sesubram(a)redhat.com&gt; Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com&gt; Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com&gt; Gerrit-Reviewer: Selvasundaram <sesubram(a)redhat.com&gt; Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com&gt;