From cc4f1fa83772bca703ccb9cff5784c5e56bbcd3f Mon Sep 17 00:00:00 2001
From: Christian Georgi <christian.georgi@sap.com>
Date: Fri, 16 May 2025 13:44:11 +0200
Subject: [PATCH] chore: more robust test execution from different root dir

When tests are executed from a different root dir, e.g in a parent
monorepo, they still need to resolve paths reliably, i.e. using
`__dirname`.

Also write config file path in case of errors.
---
 lib/compile/index.js             |  2 +-
 test/lib/compile/openapi.test.js | 28 ++++++++++++++--------------
 2 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/lib/compile/index.js b/lib/compile/index.js
index cd49672..9c522b3 100644
--- a/lib/compile/index.js
+++ b/lib/compile/index.js
@@ -104,7 +104,7 @@ function toOpenApiOptions(csdl, csn, options = {}) {
         }
       });
     } else {
-      throw new Error("Error while parsing the openapi configuration file");
+      throw new Error("Error while parsing the openapi configuration file " + result["config-file"]);
     }
   }
 
diff --git a/test/lib/compile/openapi.test.js b/test/lib/compile/openapi.test.js
index bae3fc6..513bff0 100644
--- a/test/lib/compile/openapi.test.js
+++ b/test/lib/compile/openapi.test.js
@@ -286,7 +286,7 @@ service CatalogService {
     const csn = cds.compile.to.csn(`
       service A {entity E { key ID : UUID; };};`
     );
-    const openapi = toOpenApi(csn, { 'openapi:config-file': path.resolve("./test/lib/compile/data/configFile.json") });
+    const openapi = toOpenApi(csn, { 'openapi:config-file': path.resolve(__dirname, "data/configFile.json") });
     expect(openapi.servers).toBeTruthy();
     expect(openapi).toMatchObject({ servers: [{ url: 'http://foo.bar:8080' }, { url: "http://foo.bar:8080/a/foo" }] });
     expect(openapi.info.description).toMatch(/yuml.*diagram/i);
@@ -298,7 +298,7 @@ service CatalogService {
        @title:'It is located at http://example.com:8080' service A {entity E { key ID : UUID;};};`
     );
     const options = {
-      'openapi:config-file': path.resolve("./test/lib/compile/data/configFile.json"),
+      'openapi:config-file': path.resolve(__dirname, "data/configFile.json"),
       'openapi:url': "http://example.com:8080",
       'odata-version': '4.0',
       'openapi:diagram': "false"
@@ -398,9 +398,9 @@ service CatalogService {
     const csn = cds.compile.to.csn(`
       namespace sap.odm.test;
       service A {
-        entity E1 { 
-          key id: String(4); 
-          oid: String(128); 
+        entity E1 {
+          key id: String(4);
+          oid: String(128);
           @ODM.oidReference.entityName: 'ReferencedEntityName'
           ref: Association to one E2;
         }
@@ -505,9 +505,9 @@ service CatalogService {
         url: 'https://help.sap.com/docs/product/123.html'
       }
       service A {
-        entity E1 { 
-          key id: String(4); 
-          oid: String(128); 
+        entity E1 {
+          key id: String(4);
+          oid: String(128);
         }
           }`);
     const openAPI = toOpenApi(csn);
@@ -534,17 +534,17 @@ service CatalogService {
       @OpenAPI.Extensions: {
         ![dpp-is-potentially-sensitive]: 'true'
       }
-        entity E1 { 
-          key id: String(4); 
-          oid: String(128); 
+        entity E1 {
+          key id: String(4);
+          oid: String(128);
         }
-        
+
         @OpenAPI.Extensions: {
         ![x-sap-operation-intent]: 'read-collection for function',
         ![sap-deprecated-operation] : {
           deprecationDate: '2022-12-31',
           successorOperationId: 'successorOperation',
-          notValidKey: 'notValidValue'  
+          notValidKey: 'notValidValue'
         }
       }
         function F1(param: String) returns String;
@@ -553,7 +553,7 @@ service CatalogService {
         ![x-sap-operation-intent]: 'read-collection for action'
     }
         action A1(param: String) returns String;
-          
+
           }`);
     const openAPI = toOpenApi(csn);
     expect(openAPI).toBeDefined();