| // Licensed to the Apache Software Foundation (ASF) under one | |
| // or more contributor license agreements. See the NOTICE file | |
| // distributed with this work for additional information | |
| // regarding copyright ownership. The ASF licenses this file | |
| // to you under the Apache License, Version 2.0 (the | |
| // "License"); you may not use this file except in compliance | |
| // with the License. You may obtain a copy of the License at | |
| // | |
| // http://www.apache.org/licenses/LICENSE-2.0 | |
| // | |
| // Unless required by applicable law or agreed to in writing, | |
| // software distributed under the License is distributed on an | |
| // "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY | |
| // KIND, either express or implied. See the License for the | |
| // specific language governing permissions and limitations | |
| // under the License. | |
| package com.cloud.projects; | |
| import java.util.ArrayList; | |
| import java.util.Collections; | |
| import java.util.List; | |
| import org.apache.cloudstack.acl.ControlledEntity; | |
| import org.apache.cloudstack.reservation.dao.ReservationDao; | |
| import org.apache.cloudstack.webhook.WebhookHelper; | |
| import org.apache.commons.collections.CollectionUtils; | |
| import org.junit.Assert; | |
| import org.junit.Before; | |
| import org.junit.Test; | |
| import org.junit.runner.RunWith; | |
| import org.mockito.InjectMocks; | |
| import org.mockito.Mock; | |
| import org.mockito.MockedConstruction; | |
| import org.mockito.MockedStatic; | |
| import org.mockito.Mockito; | |
| import org.mockito.Spy; | |
| import org.mockito.junit.MockitoJUnitRunner; | |
| import org.mockito.stubbing.Answer; | |
| import org.springframework.beans.factory.NoSuchBeanDefinitionException; | |
| import com.cloud.configuration.Resource.ResourceType; | |
| import com.cloud.exception.ResourceAllocationException; | |
| import com.cloud.projects.ProjectAccount.Role; | |
| import com.cloud.projects.dao.ProjectAccountDao; | |
| import com.cloud.projects.dao.ProjectDao; | |
| import com.cloud.projects.dao.ProjectInvitationDao; | |
| import com.cloud.resourcelimit.CheckedReservation; | |
| import com.cloud.user.AccountVO; | |
| import com.cloud.user.ResourceLimitService; | |
| import com.cloud.user.User; | |
| import com.cloud.user.dao.AccountDao; | |
| import com.cloud.utils.component.ComponentContext; | |
| @RunWith(MockitoJUnitRunner.class) | |
| public class ProjectManagerImplTest { | |
| @Spy | |
| @InjectMocks | |
| ProjectManagerImpl projectManager; | |
| @Mock | |
| ProjectDao projectDao; | |
| @Mock | |
| ProjectInvitationDao projectInvitationDao; | |
| @Mock | |
| ProjectAccountDao projectAccountDao; | |
| @Mock | |
| AccountDao accountDao; | |
| @Mock | |
| ResourceLimitService resourceLimitMgr; | |
| @Mock | |
| ReservationDao reservationDao; | |
| List<ProjectVO> updateProjects; | |
| @Before | |
| public void setUp() throws Exception { | |
| updateProjects = new ArrayList<>(); | |
| Mockito.when(projectDao.update(Mockito.anyLong(), Mockito.any(ProjectVO.class))).thenAnswer((Answer<Boolean>) invocation -> { | |
| ProjectVO project = (ProjectVO)invocation.getArguments()[1]; | |
| updateProjects.add(project); | |
| return true; | |
| }); | |
| } | |
| private void runUpdateProjectNameAndDisplayTextTest(boolean nonNullName, boolean nonNullDisplayText) { | |
| ProjectVO projectVO = new ProjectVO(); | |
| String newName = nonNullName ? "NewName" : null; | |
| String newDisplayText = nonNullDisplayText ? "NewDisplayText" : null; | |
| projectManager.updateProjectNameAndDisplayText(projectVO, newName, newDisplayText); | |
| if (!nonNullName && !nonNullDisplayText) { | |
| Assert.assertTrue(updateProjects.isEmpty()); | |
| } else { | |
| Assert.assertFalse(updateProjects.isEmpty()); | |
| Assert.assertEquals(1, updateProjects.size()); | |
| ProjectVO updatedProject = updateProjects.get(0); | |
| Assert.assertNotNull(updatedProject); | |
| if (nonNullName) { | |
| Assert.assertEquals(newName, updatedProject.getName()); | |
| } | |
| if (nonNullDisplayText) { | |
| Assert.assertEquals(newDisplayText, updatedProject.getDisplayText()); | |
| } | |
| } | |
| } | |
| @Test | |
| public void testUpdateProjectNameAndDisplayTextNoUpdate() { | |
| runUpdateProjectNameAndDisplayTextTest(false, false); | |
| } | |
| @Test | |
| public void testUpdateProjectNameAndDisplayTextUpdateName() { | |
| runUpdateProjectNameAndDisplayTextTest(true, false); | |
| } | |
| @Test | |
| public void testUpdateProjectNameAndDisplayTextUpdateDisplayText() { | |
| runUpdateProjectNameAndDisplayTextTest(false, true); | |
| } | |
| @Test | |
| public void testUpdateProjectNameAndDisplayTextUpdateNameDisplayText() { | |
| runUpdateProjectNameAndDisplayTextTest(true, true); | |
| } | |
| @Test | |
| public void testDeleteWebhooksForAccount() { | |
| try (MockedStatic<ComponentContext> mockedComponentContext = Mockito.mockStatic(ComponentContext.class)) { | |
| WebhookHelper webhookHelper = Mockito.mock(WebhookHelper.class); | |
| List<ControlledEntity> webhooks = List.of(Mockito.mock(ControlledEntity.class), | |
| Mockito.mock(ControlledEntity.class)); | |
| Mockito.doReturn(webhooks).when(webhookHelper).listWebhooksByAccount(Mockito.anyLong()); | |
| mockedComponentContext.when(() -> ComponentContext.getDelegateComponentOfType(WebhookHelper.class)) | |
| .thenReturn(webhookHelper); | |
| Project project = Mockito.mock(Project.class); | |
| Mockito.when(project.getProjectAccountId()).thenReturn(1L); | |
| List<? extends ControlledEntity> result = projectManager.listWebhooksForProject(project); | |
| Assert.assertEquals(2, result.size()); | |
| } | |
| } | |
| @Test | |
| public void testDeleteWebhooksForAccountNoBean() { | |
| try (MockedStatic<ComponentContext> mockedComponentContext = Mockito.mockStatic(ComponentContext.class)) { | |
| mockedComponentContext.when(() -> ComponentContext.getDelegateComponentOfType(WebhookHelper.class)) | |
| .thenThrow(NoSuchBeanDefinitionException.class); | |
| List<? extends ControlledEntity> result = | |
| projectManager.listWebhooksForProject(Mockito.mock(Project.class)); | |
| Assert.assertTrue(CollectionUtils.isEmpty(result)); | |
| } | |
| } | |
| @Test | |
| public void cleanupProjectsForUserTestNoAssociationsReturnsFalse() { | |
| Project project = Mockito.mock(Project.class); | |
| Mockito.when(project.getId()).thenReturn(100L); | |
| User user = mockUser(1L, 10L); | |
| Mockito.when(projectAccountDao.listBy(Mockito.anyLong(), Mockito.anyLong(), Mockito.anyLong())) | |
| .thenReturn(Collections.emptyList()); | |
| boolean result = projectManager.cleanupProjectsForUser(project, user); | |
| Assert.assertFalse(result); | |
| Mockito.verify(projectInvitationDao).removeBy(100L, 10L, 1L); | |
| Mockito.verify(projectAccountDao, Mockito.never()).remove(Mockito.anyLong()); | |
| Mockito.verify(resourceLimitMgr, Mockito.never()).decrementResourceCount(Mockito.anyLong(), Mockito.any(ResourceType.class)); | |
| } | |
| @Test | |
| public void cleanupProjectsForUserTestRemovesAdminAndRegularAssociations() { | |
| Project project = Mockito.mock(Project.class); | |
| Mockito.when(project.getId()).thenReturn(100L); | |
| User user = mockUser(1L, 10L); | |
| ProjectAccountVO admin = mockProjectAccount(1L, Role.Admin); | |
| ProjectAccountVO regular = mockProjectAccount(2L, Role.Regular); | |
| Mockito.when(projectAccountDao.listBy(Mockito.anyLong(), Mockito.anyLong(), Mockito.anyLong())) | |
| .thenReturn(List.of(admin, regular)); | |
| boolean result = projectManager.cleanupProjectsForUser(project, user); | |
| Assert.assertTrue(result); | |
| Mockito.verify(projectInvitationDao).removeBy(100L, 10L, 1L); | |
| Mockito.verify(projectAccountDao).remove(1L); | |
| Mockito.verify(projectAccountDao).remove(2L); | |
| Mockito.verify(resourceLimitMgr).decrementResourceCount(10L, ResourceType.project); | |
| } | |
| @Test | |
| public void cleanupProjectsForUserTestNullProject() { | |
| User user = mockUser(1L, 10L); | |
| ProjectAccountVO admin = mockProjectAccount(1L, Role.Admin); | |
| Mockito.when(projectAccountDao.listBy(Mockito.isNull(), Mockito.eq(10L), Mockito.eq(1L))) | |
| .thenReturn(List.of(admin)); | |
| boolean result = projectManager.cleanupProjectsForUser(null, user); | |
| Assert.assertTrue(result); | |
| Mockito.verify(projectInvitationDao).removeBy(Mockito.isNull(), Mockito.eq(10L), Mockito.eq(1L)); | |
| Mockito.verify(projectAccountDao).remove(1L); | |
| Mockito.verify(resourceLimitMgr).decrementResourceCount(10L, ResourceType.project); | |
| } | |
| @Test | |
| public void moveProjectAssociationsToUserTestNoProjectAccounts() throws ResourceAllocationException { | |
| User oldUser = mockUser(1L, 10L); | |
| User newUser = mockUser(2L, 20L); | |
| Mockito.when(projectAccountDao.listBy(Mockito.isNull(), Mockito.eq(10L), Mockito.eq(1L))) | |
| .thenReturn(Collections.emptyList()); | |
| projectManager.moveProjectAssociationsToUser(oldUser, newUser); | |
| Mockito.verify(projectInvitationDao).move(oldUser, newUser); | |
| Mockito.verify(projectAccountDao, Mockito.never()).move(Mockito.any(), Mockito.any()); | |
| Mockito.verifyNoInteractions(accountDao); | |
| Mockito.verifyNoInteractions(resourceLimitMgr); | |
| } | |
| @Test | |
| public void moveProjectAssociationsToUserTestSameAccount() throws ResourceAllocationException { | |
| User oldUser = mockUser(1L, 10L); | |
| User newUser = mockUser(2L, 10L); | |
| ProjectAccountVO regular = mockProjectAccount(1L, Role.Regular); | |
| Mockito.when(projectAccountDao.listBy(Mockito.isNull(), Mockito.eq(10L), Mockito.eq(1L))) | |
| .thenReturn(List.of(regular)); | |
| AccountVO oldAccount = mockAccount(10L); | |
| AccountVO newAccount = mockAccount(10L); | |
| Mockito.when(accountDao.findById(10L)).thenReturn(oldAccount).thenReturn(newAccount); | |
| try (MockedConstruction<CheckedReservation> ignored = Mockito.mockConstruction(CheckedReservation.class)) { | |
| projectManager.moveProjectAssociationsToUser(oldUser, newUser); | |
| } | |
| Mockito.verify(projectInvitationDao).move(oldUser, newUser); | |
| Mockito.verify(projectAccountDao).move(oldUser, newUser); | |
| Mockito.verify(resourceLimitMgr, Mockito.never()) | |
| .incrementResourceCount(Mockito.anyLong(), Mockito.any(ResourceType.class), Mockito.anyLong()); | |
| Mockito.verify(resourceLimitMgr, Mockito.never()) | |
| .decrementResourceCount(Mockito.anyLong(), Mockito.any(ResourceType.class), Mockito.anyLong()); | |
| } | |
| @Test | |
| public void moveProjectAssociationsToUserTestDifferentAccountsWithAdminRole() throws ResourceAllocationException { | |
| User oldUser = mockUser(1L, 10L); | |
| User newUser = mockUser(2L, 20L); | |
| ProjectAccountVO admin = mockProjectAccount(1L, Role.Admin); | |
| Mockito.when(projectAccountDao.listBy(Mockito.isNull(), Mockito.eq(10L), Mockito.eq(1L))) | |
| .thenReturn(List.of(admin)); | |
| AccountVO oldAccount = mockAccount(10L); | |
| AccountVO newAccount = mockAccount(20L); | |
| Mockito.when(accountDao.findById(10L)).thenReturn(oldAccount); | |
| Mockito.when(accountDao.findById(20L)).thenReturn(newAccount); | |
| try (MockedConstruction<CheckedReservation> ignored = Mockito.mockConstruction(CheckedReservation.class)) { | |
| projectManager.moveProjectAssociationsToUser(oldUser, newUser); | |
| } | |
| Mockito.verify(projectInvitationDao).move(oldUser, newUser); | |
| Mockito.verify(projectAccountDao).move(oldUser, newUser); | |
| Mockito.verify(resourceLimitMgr).incrementResourceCount(20L, ResourceType.project, 1L); | |
| Mockito.verify(resourceLimitMgr).decrementResourceCount(10L, ResourceType.project, 1L); | |
| } | |
| @Test | |
| public void moveProjectAssociationsToUserTestDifferentAccountsWithoutAdminRole() throws ResourceAllocationException { | |
| User oldUser = mockUser(1L, 10L); | |
| User newUser = mockUser(2L, 20L); | |
| ProjectAccountVO regular = mockProjectAccount(1L, Role.Regular); | |
| Mockito.when(projectAccountDao.listBy(Mockito.isNull(), Mockito.eq(10L), Mockito.eq(1L))) | |
| .thenReturn(List.of(regular)); | |
| AccountVO oldAccount = mockAccount(10L); | |
| AccountVO newAccount = mockAccount(20L); | |
| Mockito.when(accountDao.findById(10L)).thenReturn(oldAccount); | |
| Mockito.when(accountDao.findById(20L)).thenReturn(newAccount); | |
| try (MockedConstruction<CheckedReservation> ignored = Mockito.mockConstruction(CheckedReservation.class)) { | |
| projectManager.moveProjectAssociationsToUser(oldUser, newUser); | |
| } | |
| Mockito.verify(projectInvitationDao).move(oldUser, newUser); | |
| Mockito.verify(projectAccountDao).move(oldUser, newUser); | |
| Mockito.verify(resourceLimitMgr, Mockito.never()) | |
| .incrementResourceCount(Mockito.anyLong(), Mockito.any(ResourceType.class), Mockito.anyLong()); | |
| Mockito.verify(resourceLimitMgr, Mockito.never()) | |
| .decrementResourceCount(Mockito.anyLong(), Mockito.any(ResourceType.class), Mockito.anyLong()); | |
| } | |
| private User mockUser(long id, long accountId) { | |
| User user = Mockito.mock(User.class); | |
| Mockito.when(user.getId()).thenReturn(id); | |
| Mockito.when(user.getAccountId()).thenReturn(accountId); | |
| return user; | |
| } | |
| private AccountVO mockAccount(long id) { | |
| AccountVO account = Mockito.mock(AccountVO.class); | |
| Mockito.when(account.getId()).thenReturn(id); | |
| return account; | |
| } | |
| private ProjectAccountVO mockProjectAccount(long id, Role role) { | |
| ProjectAccountVO projectAccount = Mockito.mock(ProjectAccountVO.class); | |
| Mockito.when(projectAccount.getId()).thenReturn(id); | |
| Mockito.when(projectAccount.getAccountRole()).thenReturn(role); | |
| return projectAccount; | |
| } | |
| } |