[JCR-2756] Shareable node test failures

• Type: Bug
• Status: Patch Available
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 2.0, 2.1, 2.1.1, 2.2.13, 2.4.3, 2.5.2
• Component/s: jackrabbit-core, jackrabbit-jcr-tests
• Assignee: Unassigned
• Reporter: Jukka Zitting
• Created: 24/Sep/10 12:32
• Updated: 07/Oct/12 01:58

Description

There's a problem in the ShareableNodeTest class that makes it always throw NotExcecutableExceptions due to a null argument being passed to ensureKnowsNodeType(). The following patch fixes this problem:

— a/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/ShareableNodeTest.java
+++ b/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/ShareableNodeTest.java
@@ -45,8 +45,6 @@ import org.apache.jackrabbit.test.NotExecutableException;
*/
public class ShareableNodeTest extends AbstractJCRTest {

• private String mixShareable;
  -
  protected void setUp() throws Exception {
  super.setUp();
  try {
  @@ -56,7 +54,6 @@ public class ShareableNodeTest extends AbstractJCRTest { cleanUp(); throw e; }
• mixShareable = superuser.getNamespacePrefix(NS_MIX_URI) + ":shareable";
  }

protected void tearDown() throws Exception {

Once this patch is applied, we get the following test failures:

testGetName(org.apache.jackrabbit.test.api.ShareableNodeTest)
testGetNode(org.apache.jackrabbit.test.api.ShareableNodeTest)
testGetNodes(org.apache.jackrabbit.test.api.ShareableNodeTest)
testGetNodesByPattern(org.apache.jackrabbit.test.api.ShareableNodeTest)

The problem seems to be caused by the ItemManager already having the non-shared version of a node cached when the mix:shareable mixin is added.