fix: enumeration issue with session context in @cap-js/hana (#399)

Co-authored-by: Johannes Vogel <31311694+johannes-vogel@users.noreply.github.com>

db-service/lib/common/session-context.js

@@ -28,5 +28,19 @@ class TemporalSessionContext extends SessionContext {
   }
 }
 
+// Set all getters as enumerable
+const iterate = { enumerable: true }
+const getters = (obj) => {
+  const prot = obj.prototype
+  const patch = {}
+  for (const [key, value] of Object.entries(Object.getOwnPropertyDescriptors(prot))) {
+    if (!value.get) continue
+    patch[key] = iterate
+  }
+  Object.defineProperties(prot, patch)
+}
+getters(SessionContext)
+getters(TemporalSessionContext)
+
 // REVISIT: only set temporal context if required!
 module.exports = TemporalSessionContext

hana/lib/drivers/hana-client.js

@@ -45,7 +45,9 @@ class HANAClientDriver extends driver {
   }
 
   set(variables) {
-    Object.keys(variables).forEach(k => this._native.setClientInfo(k, variables[k]))
+    for(const key in variables) {
+      this._native.setClientInfo(key, variables[key])
+    }
   }
 
   async prepare(sql) {

hana/lib/drivers/hdb.js

@@ -40,7 +40,9 @@ class HDBDriver extends driver {
 
   set(variables) {
     const clientInfo = this._native._connection.getClientInfo()
-    Object.keys(variables).forEach(k => clientInfo.setProperty(k, variables[k]))
+    for(const key in variables) {
+      clientInfo.setProperty(key, variables[key])
+    }
   }
 
   async validate() {