Fix getParentPath with content name like Test(1)

packages/sn-client-utils/src/path-helper.ts

@@ -24,7 +24,7 @@ export class PathHelper {
    * @param path The path to be splitted
    */
   public static getSegments(path: string) {
-    if (!path) {
+    if (!path || path.startsWith('content') || path.startsWith('/content')) {
       return []
     }
     // Split path at / and remove empty strings
@@ -34,7 +34,7 @@ export class PathHelper {
       throw new Error(`Couldn't get the segments for ${path}`)
     }
     // Match if last item is Root/content(123) or Root/Example('content')
-    const matches = lastItem.match(/(\('\w+'\)|\(\d+\)$)/g)
+    const matches = lastItem.match(/(\('\w+'\)$)/g)
     if (!matches) {
       return [...splitted, lastItem]
     }
@@ -44,11 +44,11 @@ export class PathHelper {
   }
 
   /**
-   * Checks if a specific segment is an Item segment or not (like "('Content1')"" or "(654)")
+   * Checks if a specific segment is an Item segment or not (like "('Content1')" or "Test(1)")
    * @param segment The segment to be examined
    */
   public static isItemSegment(segment: string): boolean {
-    return RegExp(/\('+[\s\S]+'\)$/).test(segment) || RegExp(/\(+\d+\)$/).test(segment)
+    return RegExp(/\('+[\s\S]+'\)$/).test(segment) || RegExp(/\(+\w+\d+\)$/).test(segment)
   }
 
   /**

packages/sn-client-utils/test/path-helper.test.ts

@@ -173,11 +173,14 @@ export const pathHelperTests = describe('PathHelper', () => {
       expect(PathHelper.getSegments("/Root/Example('Content1')")).toEqual(['Root', 'Example', "('Content1')"])
     })
     it('Should split the path to segments', () => {
-      expect(PathHelper.getSegments('/Root/Example(123)')).toEqual(['Root', 'Example', '(123)'])
+      expect(PathHelper.getSegments('/Root/Example(123)')).toEqual(['Root', 'Example(123)'])
     })
     it('Should split the path to segments', () => {
       expect(PathHelper.getSegments('/Root/Example/100pages(3).pdf')).toEqual(['Root', 'Example', '100pages(3).pdf'])
     })
+    it('Should split the path to segments', () => {
+      expect(PathHelper.getSegments('/content(123)')).toEqual([])
+    })
     it('should throw an error if the path is /', () => {
       expect(() => PathHelper.getSegments('/')).toThrow()
     })
@@ -195,8 +198,20 @@ export const pathHelperTests = describe('PathHelper', () => {
       expect(PathHelper.getParentPath("Root/Example('Content')")).toBe('Root/Example')
     })
 
+    it('Should return the parent path in case of more than 1 segments with item path', () => {
+      expect(PathHelper.getParentPath('Root/Memo/Test(1)')).toBe('Root/Memo')
+    })
+
     it('Should return the path in case of 1 segments', () => {
       expect(PathHelper.getParentPath('Root')).toBe('Root')
     })
+
+    it('Should return the path in case of 1 segments', () => {
+      expect(PathHelper.getParentPath('content(123)')).toBe('')
+    })
+
+    it('Should return the path in case of 1 segments', () => {
+      expect(PathHelper.getParentPath('/content(123)')).toBe('')
+    })
   })
 })