You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by en...@apache.org on 2020/10/19 19:14:16 UTC

[sling-org-apache-sling-launchpad-integration-tests] branch master updated: Fix invalid assumption after bugfix included in JCR Jackrabbit Access Manager 3.0.8

This is an automated email from the ASF dual-hosted git repository.

enorman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-launchpad-integration-tests.git


The following commit(s) were added to refs/heads/master by this push:
     new eba9b8d  Fix invalid assumption after bugfix included in JCR Jackrabbit Access Manager 3.0.8
eba9b8d is described below

commit eba9b8d77110d1da1b41bf379cb8f80923514677
Author: Eric Norman <en...@apache.org>
AuthorDate: Mon Oct 19 12:14:05 2020 -0700

    Fix invalid assumption after bugfix included in JCR Jackrabbit Access
    Manager 3.0.8
---
 .../webapp/integrationtest/userManager/UserPrivilegesInfoTest.java  | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/main/java/org/apache/sling/launchpad/webapp/integrationtest/userManager/UserPrivilegesInfoTest.java b/src/main/java/org/apache/sling/launchpad/webapp/integrationtest/userManager/UserPrivilegesInfoTest.java
index 05e7589..a484765 100644
--- a/src/main/java/org/apache/sling/launchpad/webapp/integrationtest/userManager/UserPrivilegesInfoTest.java
+++ b/src/main/java/org/apache/sling/launchpad/webapp/integrationtest/userManager/UserPrivilegesInfoTest.java
@@ -304,7 +304,7 @@ public class UserPrivilegesInfoTest {
 	public void testCanRemoveUser() throws IOException, JsonException {
 		testUserId = H.createTestUser();
 
-		//1. verify user can not remove themselves
+		//1. verify user can remove themselves as they have jcr:all permissions by default in the starter
 		String getUrl = HttpTest.HTTP_BASE_URL + "/system/userManager/user/" + testUserId + ".privileges-info.json";
 
 		//fetch the JSON for the test page to verify the settings.
@@ -314,8 +314,8 @@ public class UserPrivilegesInfoTest {
 		assertNotNull(json);
 		JsonObject jsonObj = JsonUtil.parseObject(json);
 		
-		//user can not remove themselves
-		assertEquals(false, jsonObj.getBoolean("canRemove"));
+		//user can remove themself
+		assertEquals(true, jsonObj.getBoolean("canRemove"));
 		
 		
 		//2. now try another user