Merge pull request #4073 from sarkapalkovicova/fix_approve_applications_order

fix(core): approve applications order

Author: xflord

perun-registrar-lib/src/main/java/cz/metacentrum/perun/registrar/impl/RegistrarManagerImpl.java

@@ -1838,7 +1838,9 @@ public Application approveApplication(PerunSession sess, int appId) throws Perun
 	}
 
 	@Override
+	@Transactional(rollbackFor = Exception.class)
 	public void approveApplications(PerunSession sess, List<Integer> applicationIds) throws PerunException {
+		Collections.sort(applicationIds);
 		for (Integer id : applicationIds) {
 			approveApplication(sess, id);
 		}

perun-registrar-lib/src/test/java/cz/metacentrum/perun/registrar/RegistrarBaseIntegrationTest.java

@@ -1177,13 +1177,35 @@ public void testApproveApplications() throws PerunException {
 		registrarManager.submitApplication(session, application1, new ArrayList<>());
 		registrarManager.submitApplication(session, application2, new ArrayList<>());
 
-		registrarManager.approveApplications(session, List.of(application1.getId(), application2.getId()));
+		registrarManager.approveApplications(session, new ArrayList<>(Arrays.asList(application1.getId(), application2.getId())));
 
 		List<Integer> approvedAppIds = registrarManager.getApplicationsForVo(session, vo, List.of("APPROVED"), false).stream().map(Application::getId).toList();
 		assertEquals(2, approvedAppIds.size());
 		assertThat(approvedAppIds).containsOnly(application1.getId(), application2.getId());
 	}
 
+	@Test
+	public void testApproveApplicationsOrder() throws PerunException {
+		User user = new User(1, "User1", "Test1", "", "", "");
+		user = perun.getUsersManagerBl().createUser(session, user);
+
+		Group group = new Group("Test", "Test group");
+		perun.getGroupsManagerBl().createGroup(session, vo, group);
+		registrarManager.createApplicationFormInGroup(session, group);
+
+		Application applicationToVo = prepareApplicationToVo(user);
+		applicationToVo = registrarManager.submitApplication(session, applicationToVo, new ArrayList<>());
+
+		Application applicationToGroup = prepareApplicationToGroup(user, group);
+		applicationToGroup = registrarManager.submitApplication(session, applicationToGroup, new ArrayList<>());
+
+		registrarManager.approveApplications(session, new ArrayList<>(Arrays.asList(applicationToGroup.getId(), applicationToVo.getId())));
+
+		List<Integer> approvedAppIds = registrarManager.getApplicationsForVo(session, vo, List.of("APPROVED"), true).stream().map(Application::getId).toList();
+		assertEquals(2, approvedAppIds.size());
+		assertThat(approvedAppIds).containsOnly(applicationToVo.getId(), applicationToGroup.getId());
+	}
+
 	@Test
 	public void testEmbeddedGroupsSubmission_groupAutoApprove() throws PerunException {
 		GroupsManager groupsManager = perun.getGroupsManager();