fix(): fix aliases issue (plugin)

nestjs · Jan 28, 2020 · cc93dfa · cc93dfa
1 parent 9847e4c
commit cc93dfa
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/lib/plugin/utils/plugin-utils.ts b/lib/plugin/utils/plugin-utils.ts
@@ -7,6 +7,7 @@ import {
   getTypeArguments,
   isArray,
   isBoolean,
+  isEnum,
   isInterface,
   isNumber,
   isString
@@ -69,6 +70,12 @@ export function getTypeReferenceAsString(
     ) {
       return 'Object';
     }
+    if (isEnum(type)) {
+      return undefined;
+    }
+    if (type.aliasSymbol) {
+      return 'Object';
+    }
     return undefined;
   } catch {
     return undefined;

diff --git a/test/plugin/fixtures/create-cat-alt.dto.ts b/test/plugin/fixtures/create-cat-alt.dto.ts
@@ -10,19 +10,28 @@ interface Node {
     id: number;
 }
 
+type AliasedType = {
+    type: string;
+};
+type NumberAlias = number;
+
 export class CreateCatDto2 {
   name: string;
   age: number = 3;
   tags: string[];
   status: Status = Status.ENABLED;
   readonly breed?: string;
   nodes: Node[];
+  alias: AliasedType;
+  numberAlias: NumberAlias;
   nested: {
       first: string,
       second: number,
       status: Status,
       tags: string[],
       nodes: Node[]
+      alias: AliasedType,
+      numberAlias: NumberAlias,
   }
 }
 `;
@@ -38,7 +47,7 @@ export class CreateCatDto2 {
         this.status = Status.ENABLED;
     }
     static _OPENAPI_METADATA_FACTORY() {
-        return { name: { required: true, type: () => String }, age: { required: true, type: () => Number, default: 3 }, tags: { required: true, type: () => [String] }, status: { required: true, default: Status.ENABLED, enum: Status }, breed: { required: false, type: () => String }, nodes: { required: true, type: () => [Object] }, nested: { required: true, type: () => ({ first: { required: true, type: () => String }, second: { required: true, type: () => Number }, status: { required: true, enum: Status }, tags: { required: true, type: () => [String] }, nodes: { required: true, type: () => [Object] } }) } };
+        return { name: { required: true, type: () => String }, age: { required: true, type: () => Number, default: 3 }, tags: { required: true, type: () => [String] }, status: { required: true, default: Status.ENABLED, enum: Status }, breed: { required: false, type: () => String }, nodes: { required: true, type: () => [Object] }, alias: { required: true, type: () => Object }, numberAlias: { required: true, type: () => Number }, nested: { required: true, type: () => ({ first: { required: true, type: () => String }, second: { required: true, type: () => Number }, status: { required: true, enum: Status }, tags: { required: true, type: () => [String] }, nodes: { required: true, type: () => [Object] }, alias: { required: true, type: () => Object }, numberAlias: { required: true, type: () => Number } }) } };
     }
 }
 `;