feat: allow custom types with inner types (#278)

* feat: allow custom types with inner types * fix: concat multiple inner types
electron · Oct 20, 2024 · c57ff0c · c57ff0c
1 parent a631fee
commit c57ff0c
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 0 deletions.
diff --git a/src/utils.ts b/src/utils.ts
@@ -255,6 +255,9 @@ export const typify = (
       // we'll have to qualify it with the Node.js namespace.
       return 'NodeJS.ReadableStream';
   }
+  // Custom type
+  if (innerTypes)
+    return `${typeAsString}<${innerTypes.map((innerType) => typify(innerType)).join(', ')}>`;
   return typeAsString;
 };
 export const paramify = (paramName: string) => {

diff --git a/test/utils.spec.ts b/test/utils.spec.ts
@@ -102,6 +102,38 @@ describe('utils', () => {
     it('should map node objects to the correct type', () => {
       expect(utils.typify('buffer')).toEqual('Buffer');
     });
+
+    it('should convert custom types with inner types', () => {
+      expect(
+        utils.typify({
+          collection: false,
+          innerTypes: [
+            {
+              collection: false,
+              type: 'T',
+            },
+          ],
+          type: 'Foo',
+        }),
+      ).toEqual('Foo<T>');
+
+      expect(
+        utils.typify({
+          collection: false,
+          innerTypes: [
+            {
+              collection: false,
+              type: 'A',
+            },
+            {
+              collection: false,
+              type: 'B',
+            },
+          ],
+          type: 'Foo',
+        }),
+      ).toEqual('Foo<A, B>');
+    });
   });
 
   describe('paramify', () => {