Skip to content

Commit 34fb78b

Browse files
author
Julien Hervot de Mattos Vaz
committed
Fix QuotaResponseBuilderImplTest
1 parent 4d3ad22 commit 34fb78b

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

plugins/database/quota/src/test/java/org/apache/cloudstack/api/response/QuotaResponseBuilderImplTest.java

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -198,6 +198,9 @@ public void setup() {
198198
@Mock
199199
JsInterpreterHelper jsInterpreterHelperMock = Mockito.mock(JsInterpreterHelper.class);
200200

201+
@Mock
202+
QuotaSummaryCmd quotaSummaryCmdMock = Mockito.mock(QuotaSummaryCmd.class);
203+
201204
private QuotaTariffVO makeTariffTestData() {
202205
QuotaTariffVO tariffVO = new QuotaTariffVO();
203206
tariffVO.setUsageType(QuotaTypes.IP_ADDRESS);
@@ -766,16 +769,15 @@ public void getQuotaSummaryResponseWithListAllTestAccountNameIsNullAndDomainIdIs
766769
public void getQuotaSummaryResponseWithListAllTestAccountNameAndDomainIdAreNotNullPassDomainId() {
767770
Long expectedDomainId = 9837l;
768771

769-
QuotaSummaryCmd cmd = new QuotaSummaryCmd();
770-
cmd.setAccountName("test");
771-
cmd.setDomainId(expectedDomainId);
772+
Mockito.lenient().doReturn("test").when(quotaSummaryCmdMock).getAccountName();
773+
Mockito.doReturn(expectedDomainId).when(quotaSummaryCmdMock).getDomainId();
772774

773775
Mockito.doReturn(domainVoMock).when(domainDaoMock).findByIdIncludingRemoved(Mockito.anyLong());
774776

775777
Mockito.doReturn(null).when(quotaResponseBuilderSpy).getDomainPathByDomainIdForDomainAdmin(Mockito.any());
776778
Mockito.doReturn(quotaSummaryResponseMock1).when(quotaResponseBuilderSpy).getQuotaSummaryResponse(Mockito.any(), Mockito.any(), Mockito.any(), Mockito.any(), Mockito.any());
777779

778-
Pair<List<QuotaSummaryResponse>, Integer> result = quotaResponseBuilderSpy.getQuotaSummaryResponseWithListAll(cmd, accountMock);
780+
Pair<List<QuotaSummaryResponse>, Integer> result = quotaResponseBuilderSpy.getQuotaSummaryResponseWithListAll(quotaSummaryCmdMock, accountMock);
779781

780782
Assert.assertEquals(quotaSummaryResponseMock1, result);
781783
Mockito.verify(quotaResponseBuilderSpy).getQuotaSummaryResponse(Mockito.any(), Mockito.any(), Mockito.eq(expectedDomainId), Mockito.any(), Mockito.any());

0 commit comments

Comments
 (0)