You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by re...@apache.org on 2011/12/28 16:32:46 UTC

svn commit: r1225207 [1/3] - in /jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api: ./ lock/ observation/ query/ retention/ security/ version/

Author: reschke
Date: Wed Dec 28 15:32:44 2011
New Revision: 1225207

URL: http://svn.apache.org/viewvc?rev=1225207&view=rev
Log:
JCR-3196: reduce deprecation warning noise in TCK testsReduce warning noise

avoid Item.save() (work-in-progress)

Modified:
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/AbstractImportXmlTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/AbstractWorkspaceCopyTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/AddNodeTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NamespaceRegistryTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeAddMixinTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeCanAddMixinTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeItemIsModifiedTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeItemIsNewTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeRemoveMixinTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeSetPrimaryTypeTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/PropertyItemIsModifiedTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/PropertyItemIsNewTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/PropertyTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/ReferencesTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SerializationTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SessionRemoveItemTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SessionTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyAssumeTypeTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyBooleanTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyCalendarTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyConstraintViolationExceptionTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyDoubleTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyInputStreamTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyLongTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyNodeTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyStringTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyValueTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueBinaryTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueConstraintViolationExceptionTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueInputStreamTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueStringTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueValueFormatExceptionTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueVersionExceptionTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/ShareableNodeTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/ValueFactoryTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/WorkspaceCopySameNameSibsTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/WorkspaceCopyTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/WorkspaceMoveSameNameSibsTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/WorkspaceMoveTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/lock/AbstractLockTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/lock/DeepLockTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/lock/LockManagerTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/lock/LockTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/lock/SetValueLockExceptionTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/observation/AddEventListenerTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/observation/EventIteratorTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/observation/EventTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/observation/GetDateTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/observation/GetIdentifierTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/observation/GetInfoTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/observation/GetUserDataTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/observation/LockingTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/observation/NodeAddedTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/observation/NodeMovedTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/observation/NodeRemovedTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/observation/NodeReorderTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/observation/PropertyAddedTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/observation/PropertyChangedTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/observation/PropertyRemovedTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/observation/WorkspaceOperationTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/query/AbstractQueryLevel2Test.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/query/ElementTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/query/OrderByMultiTypeTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/query/QueryResultNodeIteratorTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/query/SQLJoinTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/query/SaveTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/query/TextNodeTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/retention/HoldTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/retention/RetentionPolicyTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/security/RSessionAccessControlTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/version/AbstractOnParentVersionTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/version/GetReferencesNodeTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/version/MergeSubNodeTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/version/OnParentVersionAbortTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/version/RemoveVersionTest.java

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/AbstractImportXmlTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/AbstractImportXmlTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/AbstractImportXmlTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/AbstractImportXmlTest.java Wed Dec 28 15:32:44 2011
@@ -358,7 +358,7 @@ abstract class AbstractImportXmlTest ext
         }
         n1.addMixin(mixReferenceable);
         // make sure jcr:uuid is available
