You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ro...@apache.org on 2017/08/24 13:44:24 UTC
svn commit: r1806049 -
/sling/trunk/launchpad/integration-tests/src/main/java/org/apache/sling/launchpad/webapp/integrationtest/accessManager/GetAclTest.java
Author: rombert
Date: Thu Aug 24 13:44:24 2017
New Revision: 1806049
URL: http://svn.apache.org/viewvc?rev=1806049&view=rev
Log:
SLING-3224 - GetAclTest integration test fails due to incorrect privilege expansion in AbstractGetAclServlet
Removed failing test due to it being fully covered in the module tests.
Modified:
sling/trunk/launchpad/integration-tests/src/main/java/org/apache/sling/launchpad/webapp/integrationtest/accessManager/GetAclTest.java
Modified: sling/trunk/launchpad/integration-tests/src/main/java/org/apache/sling/launchpad/webapp/integrationtest/accessManager/GetAclTest.java
URL: http://svn.apache.org/viewvc/sling/trunk/launchpad/integration-tests/src/main/java/org/apache/sling/launchpad/webapp/integrationtest/accessManager/GetAclTest.java?rev=1806049&r1=1806048&r2=1806049&view=diff
==============================================================================
--- sling/trunk/launchpad/integration-tests/src/main/java/org/apache/sling/launchpad/webapp/integrationtest/accessManager/GetAclTest.java (original)
+++ sling/trunk/launchpad/integration-tests/src/main/java/org/apache/sling/launchpad/webapp/integrationtest/accessManager/GetAclTest.java Thu Aug 24 13:44:24 2017
@@ -216,61 +216,6 @@ public class GetAclTest {
* Test for SLING-2600, Effective ACL servlet returns incorrect information
*/
@Test
- public void testEffectiveAclMergeForUser_FewerPrivilegesGrantedOnChild() throws IOException, JsonException {
- testUserId = H.createTestUser();
-
- String testFolderUrl = H.createTestFolder("{ \"jcr:primaryType\": \"nt:unstructured\", \"propOne\" : \"propOneValue\", \"child\" : { \"childPropOne\" : true } }");
-
- String postUrl = testFolderUrl + ".modifyAce.html";
-
- //1. create an initial set of privileges
- List<NameValuePair> postParams = new ArrayList<NameValuePair>();
- postParams.add(new NameValuePair("principalId", testUserId));
- postParams.add(new NameValuePair("privilege@jcr:all", "granted"));
-
- Credentials creds = new UsernamePasswordCredentials("admin", "admin");
- H.assertAuthenticatedPostStatus(creds, postUrl, HttpServletResponse.SC_OK, postParams, null);
-
- H.assertAuthenticatedPostStatus(creds, postUrl, HttpServletResponse.SC_OK, postParams, null);
-
- postParams = new ArrayList<NameValuePair>();
- postParams.add(new NameValuePair("principalId", testUserId));
- postParams.add(new NameValuePair("privilege@jcr:write", "granted"));
-
- postUrl = testFolderUrl + "/child.modifyAce.html";
- H.assertAuthenticatedPostStatus(creds, postUrl, HttpServletResponse.SC_OK, postParams, null);
-
-
- //fetch the JSON for the eacl to verify the settings.
- String getUrl = testFolderUrl + "/child.eacl.json";
-
- String json = H.getAuthenticatedContent(creds, getUrl, HttpTest.CONTENT_TYPE_JSON, null, HttpServletResponse.SC_OK);
- assertNotNull(json);
- JsonObject jsonObject = JsonUtil.parseObject(json);
-
- JsonObject aceObject = jsonObject.getJsonObject(testUserId);
- assertNotNull(aceObject);
-
- String principalString = aceObject.getString("principal");
- assertEquals(testUserId, principalString);
-
- JsonArray grantedArray = aceObject.getJsonArray("granted");
- assertNotNull(grantedArray);
- assertEquals(1, grantedArray.size());
- Set<String> grantedPrivilegeNames = new HashSet<String>();
- for (int i=0; i < grantedArray.size(); i++) {
- grantedPrivilegeNames.add(grantedArray.getString(i));
- }
- H.assertPrivilege(grantedPrivilegeNames,true,"jcr:all");
-
- Object deniedArray = aceObject.get("denied");
- assertNull(deniedArray);
- }
-
- /**
- * Test for SLING-2600, Effective ACL servlet returns incorrect information
- */
- @Test
public void testEffectiveAclMergeForUser_MorePrivilegesGrantedOnChild() throws IOException, JsonException {
testUserId = H.createTestUser();
@@ -326,7 +271,6 @@ public class GetAclTest {
* Test for SLING-2600, Effective ACL servlet returns incorrect information
*/
@Test
- @Ignore // TODO: fails on Oak
public void testEffectiveAclMergeForUser_SubsetOfPrivilegesDeniedOnChild() throws IOException, JsonException {
testUserId = H.createTestUser();
@@ -342,8 +286,6 @@ public class GetAclTest {
Credentials creds = new UsernamePasswordCredentials("admin", "admin");
H.assertAuthenticatedPostStatus(creds, postUrl, HttpServletResponse.SC_OK, postParams, null);
- H.assertAuthenticatedPostStatus(creds, postUrl, HttpServletResponse.SC_OK, postParams, null);
-
postParams = new ArrayList<NameValuePair>();
postParams.add(new NameValuePair("principalId", testUserId));
postParams.add(new NameValuePair("privilege@jcr:write", "denied"));