(jackrabbit-oak) branch issue/oak-10624 updated: OAK-10624: o.a.j.o.namepath.JcrPathParser does not accept some valid local names containing "{" or "}".

[baedke]

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

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


The following commit(s) were added to refs/heads/issue/oak-10624 by this push:
     new 860afc2cbc OAK-10624: o.a.j.o.namepath.JcrPathParser does not accept 
some valid local names containing "{" or "}".
860afc2cbc is described below

commit 860afc2cbc6cd2e7d29c3545f1dd8ec50bbbb88e
Author: Manfred Baedke <manfred.bae...@gmail.com>
AuthorDate: Wed Jan 24 15:14:14 2024 +0100

    OAK-10624: o.a.j.o.namepath.JcrPathParser does not accept some valid local 
names containing "{" or "}".
    
    Corrected test case.
---
 .../oak/namepath/impl/NamePathMapperImplTest.java     | 19 +++++--------------
 1 file changed, 5 insertions(+), 14 deletions(-)

diff --git 
a/oak-core/src/test/java/org/apache/jackrabbit/oak/namepath/impl/NamePathMapperImplTest.java
 
b/oak-core/src/test/java/org/apache/jackrabbit/oak/namepath/impl/NamePathMapperImplTest.java
index a6c5698cab..e9c86f0726 100644
--- 
a/oak-core/src/test/java/org/apache/jackrabbit/oak/namepath/impl/NamePathMapperImplTest.java
+++ 
b/oak-core/src/test/java/org/apache/jackrabbit/oak/namepath/impl/NamePathMapperImplTest.java
@@ -210,6 +210,11 @@ public class NamePathMapperImplTest {
                 "/parent/sub/childB4",
                 "/parent/sub/}childB5",
                 "/parent/sub/{childB6}",
+                "/parent/sub/{childB7",
+                "/parent/sub/{childB7",
+                "/parent/{",
+                "/parent/{childA1",
+                "/parent/{{childA2"
         };
 
         for (String path : paths) {
@@ -217,20 +222,6 @@ public class NamePathMapperImplTest {
         }
     }
     
-    @Test
-    public void testIllegalBracketsInPaths() throws Exception {
-        String[] paths = {
-                "/parent/sub/{childB7", 
-                "/parent/sub/{childB7",
-                "/parent/{", 
-                "/parent/{childA1", 
-                "/parent/{{childA2"        };
-
-        for (String path : paths) {
-            assertNull(npMapper.getOakPath(path));
-        }
-    }    
-
     @Test
     public void testWhitespace() {
         String[] paths = new String[] {