-        testRootNode.save();
+        testRootNode.getSession().save();
         return n1.getUUID();
     }
 

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/AbstractWorkspaceCopyTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/AbstractWorkspaceCopyTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/AbstractWorkspaceCopyTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/AbstractWorkspaceCopyTest.java Wed Dec 28 15:32:44 2011
@@ -73,7 +73,7 @@ abstract class AbstractWorkspaceCopyTest
         // create a non-referenceable node
         try {
             node2 = testRootNode.addNode(nodeName2, testNodeType);
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (RepositoryException e) {
             fail("Failed to createtest node." + e.getMessage());
         }

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/AddNodeTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/AddNodeTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/AddNodeTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/AddNodeTest.java Wed Dec 28 15:32:44 2011
@@ -46,7 +46,7 @@ public class AddNodeTest extends Abstrac
      */
     public void testName() throws RepositoryException {
         Node n1 = testRootNode.addNode(nodeName1, testNodeType);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Wrong node name.", n1.getName(), nodeName1);
     }
 
@@ -55,7 +55,7 @@ public class AddNodeTest extends Abstrac
      */
     public void testNodeType() throws RepositoryException {
         Node n1 = testRootNode.addNode(nodeName1, testNodeType);
-        testRootNode.save();
+        testRootNode.getSession().save();
         String ntName = n1.getPrimaryNodeType().getName();
         assertEquals("Wrong node NodeType name.", testNodeType, ntName);
     }
@@ -68,7 +68,7 @@ public class AddNodeTest extends Abstrac
         if (testRootNode.getDefinition().allowsSameNameSiblings()) {
             Node n1 = testRootNode.addNode(nodeName1, testNodeType);
             Node n2 = testRootNode.addNode(nodeName1, testNodeType);
-            testRootNode.save();
+            testRootNode.getSession().save();
             assertEquals("Names of same name siblings are not equal.",
                     n1.getName(), n2.getName());
         } else {
@@ -143,7 +143,7 @@ public class AddNodeTest extends Abstrac
      */
     public void testPath() throws RepositoryException {
         Node n1 = testRootNode.addNode(nodeName1, testNodeType);
-        testRootNode.save();
+        testRootNode.getSession().save();
         String expected = testRootNode.getPath() + "/" + nodeName1;
         assertEquals("Wrong path for created node.", expected, n1.getPath());
     }

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NamespaceRegistryTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NamespaceRegistryTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NamespaceRegistryTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NamespaceRegistryTest.java Wed Dec 28 15:32:44 2011
@@ -142,18 +142,18 @@ public class NamespaceRegistryTest exten
 
         try {
             created = testRootNode.addNode(namespacePrefix + ":root");
-            testRootNode.save();
+            testRootNode.getSession().save();
         }
         catch (RepositoryException ex) {
             // that didn't work; maybe the repository allows a property here?
             testRootNode.getSession().refresh(false);
             created = testRootNode.setProperty(namespacePrefix + ":root", "test");
-            testRootNode.save();
+            testRootNode.getSession().save();
         }
 
         // Need to remove it here, otherwise teardown can't unregister the NS.
         testRootNode.getSession().getItem(created.getPath()).remove();
-        testRootNode.save();
+        testRootNode.getSession().save();
     }
 
     /**

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeAddMixinTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeAddMixinTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeAddMixinTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeAddMixinTest.java Wed Dec 28 15:32:44 2011
@@ -70,7 +70,7 @@ public class NodeAddMixinTest extends Ab
 
         // it is implementation-specific if a added mixin is available
         // before or after save therefore save before further tests
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // test if added mixin is available by node.getMixinNodeTypes()
         NodeType mixins[] = node.getMixinNodeTypes();
@@ -180,7 +180,7 @@ public class NodeAddMixinTest extends Ab
         Node node = testRootNode.addNode(nodeName1, testNodeType);
         // or try to make it lockable if it is not
         ensureMixinType(node, mixLockable);
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         String mixinName = NodeMixinUtil.getAddableMixinName(session, node);
         if (mixinName == null) {
@@ -235,7 +235,7 @@ public class NodeAddMixinTest extends Ab
         Node node = testRootNode.addNode(nodeName1, testNodeType);
         // or try to make it versionable if it is not
         ensureMixinType(node, mixVersionable);
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         String mixinName = NodeMixinUtil.getAddableMixinName(session, node);
         if (mixinName == null || node.isNodeType(mixinName)) {
@@ -267,7 +267,7 @@ public class NodeAddMixinTest extends Ab
         Node node = testRootNode.addNode(nodeName1, testNodeType);
         ensureMixinType(node, mixReferenceable);
         // implementation specific: mixin may take effect only upon save
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // check that it did
         assertTrue(node.isNodeType(mixReferenceable));

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeCanAddMixinTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeCanAddMixinTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeCanAddMixinTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeCanAddMixinTest.java Wed Dec 28 15:32:44 2011
@@ -55,7 +55,7 @@ public class NodeCanAddMixinTest extends
         Node node = testRootNode.addNode(nodeName1, testNodeType);
         // or try to make it lockable if it is not
         ensureMixinType(node, mixLockable);
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         String mixinName = NodeMixinUtil.getAddableMixinName(session, node);
         if (mixinName == null) {
@@ -99,7 +99,7 @@ public class NodeCanAddMixinTest extends
         Node node = testRootNode.addNode(nodeName1, testNodeType);
         // or try to make it versionable if it is not
         ensureMixinType(node, mixVersionable);
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         String mixinName = NodeMixinUtil.getAddableMixinName(session, node);
         if (mixinName == null) {

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeItemIsModifiedTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeItemIsModifiedTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeItemIsModifiedTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeItemIsModifiedTest.java Wed Dec 28 15:32:44 2011
@@ -63,7 +63,7 @@ public class NodeItemIsModifiedTest exte
      */
     public void testPersistentNewNodeItemIsModified () throws RepositoryException {
         Node testNode = testRootNode.addNode(nodeName1, testNodeType);
-        testRootNode.save();
+        testRootNode.getSession().save();
         Item testNodeItem = superuser.getItem(testNode.getPath());
         // check testNodeItem.isModified() for a new NodeItem after save
         assertFalse("Item.isModified() must return false after a new NodeItem is added and the parent Node is saved", testNodeItem.isModified());
@@ -78,7 +78,7 @@ public class NodeItemIsModifiedTest exte
      */
     public void testTransientNodeItemIsModified () throws RepositoryException {
         Node testNode = testRootNode.addNode(nodeName1, testNodeType);
-        testRootNode.save();
+        testRootNode.getSession().save();
         // modify the persistent testNode
         testNode.setProperty(propertyName1, "test");
         Item testNodeItem = superuser.getItem(testNode.getPath());
@@ -95,7 +95,7 @@ public class NodeItemIsModifiedTest exte
      */
     public void testPersistentNodeItemIsModified () throws RepositoryException {
         Node testNode = testRootNode.addNode(nodeName1, testNodeType);
-        testRootNode.save();
+        testRootNode.getSession().save();
         // modify the persistent testNode
         testNode.setProperty(propertyName1, "test");
         testNode.save();

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeItemIsNewTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeItemIsNewTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeItemIsNewTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeItemIsNewTest.java Wed Dec 28 15:32:44 2011
@@ -58,7 +58,7 @@ public class NodeItemIsNewTest extends A
      */
     public void testPersistentNodeItemIsNew () throws RepositoryException {
         Node testNode = testRootNode.addNode(nodeName1, testNodeType);
-        testRootNode.save();
+        testRootNode.getSession().save();
         Item testNodeItem = superuser.getItem(testNode.getPath());
         // check testNodeItem is new after save
         assertFalse("Item.isNew() must return false after a new NodeItem is added and the parent Node is saved", testNodeItem.isNew());

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeRemoveMixinTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeRemoveMixinTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeRemoveMixinTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeRemoveMixinTest.java Wed Dec 28 15:32:44 2011
@@ -57,7 +57,7 @@ public class NodeRemoveMixinTest extends
         }
 
         node.addMixin(mixinName);
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         try {
             node.removeMixin(mixinName);
@@ -86,7 +86,7 @@ public class NodeRemoveMixinTest extends
 
         // it is implementation-specific if a removed mixin isn't available
         // before or after save therefore save before further tests
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // test if removed mixin isn't available anymore by node.getMixinNodeTypes()
         assertTrue("removeMixin(String mixinName) did not remove mixin.",
@@ -110,7 +110,7 @@ public class NodeRemoveMixinTest extends
         }
 
         node.addMixin(mixinName);
-        testRootNode.save();
+        testRootNode.getSession().save();
 
 
         String notAssignedMixin = NodeMixinUtil.getNotAssignedMixinName(session, node);
@@ -150,7 +150,7 @@ public class NodeRemoveMixinTest extends
         Node node = testRootNode.addNode(nodeName1, testNodeType);
         // or try to make it lockable if it is not
         ensureMixinType(node, mixLockable);
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         String mixinName = NodeMixinUtil.getAddableMixinName(session, node);
         if (mixinName == null) {
@@ -158,7 +158,7 @@ public class NodeRemoveMixinTest extends
         }
 
         node.addMixin(mixinName);
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // remove first slash of path to get rel path to root
         String pathRelToRoot = node.getPath().substring(1);
@@ -208,7 +208,7 @@ public class NodeRemoveMixinTest extends
         Node node = testRootNode.addNode(nodeName1, testNodeType);
         // or try to make it versionable if it is not
         ensureMixinType(node, mixVersionable);
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         String mixinName = NodeMixinUtil.getAddableMixinName(session, node);
         if (mixinName == null || node.isNodeType(mixinName)) {
@@ -216,7 +216,7 @@ public class NodeRemoveMixinTest extends
         }
 
         node.addMixin(mixinName);
-        testRootNode.save();
+        testRootNode.getSession().save();
         node.checkin();
 
         try {

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeSetPrimaryTypeTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeSetPrimaryTypeTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeSetPrimaryTypeTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeSetPrimaryTypeTest.java Wed Dec 28 15:32:44 2011
@@ -212,7 +212,7 @@ public class NodeSetPrimaryTypeTest exte
         Node node = testRootNode.addNode(nodeName1, testNodeType);
         // or try to make it lockable if it is not
         ensureMixinType(node, mixLockable);
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         String primaryTypeName = getPrimaryTypeName(session, node);
         if (primaryTypeName == null) {

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/NodeTest.java Wed Dec 28 15:32:44 2011
@@ -658,7 +658,7 @@ public class NodeTest extends AbstractJC
         Node node = testRootNode.addNode(nodeName1, testNodeType);
         // or try to make it lockable if it is not
         ensureMixinType(node, mixLockable);
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // remove first slash of path to get rel path to root
         String pathRelToRoot = node.getPath().substring(1);
@@ -701,7 +701,7 @@ public class NodeTest extends AbstractJC
         ensureMixinType(node, mixLockable);
         // create a child node
         Node subNode = node.addNode(nodeName2, testNodeType);
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // lock the node
         // remove first slash of path to get rel path to root
@@ -780,7 +780,7 @@ public class NodeTest extends AbstractJC
 
         // create a node and save it
         Node testNode1 = testRootNode.addNode(nodeName1, testNodeType);
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // accuire the same node with a different session
         Session session = getHelper().getReadOnlySession();

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/PropertyItemIsModifiedTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/PropertyItemIsModifiedTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/PropertyItemIsModifiedTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/PropertyItemIsModifiedTest.java Wed Dec 28 15:32:44 2011
@@ -46,7 +46,7 @@ public class PropertyItemIsModifiedTest 
         // build persistent node
         try {
             testNode = testRootNode.addNode(nodeName1, testNodeType);
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (RepositoryException e) {
             fail("Failed to create test node." + e.getMessage());
         }

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/PropertyItemIsNewTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/PropertyItemIsNewTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/PropertyItemIsNewTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/PropertyItemIsNewTest.java Wed Dec 28 15:32:44 2011
@@ -46,7 +46,7 @@ public class PropertyItemIsNewTest exten
         // build persistent node
         try {
             testNode = testRootNode.addNode(nodeName1, testNodeType);
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (RepositoryException e) {
             fail("Failed to create test node." + e.getMessage());
         }

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/PropertyTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/PropertyTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/PropertyTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/PropertyTest.java Wed Dec 28 15:32:44 2011
@@ -55,7 +55,7 @@ public class PropertyTest extends Abstra
         // create a node, add a property and save it
         Node testNode1 = testRootNode.addNode(nodeName1, testNodeType);
         Property prop1 = testNode1.setProperty(propertyName1, "value1");
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // accuire the same property through a different session
         Session session = getHelper().getSuperuserSession();

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/ReferencesTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/ReferencesTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/ReferencesTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/ReferencesTest.java Wed Dec 28 15:32:44 2011
@@ -45,7 +45,7 @@ public class ReferencesTest extends Abst
         ensureMixinType(n1, mixReferenceable);
 
         // with some impls. the mixin type has only affect upon save
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // make sure the node is now referenceable
         assertTrue("test node should be mix:referenceable", n1.isNodeType(mixReferenceable));
@@ -60,7 +60,7 @@ public class ReferencesTest extends Abst
         ensureCanSetProperty(n2, propertyName1, values);
 
         Property p1 = n2.setProperty(propertyName1, values);
-        testRootNode.save();
+        testRootNode.getSession().save();
         PropertyIterator iter = n1.getReferences();
         if (iter.hasNext()) {
             assertEquals("Wrong referer", iter.nextProperty().getPath(), p1.getPath());
@@ -71,7 +71,7 @@ public class ReferencesTest extends Abst
         // create references: n3.p1 -> n1
         Node n3 = testRootNode.addNode(nodeName3, testNodeType);
         n3.setProperty(propertyName1, n1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         iter = n1.getReferences();
         while (iter.hasNext()) {
             Property p = iter.nextProperty();
@@ -92,7 +92,7 @@ public class ReferencesTest extends Abst
 
         // remove reference n3.p1 -> n1
         testRootNode.getNode(nodeName3).getProperty(propertyName1).remove();
-        testRootNode.save();
+        testRootNode.getSession().save();
         iter = n1.getReferences();
         if (iter.hasNext()) {
             assertEquals("Wrong referer", iter.nextProperty().getParent().getPath(), testRootNode.getNode(nodeName2).getPath());
@@ -105,7 +105,7 @@ public class ReferencesTest extends Abst
 
         // remove reference n2.p1 -> n1
         testRootNode.getNode(nodeName2).getProperty(propertyName1).setValue(new Value[0]);
-        testRootNode.save();
+        testRootNode.getSession().save();
         iter = n1.getReferences();
         if (iter.hasNext()) {
             fail("too many referers: " + iter.nextProperty().getPath());
@@ -120,7 +120,7 @@ public class ReferencesTest extends Abst
         ensureMixinType(n1, mixReferenceable);
 
         // with some impls. the mixin type has only affect upon save
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // make sure the node is now referenceable
         assertTrue("test node should be mix:referenceable", n1.isNodeType(mixReferenceable));
@@ -143,7 +143,7 @@ public class ReferencesTest extends Abst
         Property p1 = n2.setProperty(propertyName1, values);
         Property p2 = n2.setProperty(propertyName2, values);
         Property p3 = n3.setProperty(propertyName1, n1);
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // get references with name propertyName1
         // (should return p1 and p3))
@@ -168,7 +168,7 @@ public class ReferencesTest extends Abst
 
         // remove reference n3.p1 -> n1
         testRootNode.getNode(nodeName3).getProperty(propertyName1).remove();
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // get references with name propertyName1
         // (should return p1))
@@ -182,7 +182,7 @@ public class ReferencesTest extends Abst
 
         // remove reference n2.p1 -> n1
         p1.remove();
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // get references with name propertyName1
         // (should nothing))
@@ -202,7 +202,7 @@ public class ReferencesTest extends Abst
         ensureMixinType(n1, mixReferenceable);
 
         // with some impls. the mixin type has only affect upon save
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // make sure the node is now referenceable
         assertTrue("test node should be mix:referenceable", n1.isNodeType(mixReferenceable));
@@ -215,10 +215,10 @@ public class ReferencesTest extends Abst
         ensureCanSetProperty(n2, propertyName1, n2.getSession().getValueFactory().createValue(n1));
 
         n2.setProperty(propertyName1, n1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Wrong reference target.", n2.getProperty(propertyName1).getNode().getUUID(), n1.getUUID());
         n2.remove();
-        testRootNode.save();
+        testRootNode.getSession().save();
     }
 
     /**
@@ -231,7 +231,7 @@ public class ReferencesTest extends Abst
         ensureMixinType(n2, mixReferenceable);
 
         // with some impls. the mixin type has only affect upon save
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // make sure the nodes are now referenceable
         assertTrue("test node should be mix:referenceable", n1.isNodeType(mixReferenceable));
@@ -245,7 +245,7 @@ public class ReferencesTest extends Abst
         ensureCanSetProperty(n3, propertyName1, n3.getSession().getValueFactory().createValue(n1));
 
         n3.setProperty(propertyName1, n1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Wrong reference target.", n3.getProperty(propertyName1).getNode().getUUID(), n1.getUUID());
         PropertyIterator iter = n1.getReferences();
         if (iter.hasNext()) {

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SerializationTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SerializationTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SerializationTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SerializationTest.java Wed Dec 28 15:32:44 2011
@@ -208,7 +208,7 @@ public class SerializationTest extends A
             //A LockException is thrown if a lock prevents the addition of the subtree.
             Node lNode = testRootNode.addNode(nodeName1);
             ensureMixinType(lNode, mixLockable);
-            testRootNode.save();
+            testRootNode.getSession().save();
             Lock lock = lNode.lock(true, true);
             session.removeLockToken(lock.getLockToken());   //remove the token, so the lock is for me, too
             FileInputStream in = new FileInputStream(file);

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SessionRemoveItemTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SessionRemoveItemTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SessionRemoveItemTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SessionRemoveItemTest.java Wed Dec 28 15:32:44 2011
@@ -47,7 +47,7 @@ public class SessionRemoveItemTest exten
         readOnlySession = getHelper().getReadOnlySession();
 
         removeNode = testRootNode.addNode(nodeName1, testNodeType);
-        testRootNode.save();
+        testRootNode.getSession().save();
         nPath = removeNode.getPath();
     }
 

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SessionTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SessionTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SessionTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SessionTest.java Wed Dec 28 15:32:44 2011
@@ -264,7 +264,7 @@ public class SessionTest extends Abstrac
         // add a sub node (the one that is tried to move later on)
         Node srcNode = lockableNode.addNode(nodeName1, testNodeType);
 
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // remove first slash of path to get rel path to root
         String pathRelToRoot = lockableNode.getPath().substring(1);
@@ -278,7 +278,7 @@ public class SessionTest extends Abstrac
             try {
                 String destPath = testRoot + "/" + nodeName2;
                 session.move(srcNode.getPath(), destPath);
-                testRootNode.save();
+                testRootNode.getSession().save();
                 fail("A LockException is thrown either immediately or on save  if a lock prevents the move.");
             } catch (LockException e){
                 // success

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyAssumeTypeTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyAssumeTypeTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyAssumeTypeTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyAssumeTypeTest.java Wed Dec 28 15:32:44 2011
@@ -372,7 +372,7 @@ public class SetPropertyAssumeTypeTest e
         ensureMixinType(referenceableNode, mixReferenceable);
 
         // some implementations may require a save after addMixin()
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         Property prop = testNode.setProperty(testPropName, referenceableNode);
         assertEquals("setProperty(String, Value) of a property of type undefined " +
@@ -456,7 +456,7 @@ public class SetPropertyAssumeTypeTest e
             cal.setTime(new Date(0));
             Value v = superuser.getValueFactory().createValue(ISO8601.format(cal));
             testNode.setProperty(propertyName1, v, PropertyType.DATE);
-            testRootNode.save();
+            testRootNode.getSession().save();
             fail("Node.setProperty(String, Value, int) must throw a " +
                  "ConstraintViolationExcpetion if the type parameter and the " +
                  "type of the property do not match." );
@@ -479,7 +479,7 @@ public class SetPropertyAssumeTypeTest e
             Calendar cal = Calendar.getInstance();
             cal.setTime(new Date(0));
             testNode.setProperty(propertyName1, ISO8601.format(cal), PropertyType.DATE);
-            testRootNode.save();
+            testRootNode.getSession().save();
             fail("Node.setProperty(String, Value, int) must throw a " +
                  "ConstraintViolationExcpetion if the type parameter and the " +
                  "type of the property do not match." );
@@ -500,7 +500,7 @@ public class SetPropertyAssumeTypeTest e
 
         try {
             testNode.setProperty(propertyName1, stringValues, PropertyType.DATE);
-            testRootNode.save();
+            testRootNode.getSession().save();
             fail("Node.setProperty(String, Value, int) must throw a " +
                     "ConstraintViolationExcpetion or a ValueFormatException if " +
                     "the type parameter and the type of the property do not match.");

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyBooleanTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyBooleanTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyBooleanTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyBooleanTest.java Wed Dec 28 15:32:44 2011
@@ -76,7 +76,7 @@ public class SetPropertyBooleanTest exte
      */
     public void testNewBooleanPropertyParent() throws Exception {
         testNode.setProperty(propertyName1, true);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Setting property with Node.setProperty(String, boolean) and parentNode.save() not working",
                 true,
                 testNode.getProperty(propertyName1).getBoolean());
@@ -88,9 +88,9 @@ public class SetPropertyBooleanTest exte
      */
     public void testModifyBooleanPropertyParent() throws Exception {
         testNode.setProperty(propertyName1, true);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, false);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Modifying property with Node.setProperty(String, boolean) and parentNode.save() not working",
                 false,
                 testNode.getProperty(propertyName1).getBoolean());
@@ -117,9 +117,9 @@ public class SetPropertyBooleanTest exte
      */
     public void testRemoveBooleanPropertyParent() throws Exception {
         testNode.setProperty(propertyName1, true);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, (Value) null);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertFalse("Removing boolean property with Node.setProperty(String, null) and parentNode.save() not working",
                 testNode.hasProperty(propertyName1));
     }

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyCalendarTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyCalendarTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyCalendarTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyCalendarTest.java Wed Dec 28 15:32:44 2011
@@ -81,7 +81,7 @@ public class SetPropertyCalendarTest ext
      */
     public void testNewCalendarPropertyParent() throws Exception {
         testNode.setProperty(propertyName1, c1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Setting property with Node.setProperty(String, Calendar) and parentNode.save() not working",
                 vf.createValue(c1),
                 testNode.getProperty(propertyName1).getValue());
@@ -93,9 +93,9 @@ public class SetPropertyCalendarTest ext
      */
     public void testModifyCalendarPropertyParent() throws Exception {
         testNode.setProperty(propertyName1, c1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, c2);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Modifying property with Node.setProperty(String, Calendar) and parentNode.save() not working",
                 vf.createValue(c2),
                 testNode.getProperty(propertyName1).getValue());
@@ -122,9 +122,9 @@ public class SetPropertyCalendarTest ext
      */
     public void testRemoveCalendarPropertyParent() throws Exception {
         testNode.setProperty(propertyName1, c1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, (Calendar) null);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertFalse("Removing property with Node.setProperty(String, (Calendar)null) and parentNode.save() not working",
                 testNode.hasProperty(propertyName1));
     }

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyConstraintViolationExceptionTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyConstraintViolationExceptionTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyConstraintViolationExceptionTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyConstraintViolationExceptionTest.java Wed Dec 28 15:32:44 2011
@@ -75,7 +75,7 @@ public class SetPropertyConstraintViolat
         try {
             String nodeType = propDef.getDeclaringNodeType().getName();
             node = testRootNode.addNode(nodeName2, nodeType);
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (ConstraintViolationException e) {
             // implementation specific constraints do not allow to set up test environment
             throw new NotExecutableException("Not able to create required test items.");
@@ -134,7 +134,7 @@ public class SetPropertyConstraintViolat
         try {
             String nodeType = propDef.getDeclaringNodeType().getName();
             node = testRootNode.addNode(nodeName2, nodeType);
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (ConstraintViolationException e) {
             // implementation specific constraints do not allow to set up test environment
             throw new NotExecutableException("Not able to create required test items.");
@@ -193,7 +193,7 @@ public class SetPropertyConstraintViolat
         try {
             String nodeType = propDef.getDeclaringNodeType().getName();
             node = testRootNode.addNode(nodeName2, nodeType);
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (ConstraintViolationException e) {
             // implementation specific constraints do not allow to set up test environment
             throw new NotExecutableException("Not able to create required test items.");
@@ -253,7 +253,7 @@ public class SetPropertyConstraintViolat
         try {
             String nodeType = propDef.getDeclaringNodeType().getName();
             node = testRootNode.addNode(nodeName2, nodeType);
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (ConstraintViolationException e) {
             // implementation specific constraints do not allow to set up test environment
             throw new NotExecutableException("Not able to create required test items.");
@@ -315,7 +315,7 @@ public class SetPropertyConstraintViolat
         try {
             String nodeType = propDef.getDeclaringNodeType().getName();
             node = testRootNode.addNode(nodeName2, nodeType);
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (ConstraintViolationException e) {
             // implementation specific constraints do not allow to set up test environment
             throw new NotExecutableException("Not able to create required test items.");
@@ -398,7 +398,7 @@ public class SetPropertyConstraintViolat
             nodeNotSatisfied = testRootNode.addNode(nodeName4, nodeTypeNotSatisfied);
             ensureMixinType(nodeNotSatisfied, mixReferenceable);
 
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (ConstraintViolationException e) {
             // implementation specific constraints do not allow to set up test environment
             throw new NotExecutableException("Not able to create required test items.");

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyDoubleTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyDoubleTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyDoubleTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyDoubleTest.java Wed Dec 28 15:32:44 2011
@@ -83,7 +83,7 @@ public class SetPropertyDoubleTest exten
      */
     public void testNewDoublePropertyParent() throws Exception {
         testNode.setProperty(propertyName1, d1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Setting property with Node.setProperty(String, double) and parentNode.save() not working",
                 new Double(d1),
                 new Double(testNode.getProperty(propertyName1).getDouble()));
@@ -95,9 +95,9 @@ public class SetPropertyDoubleTest exten
      */
     public void testModifyDoublePropertyParent() throws Exception {
         testNode.setProperty(propertyName1, d1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, d2);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Modifying property with Node.setProperty(String, double) and parentNode.save() not working",
                 new Double(d2),
                 new Double(testNode.getProperty(propertyName1).getDouble()));
@@ -124,9 +124,9 @@ public class SetPropertyDoubleTest exten
      */
     public void testRemoveDoublePropertyParent() throws Exception {
         testNode.setProperty(propertyName1, d1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, (Value) null);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertFalse("Removing double property with Node.setProperty(String, null) and parentNode.save() not working",
                 testNode.hasProperty(propertyName1));
     }

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyInputStreamTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyInputStreamTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyInputStreamTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyInputStreamTest.java Wed Dec 28 15:32:44 2011
@@ -48,7 +48,7 @@ public class SetPropertyInputStreamTest 
     protected void setUp() throws Exception {
         super.setUp();
         testNode = testRootNode.addNode(nodeName1, testNodeType);
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // special case for repositories that do allow binary property
         // values, but only on jcr:content/jcr:data
@@ -105,7 +105,7 @@ public class SetPropertyInputStreamTest 
      */
     public void testNewInputStreamPropertyParent() throws Exception {
         testNode.setProperty(propertyName1, is1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         is1 = new ByteArrayInputStream(bytes1);
         InputStream in = testNode.getProperty(propertyName1).getStream();
         try {
@@ -122,9 +122,9 @@ public class SetPropertyInputStreamTest 
      */
     public void testModifyInputStreamPropertyParent() throws Exception {
         testNode.setProperty(propertyName1, is1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, is2);
-        testRootNode.save();
+        testRootNode.getSession().save();
         is2 = new ByteArrayInputStream(bytes2);
         InputStream in = testNode.getProperty(propertyName1).getStream();
         try {
@@ -162,7 +162,7 @@ public class SetPropertyInputStreamTest 
      */
     public void testRemoveInputStreamPropertyParent() throws Exception {
         testNode.setProperty(propertyName1, is1);
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         Property property = testNode.getProperty(propertyName1);
         if (property.getDefinition().isMandatory() || property.getDefinition().isProtected()) {
@@ -170,7 +170,7 @@ public class SetPropertyInputStreamTest 
         }
 
         testNode.setProperty(propertyName1, (InputStream) null);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertFalse("Removing property with Node.setProperty(String, (InputStream)null) and parentNode.save() not working",
                 testNode.hasProperty(propertyName1));
     }

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyLongTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyLongTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyLongTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyLongTest.java Wed Dec 28 15:32:44 2011
@@ -79,7 +79,7 @@ public class SetPropertyLongTest extends
      */
     public void testNewLongPropertyParent() throws Exception {
         testNode.setProperty(propertyName1, l1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Setting property with Node.setProperty(String, long) and parentNode.save() not working",
                 new Long(l1),
                 new Long(testNode.getProperty(propertyName1).getLong()));
@@ -91,9 +91,9 @@ public class SetPropertyLongTest extends
      */
     public void testModifyLongPropertyParent() throws Exception {
         testNode.setProperty(propertyName1, l1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, l2);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Modifying property with Node.setProperty(String, long) and parentNode.save() not working",
                 new Long(l2),
                 new Long(testNode.getProperty(propertyName1).getLong()));
@@ -120,9 +120,9 @@ public class SetPropertyLongTest extends
      */
     public void testRemoveLongPropertyParent() throws Exception {
         testNode.setProperty(propertyName1, l1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, (Value) null);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertFalse("Removing long property with Node.setProperty(String, null) and parentNode.save() not working",
                 testNode.hasProperty(propertyName1));
     }

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyNodeTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyNodeTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyNodeTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyNodeTest.java Wed Dec 28 15:32:44 2011
@@ -50,7 +50,7 @@ public class SetPropertyNodeTest extends
         => call save in order to make sure, that the uuid is created and contains
            a value generated by the system.
         */
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // abort test if the repository does not allow setting
         // reference properties on this node
@@ -103,7 +103,7 @@ public class SetPropertyNodeTest extends
      */
     public void testNewNodePropertyParent() throws Exception {
         testNode.setProperty(propertyName1, n1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Setting property with Node.setProperty(String, Node) and parentNode.save() not working",
                 n1.getUUID(),
                 testNode.getProperty(propertyName1).getString());
@@ -115,9 +115,9 @@ public class SetPropertyNodeTest extends
      */
     public void testModifyNodePropertyParent() throws Exception {
         testNode.setProperty(propertyName1, n1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, n2);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Modifying property with Node.setProperty(String, Node) and parentNode.save() not working",
                 n2.getUUID(),
                 testNode.getProperty(propertyName1).getString());
@@ -144,9 +144,9 @@ public class SetPropertyNodeTest extends
      */
     public void testRemoveNodePropertyParent() throws Exception {
         testNode.setProperty(propertyName1, n1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, (Node) null);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertFalse("Removing property with Node.setProperty(String, (Node)null) and parentNode.save() not working",
                 testNode.hasProperty(propertyName1));
     }

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyStringTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyStringTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyStringTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyStringTest.java Wed Dec 28 15:32:44 2011
@@ -120,7 +120,7 @@ public class SetPropertyStringTest exten
      */
     public void testNewStringPropertyParent() throws Exception {
         testNode.setProperty(propertyName1, s1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Setting property with Node.setProperty(String, String) and parentNode.save() not working",
                 s1,
                 testNode.getProperty(propertyName1).getString());
@@ -132,9 +132,9 @@ public class SetPropertyStringTest exten
      */
     public void testModifyStringPropertyParent() throws Exception {
         testNode.setProperty(propertyName1, s1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, s2);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Modifying property with Node.setProperty(String, String) and parentNode.save() not working",
                 s2,
                 testNode.getProperty(propertyName1).getString());
@@ -161,9 +161,9 @@ public class SetPropertyStringTest exten
      */
     public void testRemoveStringPropertyParent() throws Exception {
         testNode.setProperty(propertyName1, s1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, (String) null);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertFalse("Removing property with Node.setProperty(String, (String)null) and parentNode.save() not working",
                 testNode.hasProperty(propertyName1));
     }
@@ -203,7 +203,7 @@ public class SetPropertyStringTest exten
      */
     public void testNewStringPropertyParentWithPropertyType() throws Exception {
         testNode.setProperty(propertyName1, s1, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Setting property with Node.setProperty(String, String, int) and parentNode.save() not working",
                 s1,
                 testNode.getProperty(propertyName1).getString());
@@ -215,9 +215,9 @@ public class SetPropertyStringTest exten
      */
     public void testModifyStringPropertyParentWithPropertyType() throws Exception {
         testNode.setProperty(propertyName1, s1, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, s2, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Modifying property with Node.setProperty(String, String, int) and parentNode.save() not working",
                 s2,
                 testNode.getProperty(propertyName1).getString());
@@ -244,9 +244,9 @@ public class SetPropertyStringTest exten
      */
     public void testRemoveStringPropertyParentWithPropertyType() throws Exception {
         testNode.setProperty(propertyName1, s1, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, (String) null, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertFalse("Removing property with Node.setProperty(String, (String)null, int) and parentNode.save() not working",
                 testNode.hasProperty(propertyName1));
     }
@@ -286,7 +286,7 @@ public class SetPropertyStringTest exten
      */
     public void testNewStringArrayPropertyParent() throws Exception {
         testNode.setProperty(propertyName2, sArray1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Setting properties with Node.setProperty(String, String[]) and parentNode.save() not working",
                 Arrays.asList(vArray1),
                 Arrays.asList(testNode.getProperty(propertyName2).getValues()));
@@ -298,9 +298,9 @@ public class SetPropertyStringTest exten
      */
     public void testModifyStringArrayPropertyParent() throws Exception {
         testNode.setProperty(propertyName2, sArray1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName2, sArray2);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Modifying properties with Node.setProperty(String, String[]) and parentNode.save() not working",
                 Arrays.asList(vArray2),
                 Arrays.asList(testNode.getProperty(propertyName2).getValues()));
@@ -346,9 +346,9 @@ public class SetPropertyStringTest exten
      */
     public void testRemoveStringArrayPropertyParent() throws Exception {
         testNode.setProperty(propertyName2, sArray1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName2, (String[]) null);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertFalse("Removing property with Node.setProperty(String, (String[])null) and parentNode.save() not working",
                 testNode.hasProperty(propertyName2));
     }
@@ -400,7 +400,7 @@ public class SetPropertyStringTest exten
      */
     public void testNewStringArrayPropertyParentWithPropertyType() throws Exception {
         testNode.setProperty(propertyName2, sArray1, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Setting properties with Node.setProperty(String, String[], int) and parentNode.save() not working",
                 Arrays.asList(vArray1),
                 Arrays.asList(testNode.getProperty(propertyName2).getValues()));
@@ -412,9 +412,9 @@ public class SetPropertyStringTest exten
      */
     public void testModifyStringArrayPropertyParentWithPropertyType() throws Exception {
         testNode.setProperty(propertyName2, sArray1, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName2, sArray2, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Modifying properties with Node.setProperty(String, String[], int) and parentNode.save() not working",
                 Arrays.asList(vArray2),
                 Arrays.asList(testNode.getProperty(propertyName2).getValues()));
@@ -460,9 +460,9 @@ public class SetPropertyStringTest exten
      */
     public void testRemoveStringArrayPropertyParentWithPropertyType() throws Exception {
         testNode.setProperty(propertyName2, sArray1, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName2, (String[]) null, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertFalse("Removing property with Node.setProperty(String, (String[])null, int) and parentNode.save() not working",
                 testNode.hasProperty(propertyName2));
     }

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyValueTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyValueTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyValueTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyValueTest.java Wed Dec 28 15:32:44 2011
@@ -130,7 +130,7 @@ public class SetPropertyValueTest extend
      */
     public void testNewValuePropertyParent() throws Exception {
         testNode.setProperty(propertyName1, v1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Setting property with Node.setProperty(String, Value) and parentNode.save() not working",
                 v1,
                 testNode.getProperty(propertyName1).getValue());
@@ -142,9 +142,9 @@ public class SetPropertyValueTest extend
      */
     public void testModifyValuePropertyParent() throws Exception {
         testNode.setProperty(propertyName1, v1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, v2);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Modifying property with Node.setProperty(String, Value) and parentNode.save() not working",
                 v2,
                 testNode.getProperty(propertyName1).getValue());
@@ -171,9 +171,9 @@ public class SetPropertyValueTest extend
      */
     public void testRemoveValuePropertyParent() throws Exception {
         testNode.setProperty(propertyName1, v1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, (Value) null);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertFalse("Removing property with Node.setProperty(String, (Value)null) and parentNode.save() not working",
                 testNode.hasProperty(propertyName1));
     }
@@ -214,7 +214,7 @@ public class SetPropertyValueTest extend
      */
     public void testNewValuePropertyParentWithPropertyType() throws Exception {
         testNode.setProperty(propertyName1, v1, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Setting property with Node.setProperty(String, Value, int) and parentNode.save() not working",
                 v1,
                 testNode.getProperty(propertyName1).getValue());
@@ -226,9 +226,9 @@ public class SetPropertyValueTest extend
      */
     public void testModifyValuePropertyParentWithPropertyType() throws Exception {
         testNode.setProperty(propertyName1, v1, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, v2, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Modifying property with Node.setProperty(String, Value, int) and parentNode.save() not working",
                 v2,
                 testNode.getProperty(propertyName1).getValue());
@@ -255,9 +255,9 @@ public class SetPropertyValueTest extend
      */
     public void testRemoveValuePropertyParentWithPropertyType() throws Exception {
         testNode.setProperty(propertyName1, v1, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName1, (Value) null, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertFalse("Removing property with Node.setProperty(String, (Value)null, int) and parentNode.save() not working",
                 testNode.hasProperty(propertyName1));
     }
@@ -298,7 +298,7 @@ public class SetPropertyValueTest extend
      */
     public void testNewValueArrayPropertyParent() throws Exception {
         testNode.setProperty(propertyName2, vArray1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Setting properties with Node.setProperty(String, Value[]) and parentNode.save() not working",
                 Arrays.asList(vArray1),
                 Arrays.asList(testNode.getProperty(propertyName2).getValues()));
@@ -310,9 +310,9 @@ public class SetPropertyValueTest extend
      */
     public void testModifyValueArrayPropertyParent() throws Exception {
         testNode.setProperty(propertyName2, vArray1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName2, vArray2);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Modifying properties with Node.setProperty(String, Value[]) and parentNode.save() not working",
                 Arrays.asList(vArray2),
                 Arrays.asList(testNode.getProperty(propertyName2).getValues()));
@@ -371,9 +371,9 @@ public class SetPropertyValueTest extend
      */
     public void testRemoveValueArrayPropertyParent() throws Exception {
         testNode.setProperty(propertyName2, vArray1);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName2, (Value[]) null);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertFalse("Removing property with Node.setProperty(String, (Value[])null) and parentNode.save() not working",
                 testNode.hasProperty(propertyName2));
     }
@@ -438,7 +438,7 @@ public class SetPropertyValueTest extend
      */
     public void testNewValueArrayPropertyParentWithPropertyType() throws Exception {
         testNode.setProperty(propertyName2, vArray1, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Setting properties with Node.setProperty(String, Value[], int) and parentNode.save() not working",
                 Arrays.asList(vArray1),
                 Arrays.asList(testNode.getProperty(propertyName2).getValues()));
@@ -450,9 +450,9 @@ public class SetPropertyValueTest extend
      */
     public void testModifyValueArrayPropertyParentWithPropertyType() throws Exception {
         testNode.setProperty(propertyName2, vArray1, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName2, vArray2, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Modifying properties with Node.setProperty(String, Value[], int) and parentNode.save() not working",
                 Arrays.asList(vArray2),
                 Arrays.asList(testNode.getProperty(propertyName2).getValues()));
@@ -511,9 +511,9 @@ public class SetPropertyValueTest extend
      */
     public void testRemoveValueArrayPropertyParentWithPropertyType() throws Exception {
         testNode.setProperty(propertyName2, vArray1, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         testNode.setProperty(propertyName2, (Value[]) null, PropertyType.STRING);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertFalse("Removing property with Node.setProperty(String, (Value[])null, int) and parentNode.save() not working",
                 testNode.hasProperty(propertyName2));
     }

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueBinaryTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueBinaryTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueBinaryTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueBinaryTest.java Wed Dec 28 15:32:44 2011
@@ -78,7 +78,7 @@ public class SetValueBinaryTest extends 
 
         // create a new node under the testRootNode
         node = testRootNode.addNode(nodeName1, testNodeType);
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // special case for repositories that do allow binary property
         // values, but only on jcr:content/jcr:data

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueConstraintViolationExceptionTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueConstraintViolationExceptionTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueConstraintViolationExceptionTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueConstraintViolationExceptionTest.java Wed Dec 28 15:32:44 2011
@@ -85,7 +85,7 @@ public class SetValueConstraintViolation
             String nodeType = propDef.getDeclaringNodeType().getName();
             node = testRootNode.addNode(nodeName2, nodeType);
             prop = node.setProperty(propDef.getName(), valueSatisfied);
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (ConstraintViolationException e) {
             // implementation specific constraints do not allow to set up test environment
             throw new NotExecutableException("Not able to create required test items.");
@@ -155,7 +155,7 @@ public class SetValueConstraintViolation
             String nodeType = propDef.getDeclaringNodeType().getName();
             node = testRootNode.addNode(nodeName2, nodeType);
             prop = node.setProperty(propDef.getName(), valueSatisfied);
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (ConstraintViolationException e) {
             // implementation specific constraints do not allow to set up test environment
             throw new NotExecutableException("Not able to create required test items.");
@@ -222,7 +222,7 @@ public class SetValueConstraintViolation
             String nodeType = propDef.getDeclaringNodeType().getName();
             node = testRootNode.addNode(nodeName2, nodeType);
             prop = node.setProperty(propDef.getName(), valueSatisfied);
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (ConstraintViolationException e) {
             // implementation specific constraints do not allow to set up test environment
             throw new NotExecutableException("Not able to create required test items.");
@@ -289,7 +289,7 @@ public class SetValueConstraintViolation
             String nodeType = propDef.getDeclaringNodeType().getName();
             node = testRootNode.addNode(nodeName2, nodeType);
             prop = node.setProperty(propDef.getName(), valueSatisfied);
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (ConstraintViolationException e) {
             // implementation specific constraints do not allow to set up test environment
             throw new NotExecutableException("Not able to create required test items.");
@@ -356,7 +356,7 @@ public class SetValueConstraintViolation
             String nodeType = propDef.getDeclaringNodeType().getName();
             node = testRootNode.addNode(nodeName2, nodeType);
             prop = node.setProperty(propDef.getName(), valueSatisfied);
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (ConstraintViolationException e) {
             // implementation specific constraints do not allow to set up test environment
             throw new NotExecutableException("Not able to create required test items.");
@@ -448,10 +448,10 @@ public class SetValueConstraintViolation
             ensureMixinType(nodeNotSatisfied, mixReferenceable);
 
             // some implementations may require a save after addMixin()
-            testRootNode.save();
+            testRootNode.getSession().save();
 
             prop = node.setProperty(propDef.getName(), nodeSatisfied);
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (ConstraintViolationException e) {
             // implementation specific constraints do not allow to set up test environment
             throw new NotExecutableException("Not able to create required test items.");
@@ -518,7 +518,7 @@ public class SetValueConstraintViolation
             String nodeType = propDef.getDeclaringNodeType().getName();
             node = testRootNode.addNode(nodeName2, nodeType);
             prop = node.setProperty(propDef.getName(), new Value[]{valueSatisfied});
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (ConstraintViolationException e) {
             // implementation specific constraints do not allow to set up test environment
             throw new NotExecutableException("Not able to create required test items.");
@@ -573,7 +573,7 @@ public class SetValueConstraintViolation
             String nodeType = propDef.getDeclaringNodeType().getName();
             node = testRootNode.addNode(nodeName2, nodeType);
             prop = node.setProperty(propDef.getName(), new Value[]{valueSatisfied});
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (ConstraintViolationException e) {
             // implementation specific constraints do not allow to set up test environment
             throw new NotExecutableException("Not able to create required test items.");
@@ -629,7 +629,7 @@ public class SetValueConstraintViolation
             String nodeType = propDef.getDeclaringNodeType().getName();
             node = testRootNode.addNode(nodeName2, nodeType);
             prop = node.setProperty(propDef.getName(), new Value[]{valueSatisfied});
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (ConstraintViolationException e) {
             // implementation specific constraints do not allow to set up test environment
             throw new NotExecutableException("Not able to create required test items.");
@@ -684,7 +684,7 @@ public class SetValueConstraintViolation
             String nodeType = propDef.getDeclaringNodeType().getName();
             node = testRootNode.addNode(nodeName2, nodeType);
             prop = node.setProperty(propDef.getName(), new Value[]{valueSatisfied});
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (ConstraintViolationException e) {
             // implementation specific constraints do not allow to set up test environment
             throw new NotExecutableException("Not able to create required test items.");
@@ -740,7 +740,7 @@ public class SetValueConstraintViolation
             String nodeType = propDef.getDeclaringNodeType().getName();
             node = testRootNode.addNode(nodeName2, nodeType);
             prop = node.setProperty(propDef.getName(), new Value[]{valueSatisfied});
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (ConstraintViolationException e) {
             // implementation specific constraints do not allow to set up test environment
             throw new NotExecutableException("Not able to create required test items.");
@@ -820,11 +820,11 @@ public class SetValueConstraintViolation
             ensureMixinType(nodeNotSatisfied, mixReferenceable);
 
             // some implementations may require a save after addMixin()
-            testRootNode.save();
+            testRootNode.getSession().save();
 
             Value valueSatisfied = superuser.getValueFactory().createValue(nodeSatisfied);
             prop = node.setProperty(propDef.getName(), new Value[]{valueSatisfied});
-            testRootNode.save();
+            testRootNode.getSession().save();
         } catch (ConstraintViolationException e) {
             // implementation specific constraints do not allow to set up test environment
             throw new NotExecutableException("Not able to create required test items.");

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueInputStreamTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueInputStreamTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueInputStreamTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueInputStreamTest.java Wed Dec 28 15:32:44 2011
@@ -68,7 +68,7 @@ public class SetValueInputStreamTest ext
 
         // create a new node under the testRootNode
         node = testRootNode.addNode(nodeName1, testNodeType);
-        testRootNode.save();
+        testRootNode.getSession().save();
 
         // special case for repositories that do allow binary property
         // values, but only on jcr:content/jcr:data

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueStringTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueStringTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueStringTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueStringTest.java Wed Dec 28 15:32:44 2011
@@ -106,7 +106,7 @@ public class SetValueStringTest extends 
      */
     public void testValueParent() throws RepositoryException {
         property1.setValue(sv2);
-        testRootNode.save();
+        testRootNode.getSession().save();
         assertEquals("Value node property not saved", sv2, property1.getValue());
     }
 

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueValueFormatExceptionTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueValueFormatExceptionTest.java?rev=1225207&r1=1225206&r2=1225207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueValueFormatExceptionTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetValueValueFormatExceptionTest.java Wed Dec 28 15:32:44 2011
@@ -217,7 +217,7 @@ public class SetValueValueFormatExceptio
             }
 
             // some implementations may require a save after addMixin()
-            testRootNode.save();
+            testRootNode.getSession().save();
 
             // make sure the node is now referenceable
             assertTrue("test node should be mix:referenceable", referenceableNode.isNodeType(mixReferenceable));
@@ -271,7 +271,7 @@ public class SetValueValueFormatExceptio
             } else {
                 n.addMixin(mixReferenceable);
                 // some implementations may require a save after addMixin()
-                testRootNode.save();
+                testRootNode.getSession().save();
             }
         }