Skip to content

Commit 235efbe

Browse files
Surya SrinivasanSurya Srinivasan
authored andcommitted
Fix test mocks to use injected _projectAccountDao and _projectDao
1 parent c645292 commit 235efbe

File tree

1 file changed

+14
-3
lines changed

1 file changed

+14
-3
lines changed

server/src/test/java/com/cloud/user/AccountManagerImplTest.java

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,9 @@
8282
@RunWith(MockitoJUnitRunner.class)
8383
public class AccountManagerImplTest extends AccountManagetImplTestBase {
8484

85+
@InjectMocks
86+
private AccountManagerImpl accountManagerImpl;
87+
8588
@Mock
8689
private UserVmManagerImpl _vmMgr;
8790
@Mock
@@ -120,9 +123,16 @@ public class AccountManagerImplTest extends AccountManagetImplTestBase {
120123

121124
@Mock
122125
private ProjectAccountVO projectAccountVO;
126+
123127
@Mock
124128
private Project project;
125129

130+
@Mock
131+
private ProjectAccountDao _projectAccountDao;
132+
133+
@Mock
134+
private ProjectDao _projectDao;
135+
126136
@Mock
127137
PasswordPolicyImpl passwordPolicyMock;
128138

@@ -1598,16 +1608,17 @@ public void testCheckIfAccountManagesOnlyDeletedProjectsDoesNotThrow() {
15981608
long accountId = 42L;
15991609
long projectId = 100L;
16001610

1601-
Mockito.when(projectAccountDao.listAdministratedProjectIds(accountId))
1611+
Mockito.when(_projectAccountDao.listAdministratedProjectIds(accountId))
16021612
.thenReturn(List.of(projectId));
16031613

16041614
ProjectVO deletedProject = Mockito.mock(ProjectVO.class);
16051615
Mockito.when(deletedProject.getRemoved()).thenReturn(new Date());
16061616

1607-
Mockito.when(projectDao.findById(projectId))
1617+
Mockito.when(_projectDao.findById(projectId))
16081618
.thenReturn(deletedProject);
16091619

1610-
accountManager.checkIfAccountManagesProjects(accountId);
1620+
accountManagerImpl.checkIfAccountManagesProjects(accountId);
1621+
16111622
}
16121623

16131624
}

0 commit comments

Comments
 (0)