Add file upload functionality to listing creation

- Implement MaxFileCountValidationPipe for limiting file uploads
- Update ListingController to handle file uploads with FilesInterceptor
- Modify ListingService to process uploaded images
- Add file validation for type and size in ListingController
- Update CreateListingDto with proper validation and type conversion

Author: jfjuanmiguel

package.json

@@ -53,6 +53,7 @@
     "@types/cache-manager-redis-store": "^2.0.4",
     "@types/express": "^4.17.17",
     "@types/jest": "^29.5.2",
+    "@types/multer": "^1.4.12",
     "@types/node": "^20.3.1",
     "@types/supertest": "^6.0.0",
     "@typescript-eslint/eslint-plugin": "^8.0.0",

pnpm-lock.yaml

@@ -0,0 +1,45 @@
+import { BadRequestException } from '@nestjs/common';
+import { MaxFileCountValidationPipe } from './max-file-count-validation.pipe';
+
+describe(`MaxFileCountPipe`, () => {
+  let maxFileCountPipe: MaxFileCountValidationPipe;
+
+  beforeEach(async () => {
+    maxFileCountPipe = new MaxFileCountValidationPipe(2);
+  });
+
+  it(`should throw an exception if number of files is greater than the max allowed`, () => {
+    // Arrange
+    const file = {} as Express.Multer.File;
+
+    // Act
+    const result = () => maxFileCountPipe.transform([file, file, file]);
+
+    // Assert
+    expect(result).toThrow(BadRequestException);
+  });
+
+  it(`should return files if count less than the max allowed`, () => {
+    // Arrange
+    const file = {} as Express.Multer.File;
+    const payload = [file];
+
+    // Act
+    const result = maxFileCountPipe.transform(payload);
+
+    // Assert
+    expect(result).toEqual(payload);
+  });
+
+  it(`should return files if count equal to max allowed`, () => {
+    // Arrange
+    const file = {} as Express.Multer.File;
+    const payload = [file, file];
+
+    // Act
+    const result = maxFileCountPipe.transform(payload);
+
+    // Assert
+    expect(result).toEqual(payload);
+  });
+});

src/common/pipes/max-file-count-validation/max-file-count-validation.pipe.spec.ts

@@ -0,0 +1,15 @@
+import { PipeTransform, Injectable, BadRequestException } from '@nestjs/common';
+
+@Injectable()
+export class MaxFileCountValidationPipe implements PipeTransform {
+  constructor(private readonly maxFiles: number) {}
+
+  transform(files: Express.Multer.File[]) {
+    if (files.length > this.maxFiles) {
+      throw new BadRequestException(
+        `Maximum of ${this.maxFiles} file uploads allowed`,
+      );
+    }
+    return files;
+  }
+}

src/common/pipes/max-file-count-validation/max-file-count-validation.pipe.ts

@@ -1,5 +1,6 @@
 import { IsInt, IsNotEmpty, IsOptional, IsString, Min } from 'class-validator';
 import { CreateListingInput } from '../listing.types';
+import { Type } from 'class-transformer';
 
 export class CreateListingDto implements CreateListingInput {
   @IsString()
@@ -29,17 +30,21 @@ export class CreateListingDto implements CreateListingInput {
 
   @IsInt()
   @Min(0)
+  @Type(() => Number)
   price: number;
 
   @IsInt()
   @Min(0)
+  @Type(() => Number)
   bathrooms: number;
 
   @IsInt()
   @Min(0)
+  @Type(() => Number)
   bedrooms: number;
 
   @IsInt()
   @Min(0)
+  @Type(() => Number)
   squareMeters: number;
 }

src/modules/listing/dto/create-listing.dto.ts

@@ -1,13 +1,40 @@
-import { Controller, Post, Body } from '@nestjs/common';
+import {
+  Controller,
+  Post,
+  Body,
+  UseInterceptors,
+  UploadedFiles,
+  FileTypeValidator,
+  ParseFilePipe,
+  MaxFileSizeValidator,
+} from '@nestjs/common';
 import { ListingService } from './listing.service';
 import { CreateListingDto } from './dto/create-listing.dto';
+import { FilesInterceptor } from '@nestjs/platform-express';
+import { MaxFileCountValidationPipe } from '../../common/pipes/max-file-count-validation/max-file-count-validation.pipe';
 
 @Controller('listing')
 export class ListingController {
   constructor(private readonly listingService: ListingService) {}
 
   @Post()
-  create(@Body() createListingDto: CreateListingDto) {
-    return this.listingService.create(createListingDto);
+  @UseInterceptors(FilesInterceptor(`images`))
+  create(
+    @Body() createListingDto: CreateListingDto,
+    @UploadedFiles(
+      new ParseFilePipe({
+        validators: [
+          new FileTypeValidator({ fileType: `.(png|jpeg|jpg)` }),
+          new MaxFileSizeValidator({ maxSize: 1024 * 1024 }),
+        ],
+      }),
+      new MaxFileCountValidationPipe(10),
+    )
+    files: Express.Multer.File[],
+  ) {
+    return this.listingService.create({
+      data: createListingDto,
+      images: files,
+    });
   }
 }

src/modules/listing/listing.controller.ts

@@ -6,9 +6,19 @@ import { DatabaseService } from '../../database/database.service';
 export class ListingService {
   constructor(private readonly databaseService: DatabaseService) {}
 
-  async create(createListingDto: CreateListingDto) {
-    return await this.databaseService.listing.create({
-      data: createListingDto,
+  async create({
+    data,
+    images,
+  }: {
+    data: CreateListingDto;
+    images: Express.Multer.File[];
+  }) {
+    const listing = await this.databaseService.listing.create({
+      data,
     });
+    for (const image of images) {
+      console.log(`image:`, image);
+    }
+    return listing;
   }
 }