fix(project-access): guard against undefined URL in uniformUrl (#4498)

The uniformUrl() function in processServices() would throw a TypeError
'Cannot read properties of undefined (reading replace)' when a service
endpoint had no path and no urlPath fallback was available.

Added an early return for falsy URL values and comprehensive tests for
the processServices() function.

Author: unseen1980

.changeset/fix-uniform-url-guard.md

@@ -0,0 +1,5 @@
+---
+'@sap-ux/project-access': patch
+---
+
+Fix: guard against undefined URL in uniformUrl to prevent TypeError when service endpoints have no path

packages/project-access/src/project/cap.ts

@@ -378,6 +378,9 @@ function extractCdsFilesFromMessage(sources: Record<string, { filename?: string
  * @returns - uniform url
  */
 function uniformUrl(url: string): string {
+    if (!url) {
+        return '';
+    }
     return url
         .replace(/\\/g, '/')
         .replace(/\/\//g, '/')

packages/project-access/test/project/cap.test.ts

@@ -23,6 +23,7 @@ import {
     isCapJavaProject,
     getCapModelAndServices,
     getCapProjectType,
+    processServices,
     readCapServiceMetadataEdmx,
     toReferenceUri,
     isCapProject,
@@ -1831,6 +1832,94 @@ describe('Test hasMinCdsVersion()', () => {
     });
 });
 
+describe('Test processServices()', () => {
+    test('should return empty array for undefined input', () => {
+        expect(processServices(undefined)).toEqual([]);
+    });
+
+    test('should return empty array for non-array input', () => {
+        expect(processServices({})).toEqual([]);
+    });
+
+    test('should handle services where endpoint path and urlPath are both undefined', () => {
+        const services = [
+            {
+                name: 'TestService',
+                endpoints: [
+                    {
+                        kind: 'odata',
+                        path: undefined
+                    }
+                ],
+                urlPath: undefined
+            }
+        ];
+        const result = processServices(services as any);
+        expect(result).toEqual([
+            {
+                name: 'TestService',
+                urlPath: '',
+                runtime: undefined
+            }
+        ]);
+    });
+
+    test('should use endpoint path when available', () => {
+        const services = [
+            {
+                name: 'TestService',
+                endpoints: [
+                    {
+                        kind: 'odata',
+                        path: 'odata/v4/test'
+                    }
+                ]
+            }
+        ];
+        const result = processServices(services as any);
+        expect(result).toEqual([
+            {
+                name: 'TestService',
+                urlPath: 'odata/v4/test',
+                runtime: undefined
+            }
+        ]);
+    });
+
+    test('should fall back to urlPath when no odata endpoint path', () => {
+        const services = [
+            {
+                name: 'TestService',
+                urlPath: 'service/path'
+            }
+        ];
+        const result = processServices(services as any);
+        expect(result).toEqual([
+            {
+                name: 'TestService',
+                urlPath: 'service/path',
+                runtime: undefined
+            }
+        ]);
+    });
+
+    test('should filter out non-odata services with endpoints', () => {
+        const services = [
+            {
+                name: 'WebSocketService',
+                endpoints: [
+                    {
+                        kind: 'websocket',
+                        path: 'ws/test'
+                    }
+                ]
+            }
+        ];
+        const result = processServices(services as any);
+        expect(result).toEqual([]);
+    });
+});
+
 function fail(message: string) {
     expect(message).toBeFalsy();
 }