fix: add check for nested empty JSON objects (#457)

Author: Le-Caignec

.github/workflows/deploy-subgraph.yml

@@ -5,7 +5,9 @@ on:
 
 jobs:
   deploy:
-    runs-on: ubuntu-latest
+    runs-on:
+      group: Azure_runners
+      labels: [ self-hosted, Linux, X64 ]
     strategy:
       matrix:
         include:

packages/sdk/src/utils/data.ts

@@ -263,32 +263,24 @@ export const createZipFromObject = (obj: unknown): Promise<Uint8Array> => {
   );
 };
 
-export const reverseSafeSchema = function (
+export const reverseSafeSchema = (
   schema?: Array<Record<'id', string>>
-) {
-  if (!schema) {
-    return {};
-  }
-  return schema.reduce((propsAndTypes, { id: propPathColonType }) => {
-    // { id: 'nested.something:string' }
-    const [key, value] = propPathColonType.split(':');
-    // key = 'nested.something'
-    // value = 'string'
-    if (key.includes('.')) {
-      const firstKey = key.split('.')[0];
-      // firstKey = 'nested'
-      const restOfKey = key.split('.').slice(1).join('.');
-      // restOfKey = 'something'
-      const withValue = `${restOfKey}:${value}`;
-      // withValue = 'something:string'
-      propsAndTypes[firstKey] = reverseSafeSchema([
-        {
-          id: withValue,
-        },
-      ]);
-    } else {
-      propsAndTypes[key] = value;
+): Record<string, any> => {
+  if (!schema) return {};
+
+  return schema.reduce((acc, { id }) => {
+    const [path, type] = id.split(':');
+    const keys = path.split('.');
+
+    let current = acc;
+    for (let i = 0; i < keys.length - 1; i++) {
+      current[keys[i]] ??= {};
+      current = current[keys[i]];
     }
-    return propsAndTypes;
+
+    const finalKey = keys[keys.length - 1];
+    current[finalKey] = type;
+
+    return acc;
   }, {});
 };

packages/sdk/tests/e2e/constructor.test.ts

@@ -1,11 +1,12 @@
 import { describe, it, expect } from '@jest/globals';
 import { IExecDataProtector } from '../../src/index.js';
+import { getTestConfig } from '../test-utils.js';
 
 describe('When instantiating SDK without a signer', () => {
   describe('When calling a read method', () => {
     it('should work as expected', async () => {
       // --- GIVEN
-      const dataProtector = new IExecDataProtector();
+      const dataProtector = new IExecDataProtector(...getTestConfig());
 
       // --- WHEN/THEN
       await expect(