SchemaFactory trigger an error when a schema doesn't have the @Schema decorator

lib/factories/definitions.factory.ts

@@ -21,11 +21,12 @@ export class DefinitionsFactory {
         break;
       }
       const schemaMetadata = TypeMetadataStorage.getSchemaMetadataByTarget(
-        parent as Type<unknown>,
+        target as Type<unknown>,
       );
       if (!schemaMetadata) {
-        parent = Object.getPrototypeOf(parent);
-        continue;
+        throw new Error(
+          `Target class "${target}" passed in to the "DefinitionsFactory#createForClass()" method isn't a @Schema.`,
+        );
       }
       schemaMetadata.properties?.forEach((item) => {
         const options = this.inspectTypeDefinition(item.options as any);

tests/e2e/mongoose.spec.ts

@@ -32,6 +32,18 @@ describe('Mongoose', () => {
       });
   });
 
+  it(`should return updated document`, (done) => {
+    const updateDto = { name: 'Nest', breed: 'Maine coon', age: Date.now() };
+    request(server)
+      .put('/cats')
+      .send(updateDto)
+      .expect(200)
+      .end((err, { body }) => {
+        expect(body.modifiedCount).toBeGreaterThan(0);
+        done();
+      });
+  });
+
   afterEach(async () => {
     await app.close();
   });

tests/e2e/schema.factory.spec.ts

@@ -26,6 +26,17 @@ class ExampleClass {
   array: Array<any>;
 }
 
+class BrokenClass {
+  @Prop({ required: true })
+  children: ChildClass;
+
+  @Prop([ChildClass])
+  nodes: ChildClass[];
+
+  @Prop()
+  array: Array<any>;
+}
+
 describe('SchemaFactory', () => {
   it('should populate the schema options', () => {
     const schema = SchemaFactory.createForClass(ExampleClass) as any;
@@ -50,4 +61,7 @@ describe('SchemaFactory', () => {
       }),
     );
   });
+  it('should break the schema', () => {
+    expect(() => SchemaFactory.createForClass(BrokenClass)).toThrow(Error);
+  });
 });

tests/src/cats/cats.controller.ts

@@ -1,6 +1,6 @@
-import { Body, Controller, Get, Post } from '@nestjs/common';
+import { Body, Controller, Get, Post, Put } from '@nestjs/common';
 import { CatsService } from './cats.service';
-import { CreateCatDto } from './dto/create-cat.dto';
+import { CreateCatDto, UpdateCatDto } from './dto';
 import { Cat } from './schemas/cat.schema';
 
 @Controller('cats')
@@ -12,6 +12,11 @@ export class CatsController {
     return this.catsService.create(createCatDto);
   }
 
+  @Put()
+  async update(@Body() updateCatDto: UpdateCatDto) {
+    return this.catsService.update(updateCatDto);
+  }
+
   @Get()
   async findAll(): Promise<Cat[]> {
     return this.catsService.findAll();

tests/src/cats/cats.service.ts

@@ -1,7 +1,7 @@
 import { Injectable } from '@nestjs/common';
-import { Model } from 'mongoose';
+import { Model, UpdateWriteOpResult } from 'mongoose';
 import { InjectModel } from '../../../lib';
-import { CreateCatDto } from './dto/create-cat.dto';
+import { CreateCatDto, UpdateCatDto } from './dto';
 import { Cat } from './schemas/cat.schema';
 
 @Injectable()
@@ -13,6 +13,12 @@ export class CatsService {
     return createdCat.save();
   }
 
+  async update(updateCat: UpdateCatDto): Promise<UpdateWriteOpResult> {
+    const filter = { name: updateCat.name };
+    const updatedCat = this.catModel.updateOne(filter, updateCat);
+    return updatedCat;
+  }
+
   async findAll(): Promise<Cat[]> {
     return this.catModel.find().exec();
   }

tests/src/cats/dto/index.ts

@@ -0,0 +1,2 @@
+export * from './create-cat.dto';
+export * from './update-cat.dto';

tests/src/cats/dto/update-cat.dto.ts

@@ -0,0 +1,5 @@
+export class UpdateCatDto {
+  readonly name: string;
+  readonly age: number;
+  readonly breed: string;
+}