You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

UserPermissionChangerIT.java 16KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346
  1. /*
  2. * SonarQube
  3. * Copyright (C) 2009-2024 SonarSource SA
  4. * mailto:info AT sonarsource DOT com
  5. *
  6. * This program is free software; you can redistribute it and/or
  7. * modify it under the terms of the GNU Lesser General Public
  8. * License as published by the Free Software Foundation; either
  9. * version 3 of the License, or (at your option) any later version.
  10. *
  11. * This program is distributed in the hope that it will be useful,
  12. * but WITHOUT ANY WARRANTY; without even the implied warranty of
  13. * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
  14. * Lesser General Public License for more details.
  15. *
  16. * You should have received a copy of the GNU Lesser General Public License
  17. * along with this program; if not, write to the Free Software Foundation,
  18. * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
  19. */
  20. package org.sonar.server.common.permission;
  21. import java.util.Set;
  22. import org.apache.commons.lang3.StringUtils;
  23. import org.junit.Before;
  24. import org.junit.Rule;
  25. import org.junit.Test;
  26. import org.sonar.api.resources.Qualifiers;
  27. import org.sonar.api.resources.ResourceTypes;
  28. import org.sonar.api.utils.System2;
  29. import org.sonar.api.web.UserRole;
  30. import org.sonar.core.util.SequenceUuidFactory;
  31. import org.sonar.db.DbSession;
  32. import org.sonar.db.DbTester;
  33. import org.sonar.db.component.ResourceTypesRule;
  34. import org.sonar.db.entity.EntityDto;
  35. import org.sonar.db.permission.GlobalPermission;
  36. import org.sonar.db.user.GroupDto;
  37. import org.sonar.db.user.UserDto;
  38. import org.sonar.db.user.UserIdDto;
  39. import org.sonar.server.exceptions.BadRequestException;
  40. import org.sonar.server.permission.PermissionService;
  41. import org.sonar.server.permission.PermissionServiceImpl;
  42. import static java.util.stream.Collectors.toSet;
  43. import static org.assertj.core.api.Assertions.assertThat;
  44. import static org.assertj.core.api.Assertions.assertThatThrownBy;
  45. import static org.sonar.server.common.permission.Operation.ADD;
  46. import static org.sonar.server.common.permission.Operation.REMOVE;
  47. import static org.sonar.server.permission.PermissionServiceImpl.ALL_PROJECT_PERMISSIONS;
  48. public class UserPermissionChangerIT {
  49. @Rule
  50. public DbTester db = DbTester.create(System2.INSTANCE);
  51. private final ResourceTypes resourceTypes = new ResourceTypesRule().setRootQualifiers(Qualifiers.PROJECT);
  52. private final PermissionService permissionService = new PermissionServiceImpl(resourceTypes);
  53. private final UserPermissionChanger underTest = new UserPermissionChanger(db.getDbClient(), new SequenceUuidFactory());
  54. private UserDto user1;
  55. private UserDto user2;
  56. private EntityDto privateProject;
  57. private EntityDto publicProject;
  58. @Before
  59. public void setUp() {
  60. user1 = db.users().insertUser();
  61. user2 = db.users().insertUser();
  62. privateProject = db.components().insertPrivateProject().getProjectDto();
  63. publicProject = db.components().insertPublicProject().getProjectDto();
  64. }
  65. @Test
  66. public void apply_adds_any_global_permission_to_user() {
  67. permissionService.getGlobalPermissions()
  68. .forEach(perm -> {
  69. UserPermissionChange change = new UserPermissionChange(ADD, perm.getKey(), null, UserIdDto.from(user1), permissionService);
  70. apply(change);
  71. assertThat(db.users().selectPermissionsOfUser(user1)).contains(perm);
  72. });
  73. }
  74. @Test
  75. public void apply_removes_any_global_permission_to_user() {
  76. // give ADMIN perm to user2 so that user1 is not the only one with this permission and it can be removed from user1
  77. db.users().insertGlobalPermissionOnUser(user2, GlobalPermission.ADMINISTER);
  78. permissionService.getGlobalPermissions()
  79. .forEach(perm -> db.users().insertGlobalPermissionOnUser(user1, perm));
  80. assertThat(db.users().selectPermissionsOfUser(user1))
  81. .containsOnly(permissionService.getGlobalPermissions().toArray(new GlobalPermission[0]));
  82. permissionService.getGlobalPermissions()
  83. .forEach(perm -> {
  84. UserPermissionChange change = new UserPermissionChange(REMOVE, perm.getKey(), null, UserIdDto.from(user1), permissionService);
  85. apply(change, permissionService.getGlobalPermissions().stream().map(GlobalPermission::getKey).collect(toSet()));
  86. assertThat(db.users().selectPermissionsOfUser(user1)).doesNotContain(perm);
  87. });
  88. }
  89. @Test
  90. public void apply_has_no_effect_when_adding_permission_USER_on_a_public_project() {
  91. UserPermissionChange change = new UserPermissionChange(ADD, UserRole.USER, publicProject, UserIdDto.from(user1), permissionService);
  92. apply(change);
  93. assertThat(db.users().selectEntityPermissionOfUser(user1, publicProject.getUuid())).doesNotContain(UserRole.USER);
  94. }
  95. @Test
  96. public void apply_has_no_effect_when_adding_permission_CODEVIEWER_on_a_public_project() {
  97. UserPermissionChange change = new UserPermissionChange(ADD, UserRole.CODEVIEWER, publicProject, UserIdDto.from(user1), permissionService);
  98. apply(change);
  99. assertThat(db.users().selectEntityPermissionOfUser(user1, publicProject.getUuid())).doesNotContain(UserRole.CODEVIEWER);
  100. }
  101. @Test
  102. public void apply_adds_permission_ADMIN_on_a_public_project() {
  103. applyAddsPermissionOnAPublicProject(UserRole.ADMIN);
  104. }
  105. @Test
  106. public void apply_adds_permission_ISSUE_ADMIN_on_a_public_project() {
  107. applyAddsPermissionOnAPublicProject(UserRole.ISSUE_ADMIN);
  108. }
  109. @Test
  110. public void apply_adds_permission_SCAN_EXECUTION_on_a_public_project() {
  111. applyAddsPermissionOnAPublicProject(GlobalPermission.SCAN.getKey());
  112. }
  113. private void applyAddsPermissionOnAPublicProject(String permission) {
  114. UserPermissionChange change = new UserPermissionChange(ADD, permission, publicProject, UserIdDto.from(user1), permissionService);
  115. apply(change);
  116. assertThat(db.users().selectEntityPermissionOfUser(user1, publicProject.getUuid())).containsOnly(permission);
  117. }
  118. @Test
  119. public void apply_fails_with_BadRequestException_when_removing_permission_USER_from_a_public_project() {
  120. UserPermissionChange change = new UserPermissionChange(REMOVE, UserRole.USER, publicProject, UserIdDto.from(user1), permissionService);
  121. assertThatThrownBy(() -> apply(change))
  122. .isInstanceOf(BadRequestException.class)
  123. .hasMessage("Permission user can't be removed from a public component");
  124. }
  125. @Test
  126. public void apply_fails_with_BadRequestException_when_removing_permission_CODEVIEWER_from_a_public_project() {
  127. UserPermissionChange change = new UserPermissionChange(REMOVE, UserRole.CODEVIEWER, publicProject, UserIdDto.from(user1), permissionService);
  128. assertThatThrownBy(() -> apply(change))
  129. .isInstanceOf(BadRequestException.class)
  130. .hasMessage("Permission codeviewer can't be removed from a public component");
  131. }
  132. @Test
  133. public void apply_removes_permission_ADMIN_from_a_public_project() {
  134. applyRemovesPermissionFromPublicProject(UserRole.ADMIN);
  135. }
  136. @Test
  137. public void apply_removes_permission_ISSUE_ADMIN_from_a_public_project() {
  138. applyRemovesPermissionFromPublicProject(UserRole.ISSUE_ADMIN);
  139. }
  140. @Test
  141. public void apply_removes_permission_SCAN_EXECUTION_from_a_public_project() {
  142. applyRemovesPermissionFromPublicProject(GlobalPermission.SCAN.getKey());
  143. }
  144. private void applyRemovesPermissionFromPublicProject(String permission) {
  145. db.users().insertProjectPermissionOnUser(user1, permission, publicProject);
  146. UserPermissionChange change = new UserPermissionChange(REMOVE, permission, publicProject, UserIdDto.from(user1), permissionService);
  147. apply(change, Set.of(permission));
  148. assertThat(db.users().selectEntityPermissionOfUser(user1, publicProject.getUuid())).isEmpty();
  149. }
  150. @Test
  151. public void apply_adds_any_permission_to_a_private_project() {
  152. permissionService.getAllProjectPermissions()
  153. .forEach(permission -> {
  154. UserPermissionChange change = new UserPermissionChange(ADD, permission, privateProject, UserIdDto.from(user1), permissionService);
  155. apply(change);
  156. assertThat(db.users().selectEntityPermissionOfUser(user1, privateProject.getUuid())).contains(permission);
  157. });
  158. }
  159. @Test
  160. public void apply_removes_any_permission_from_a_private_project() {
  161. permissionService.getAllProjectPermissions()
  162. .forEach(permission -> db.users().insertProjectPermissionOnUser(user1, permission, privateProject));
  163. permissionService.getAllProjectPermissions()
  164. .forEach(permission -> {
  165. UserPermissionChange change = new UserPermissionChange(REMOVE, permission, privateProject, UserIdDto.from(user1), permissionService);
  166. apply(change, ALL_PROJECT_PERMISSIONS);
  167. assertThat(db.users().selectEntityPermissionOfUser(user1, privateProject.getUuid())).doesNotContain(permission);
  168. });
  169. }
  170. @Test
  171. public void add_global_permission_to_user() {
  172. UserPermissionChange change = new UserPermissionChange(ADD, GlobalPermission.SCAN.getKey(), null, UserIdDto.from(user1), permissionService);
  173. apply(change);
  174. assertThat(db.users().selectPermissionsOfUser(user1)).containsOnly(GlobalPermission.SCAN);
  175. assertThat(db.users().selectEntityPermissionOfUser(user1, privateProject.getUuid())).isEmpty();
  176. assertThat(db.users().selectPermissionsOfUser(user2)).isEmpty();
  177. assertThat(db.users().selectEntityPermissionOfUser(user2, privateProject.getUuid())).isEmpty();
  178. }
  179. @Test
  180. public void add_project_permission_to_user() {
  181. UserPermissionChange change = new UserPermissionChange(ADD, UserRole.ISSUE_ADMIN, privateProject, UserIdDto.from(user1), permissionService);
  182. apply(change);
  183. assertThat(db.users().selectPermissionsOfUser(user1)).isEmpty();
  184. assertThat(db.users().selectEntityPermissionOfUser(user1, privateProject.getUuid())).contains(UserRole.ISSUE_ADMIN);
  185. assertThat(db.users().selectPermissionsOfUser(user2)).isEmpty();
  186. assertThat(db.users().selectEntityPermissionOfUser(user2, privateProject.getUuid())).isEmpty();
  187. }
  188. @Test
  189. public void do_nothing_when_adding_global_permission_that_already_exists() {
  190. db.users().insertGlobalPermissionOnUser(user1, GlobalPermission.ADMINISTER_QUALITY_GATES);
  191. UserPermissionChange change = new UserPermissionChange(ADD, GlobalPermission.ADMINISTER_QUALITY_GATES.getKey(), null, UserIdDto.from(user1), permissionService);
  192. apply(change);
  193. assertThat(db.users().selectPermissionsOfUser(user1)).containsOnly(GlobalPermission.ADMINISTER_QUALITY_GATES);
  194. }
  195. @Test
  196. public void fail_to_add_global_permission_on_project() {
  197. assertThatThrownBy(() -> {
  198. UserPermissionChange change = new UserPermissionChange(ADD, GlobalPermission.ADMINISTER_QUALITY_GATES.getKey(), privateProject, UserIdDto.from(user1), permissionService);
  199. apply(change);
  200. })
  201. .isInstanceOf(BadRequestException.class)
  202. .hasMessage("Invalid project permission 'gateadmin'. Valid values are [" + StringUtils.join(permissionService.getAllProjectPermissions(), ", ") + "]");
  203. }
  204. @Test
  205. public void fail_to_add_project_permission() {
  206. assertThatThrownBy(() -> {
  207. UserPermissionChange change = new UserPermissionChange(ADD, UserRole.ISSUE_ADMIN, null, UserIdDto.from(user1), permissionService);
  208. apply(change);
  209. })
  210. .isInstanceOf(BadRequestException.class)
  211. .hasMessage("Invalid global permission 'issueadmin'. Valid values are [admin, gateadmin, profileadmin, provisioning, scan]");
  212. }
  213. @Test
  214. public void remove_global_permission_from_user() {
  215. db.users().insertGlobalPermissionOnUser(user1, GlobalPermission.ADMINISTER_QUALITY_GATES);
  216. db.users().insertGlobalPermissionOnUser(user1, GlobalPermission.SCAN);
  217. db.users().insertGlobalPermissionOnUser(user2, GlobalPermission.ADMINISTER_QUALITY_GATES);
  218. db.users().insertProjectPermissionOnUser(user1, UserRole.ISSUE_ADMIN, privateProject);
  219. UserPermissionChange change = new UserPermissionChange(REMOVE, GlobalPermission.ADMINISTER_QUALITY_GATES.getKey(), null, UserIdDto.from(user1), permissionService);
  220. apply(change, Set.of(GlobalPermission.ADMINISTER_QUALITY_GATES.getKey(), GlobalPermission.SCAN.getKey(), UserRole.ISSUE_ADMIN));
  221. assertThat(db.users().selectPermissionsOfUser(user1)).containsOnly(GlobalPermission.SCAN);
  222. assertThat(db.users().selectPermissionsOfUser(user2)).containsOnly(GlobalPermission.ADMINISTER_QUALITY_GATES);
  223. assertThat(db.users().selectEntityPermissionOfUser(user1, privateProject.getUuid())).containsOnly(UserRole.ISSUE_ADMIN);
  224. }
  225. @Test
  226. public void remove_project_permission_from_user() {
  227. EntityDto project2 = db.components().insertPrivateProject().getProjectDto();
  228. db.users().insertGlobalPermissionOnUser(user1, GlobalPermission.ADMINISTER_QUALITY_GATES);
  229. db.users().insertProjectPermissionOnUser(user1, UserRole.ISSUE_ADMIN, privateProject);
  230. db.users().insertProjectPermissionOnUser(user1, UserRole.USER, privateProject);
  231. db.users().insertProjectPermissionOnUser(user2, UserRole.ISSUE_ADMIN, privateProject);
  232. db.users().insertProjectPermissionOnUser(user1, UserRole.ISSUE_ADMIN, project2);
  233. UserPermissionChange change = new UserPermissionChange(REMOVE, UserRole.ISSUE_ADMIN, privateProject, UserIdDto.from(user1), permissionService);
  234. apply(change, Set.of(GlobalPermission.ADMINISTER_QUALITY_GATES.getKey(), UserRole.ISSUE_ADMIN, UserRole.USER));
  235. assertThat(db.users().selectEntityPermissionOfUser(user1, privateProject.getUuid())).containsOnly(UserRole.USER);
  236. assertThat(db.users().selectEntityPermissionOfUser(user2, privateProject.getUuid())).containsOnly(UserRole.ISSUE_ADMIN);
  237. assertThat(db.users().selectEntityPermissionOfUser(user1, project2.getUuid())).containsOnly(UserRole.ISSUE_ADMIN);
  238. }
  239. @Test
  240. public void do_not_fail_if_removing_a_global_permission_that_does_not_exist() {
  241. UserPermissionChange change = new UserPermissionChange(REMOVE, GlobalPermission.ADMINISTER_QUALITY_GATES.getKey(), null, UserIdDto.from(user1), permissionService);
  242. apply(change);
  243. assertThat(db.users().selectPermissionsOfUser(user1)).isEmpty();
  244. }
  245. @Test
  246. public void do_not_fail_if_removing_a_project_permission_that_does_not_exist() {
  247. UserPermissionChange change = new UserPermissionChange(REMOVE, UserRole.ISSUE_ADMIN, privateProject, UserIdDto.from(user1), permissionService);
  248. apply(change);
  249. assertThat(db.users().selectEntityPermissionOfUser(user1, privateProject.getUuid())).isEmpty();
  250. }
  251. @Test
  252. public void fail_to_remove_admin_global_permission_if_no_more_admins() {
  253. db.users().insertGlobalPermissionOnUser(user1, GlobalPermission.ADMINISTER);
  254. UserPermissionChange change = new UserPermissionChange(REMOVE, GlobalPermission.ADMINISTER.getKey(), null, UserIdDto.from(user1), permissionService);
  255. DbSession session = db.getSession();
  256. Set<String> permissions = Set.of(GlobalPermission.ADMINISTER.getKey());
  257. assertThatThrownBy(() -> underTest.apply(session, permissions, change))
  258. .isInstanceOf(BadRequestException.class)
  259. .hasMessage("Last user with permission 'admin'. Permission cannot be removed.");
  260. }
  261. @Test
  262. public void remove_admin_user_if_still_other_admins() {
  263. db.users().insertGlobalPermissionOnUser(user1, GlobalPermission.ADMINISTER);
  264. GroupDto admins = db.users().insertGroup("admins");
  265. db.users().insertMember(admins, user2);
  266. db.users().insertPermissionOnGroup(admins, GlobalPermission.ADMINISTER);
  267. UserPermissionChange change = new UserPermissionChange(REMOVE, GlobalPermission.ADMINISTER.getKey(), null, UserIdDto.from(user1), permissionService);
  268. underTest.apply(db.getSession(), Set.of(GlobalPermission.ADMINISTER.getKey()), change);
  269. assertThat(db.users().selectPermissionsOfUser(user1)).isEmpty();
  270. }
  271. private void apply(UserPermissionChange change) {
  272. underTest.apply(db.getSession(), Set.of(), change);
  273. db.commit();
  274. }
  275. private void apply(UserPermissionChange change, Set<String> existingPermissions) {
  276. underTest.apply(db.getSession(), existingPermissions, change);
  277. db.commit();
  278. }
  279. }