SimpleReadWriteTests.testCreateDocument incorrect acl assertion?
----------------------------------------------------------------

                 Key: CMIS-142
                 URL: https://issues.apache.org/jira/browse/CMIS-142
             Project: Chemistry
          Issue Type: Bug
          Components: opencmis-provider
            Reporter: David Caruana


AbstractSimpleReadWriteTests (line 116):

    if (supportsManageACLs()) {
      AccessControlEntry ace = 
getObjectFactory().createAccessControlEntry(getUsername(),
          Collections.singletonList("cmis:read"));
      AccessControlList acl = getObjectFactory().createAccessControlList(
          Collections.singletonList(ace));

      AccessControlList newAcl = 
getProvider().getAclService().applyAcl(getTestRepositoryId(),
          docId, acl, null, getAclPropagation(), null);
      assertNotNull(newAcl);

This test case does not provide a removeAces value to applyAcl. Subsequently, 
in AbstractAtomPubService (line 508) the following check is made:

    if ((removeAces == null) || (removeAces.getAces() == null)) {
      throw new IllegalArgumentException("remove ACEs must not be null!");
    }

The above assertion always seem to fail for 
SimpleReadWriteTests.testCreateDocument.


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to