[jackrabbit-oak] branch issues/oak-10374 updated: OAK-10374: test coverage for modification of non-versioned node with jcr:isCheckedOut==false property

[baedke]

This is an automated email from the ASF dual-hosted git repository.

baedke pushed a commit to branch issues/oak-10374
in repository https://gitbox.apache.org/repos/asf/jackrabbit-oak.git


The following commit(s) were added to refs/heads/issues/oak-10374 by this push:
     new 90afd7dc21 OAK-10374: test coverage for modification of non-versioned 
node with jcr:isCheckedOut==false property
90afd7dc21 is described below

commit 90afd7dc21ebd4c9fba3d77e795e7eded2937242
Author: Manfred Baedke <gre55...@adobe.com>
AuthorDate: Fri Aug 4 13:47:21 2023 +0200

    OAK-10374: test coverage for modification of non-versioned node with 
jcr:isCheckedOut==false property
---
 .../oak/jcr/version/VersionableTest.java           | 27 ++++++++++++++++++++++
 1 file changed, 27 insertions(+)

diff --git 
a/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/version/VersionableTest.java
 
b/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/version/VersionableTest.java
index 42177444a6..8fd7411fec 100644
--- 
a/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/version/VersionableTest.java
+++ 
b/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/version/VersionableTest.java
@@ -362,6 +362,33 @@ public class VersionableTest extends AbstractJCRTest {
         assertFalse(guest.getNode(nodePath).isCheckedOut());
     }
 
+    public void testNonVersionableCheckedOut() throws Exception {
+        Node node = testRootNode.addNode(nodeName1, "nt:unstructured");
+        superuser.save();
+
+        assertTrue(node.isCheckedOut());
+
+        node.setProperty("jcr:isCheckedOut", false);
+        superuser.save();
+
+        assertTrue(node.isCheckedOut());
+    }
+
+    public void testModifyNonVersionableNodeWithCheckedOutProperty() throws 
Exception {
+        Node node = testRootNode.addNode(nodeName1, "nt:unstructured");
+        superuser.save();
+
+        assertTrue(node.isCheckedOut());
+
+        node.setProperty("jcr:isCheckedOut", false);
+        superuser.save();
+
+        node.setProperty("test", true);
+        superuser.save();
+
+        assertTrue(node.hasProperty("test"));
+    }
+
     private static void assertSuccessors(VersionHistory history, Set<String> 
expectedSuccessors, String versionName) throws RepositoryException {
         assertEquals(expectedSuccessors, 
getNames(history.getVersion(versionName).getSuccessors()));
     }