[ 
https://issues.apache.org/jira/browse/OAK-343?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13562802#comment-13562802
 ] 

Alex Parvulescu commented on OAK-343:
-------------------------------------

yes, I fixed it again :)
The diff mechanism is not obvious sometimes, (this time I forgot to include the 
new nodes).

Remaining failing auth related tests:
{code}
testFindAuthorizableByRelativePath(org.apache.jackrabbit.oak.jcr.security.user.FindAuthorizablesTest):
 expected result
  
testCreateGroupWithExistingPrincipal2(org.apache.jackrabbit.oak.jcr.security.user.UserManagerTest):
 Principal testPrincipal_dabdf591-e632-4d8a-ba20-387d7171f419 is already in use 
-> must throw AuthorizableExistsException.
  
testCreateGroupWithExistingPrincipal3(org.apache.jackrabbit.oak.jcr.security.user.UserManagerTest):
 Principal testPrincipal_6f98fd44-854c-47e4-b1a1-e0a2f7808a18 is already in use 
-> must throw AuthorizableExistsException.
  testCyclicGroups(org.apache.jackrabbit.oak.jcr.security.user.GroupTest)
{code}
                
> Session.getNodeByUUID requires save call
> ----------------------------------------
>
>                 Key: OAK-343
>                 URL: https://issues.apache.org/jira/browse/OAK-343
>             Project: Jackrabbit Oak
>          Issue Type: Bug
>          Components: core, jcr
>            Reporter: angela
>            Assignee: Thomas Mueller
>         Attachments: oak-343-QueryIndex.patch, OAK-343-v2.patch
>
>
> while adding mix:referenceable to a new node immediately assigns a 
> uuid and makes the node referenceable, session.getNodeByUUID only works
> after saving the changes.
> the following test illustrates this behavior:
> @Test
> public void getNodeByUUID() throws RepositoryException {
>    Node node = getNode("/foo").addNode("boo");
>    node.addMixin(JcrConstants.MIX_REFERENCEABLE);
>    assertTrue(node.isNodeType(JcrConstants.MIX_REFERENCEABLE));
>    String uuid = node.getUUID();
>    assertNotNull(uuid);
>    assertEquals(uuid, node.getIdentifier());
>    Node nAgain = node.getSession().getNodeByUUID(uuid);
>    assertTrue(nAgain.isSame(node));
>    assertTrue(nAgain.isSame(node.getSession().getNodeByIdentifier(uuid)));
> }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to