fix(plugin): unwrap string literals when used as default vals #2498

nestjs · Jun 28, 2023 · d97d01b · d97d01b
1 parent 81bed53
commit d97d01b
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/lib/plugin/visitors/model-class.visitor.ts b/lib/plugin/visitors/model-class.visitor.ts
@@ -812,7 +812,8 @@ export class ModelClassVisitor extends AbstractFileVisitor {
     if (!primitiveTypeName) {
       return undefined;
     }
-    return createPrimitiveLiteral(factory, node.getText(), primitiveTypeName);
+    const text = (node as any).text ?? node.getText();
+    return createPrimitiveLiteral(factory, text, primitiveTypeName);
   }
 
   private getInitializerPrimitiveTypeName(node: ts.Node) {

diff --git a/test/plugin/fixtures/project/cats/dto/create-cat.dto.ts b/test/plugin/fixtures/project/cats/dto/create-cat.dto.ts
@@ -63,7 +63,7 @@ export class CreateCatDto {
   age: number = 14;
 
   @ApiProperty({ name: '_breed', type: String })
-  breed: string;
+  breed: string = 'Persian';
 
   @ApiProperty({
     format: 'uri',

diff --git a/test/plugin/fixtures/serialized-meta.fixture.ts b/test/plugin/fixtures/serialized-meta.fixture.ts
@@ -125,7 +125,7 @@ export default async () => {
                 default: 14,
                 minimum: 1
               },
-              breed: { required: true, type: () => String },
+              breed: { required: true, type: () => String, default: 'Persian' },
               tags: { required: false, type: () => [String] },
               createdAt: { required: true, type: () => Date },
               urls: { required: false, type: () => [String] },