Feature brands relation

src/app.module.ts

@@ -43,7 +43,7 @@ import { OffersModule } from './offers/offers.module';
       password: process.env.DB_PASS,
       database: process.env.DB_NAME,
       autoLoadEntities: true,
-      // synchronize: true,
+      //synchronize: true,
     }),
 
     AccountStatusModule,
@@ -76,4 +76,5 @@ import { OffersModule } from './offers/offers.module';
   controllers: [AppController],
   providers: [AppService],
 })
+
 export class AppModule {}

src/brands/controllers/brands.controllers.spec.ts

@@ -10,7 +10,7 @@ describe('brandsController', () => {
   const mockBrandService = {
     create: jest.fn((dto) => {
       return {
-        id: expect.any(Number),
+        id: Date.now(),
         ...dto,
       };
     }),
@@ -38,19 +38,19 @@ describe('brandsController', () => {
   });
 
   it('should create a brand', () => {
-    expect(controller.create({ name: 'Mario', photoId: 'ph id' })).toEqual({
+    expect(controller.create({ name: 'Mario', photoId: 123 })).toEqual({
       id: expect.any(Number),
       name: 'Mario',
-      photoId: 'ph id',
+      photoId: 123,
     });
   });
 
   it('should update a brand', () => {
-    const dto: createBrandDTO = { name: 'Mario', photoId: 'ph id' };
-    expect(controller.update(1, dto)).toEqual({
-      id: expect.any(Number),
+    const brand = {name: 'Mario', photoId:123 };
+    expect(controller.update(1, brand)).toEqual({
+      id: 1,
       name: 'Mario',
-      photoId: 'ph id',
+      photoId: 123,
     });
   });
 });

src/brands/controllers/brands.controllers.ts

@@ -16,6 +16,7 @@ import { BrandsService } from '../services/brands.serveces';
 import { createBrandDTO } from '../entities/create-brands-dto.entity';
 import {
   ApiBadRequestResponse,
+  ApiNotFoundResponse,
   ApiOperation,
   ApiResponse,
   ApiTags,
@@ -51,6 +52,7 @@ export class BrandsController {
     description: 'Shows the search result for a brand by id',
     type: Brand,
   })
+  @ApiNotFoundResponse({status: 404, description: 'No Brand was found that matches that id'})
   @Get(':id')
   getOne(@Param('id') id: number) {
     return this.brandService.findOne(id);
@@ -90,6 +92,7 @@ export class BrandsController {
   @ApiBadRequestResponse({
     description: 'The brand could not be deleted',
   })
+  @ApiNotFoundResponse({status: 404, description: 'No Brand was found that matches that id'})
   @Delete()
   delete(@Param('id') id: number) {
     return this.brandService.delete(id);

src/brands/entities/brands.entity.ts

@@ -1,5 +1,6 @@
 import { ApiProperty } from '@nestjs/swagger';
-import { Entity, Column, PrimaryGeneratedColumn } from 'typeorm';
+import { Entity, Column, PrimaryGeneratedColumn, OneToOne, JoinColumn } from 'typeorm';
+import { PhotosEntity } from '../../photos/models/photos.entity';
 
 @Entity({ name: 'brands' })
 export class Brand {
@@ -19,9 +20,11 @@ export class Brand {
 
   @Column()
   @ApiProperty({
-    example: '',
+    example: 1234,
     description: 'The logo of the brand',
-    type: String,
+    type: Number,
   })
-  photoId: string;
+  @OneToOne(() => PhotosEntity)
+  @JoinColumn({ name: 'photo_id' })
+  photoId: PhotosEntity;
 }

src/brands/entities/create-brands-dto.entity.ts

@@ -1,7 +1,8 @@
 import { ApiProperty } from '@nestjs/swagger';
-import { IsString } from 'class-validator';
+import { IsString, IsInt } from 'class-validator';
 
 export class createBrandDTO {
+
   @IsString()
   @ApiProperty({
     example: '',
@@ -10,11 +11,11 @@ export class createBrandDTO {
   })
   name: string;
 
-  @IsString()
+  @IsInt()
   @ApiProperty({
-    example: '',
+    example: 123,
     description: 'The logo of the brand',
-    type: String,
+    type: Number,
   })
-  photoId: string;
+  photoId: number;
 }

src/brands/services/brands.serveces.ts

@@ -15,20 +15,16 @@ export class BrandsService {
     private brandRepo: Repository<Brand>,
   ) {}
 
-  findOne(id: number) {
+  findOne(id: number): Promise<Brand> {
     return this.brandRepo.findOne(id);
   }
 
-  findAll() {
-    return this.brandRepo.find();
-  }
-
-  create(body: any) {
+  create(body: any): Promise<Brand[]> {
     const newBrand = this.brandRepo.create(body);
     return this.brandRepo.save(newBrand);
   }
 
-  async update(id: number, body: any) {
+  async update(id: number, body: any): Promise<Brand> {
     const brand = await this.brandRepo.findOne(id);
     this.brandRepo.merge(brand, body);
     return this.brandRepo.save(brand);