ofri masad has uploaded a new change for review. Change subject: core: Fix test for UpdateQuotaCommand (#848389) ......................................................................
core: Fix test for UpdateQuotaCommand (#848389) https://bugzilla.redhat.com/848289 Fix test for UpdateQuotaCommand Change-Id: Ie1dd3afb0a9e1915b9f6b8217d80110092c7cb20 Signed-off-by: Ofri Masad <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateQuotaCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/UpdateQuotaCommandTest.java 2 files changed, 3 insertions(+), 5 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/09/7909/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateQuotaCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateQuotaCommand.java index e03d560..8cc7b78 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateQuotaCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateQuotaCommand.java @@ -99,7 +99,7 @@ setQuota(quotaParameter); } - private void afterUpdate() { + protected void afterUpdate() { boolean newSizeUnderCurrentConsumption = QuotaManager.getInstance().isStorageQuotaExceeded(getQuota().getId()); diff --git a/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/UpdateQuotaCommandTest.java b/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/UpdateQuotaCommandTest.java index 9fa8305..7ef316b 100644 --- a/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/UpdateQuotaCommandTest.java +++ b/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/UpdateQuotaCommandTest.java @@ -3,10 +3,7 @@ import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertNotNull; import static org.junit.Assert.assertTrue; -import static org.mockito.Mockito.doNothing; -import static org.mockito.Mockito.doReturn; -import static org.mockito.Mockito.spy; -import static org.mockito.Mockito.verify; +import static org.mockito.Mockito.*; import java.util.ArrayList; import java.util.List; @@ -47,6 +44,7 @@ command = spy(new UpdateQuotaCommand(params)); doReturn(quotaDAO).when(command).getQuotaDAO(); doNothing().when(command).removeQuotaFromCache(); + doNothing().when(command).afterUpdate(); } private void setUpQuota() { -- To view, visit http://gerrit.ovirt.org/7909 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ie1dd3afb0a9e1915b9f6b8217d80110092c7cb20 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: ofri masad <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
