fix: fix file type checking

nestjs · Feb 3, 2023 · a0f7ee8 · a0f7ee8
1 parent 809dadb
commit a0f7ee8
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/packages/common/pipes/file/file-type.validator.ts b/packages/common/pipes/file/file-type.validator.ts
@@ -21,7 +21,7 @@ export class FileTypeValidator extends FileValidator<FileTypeValidatorOptions> {
     return `Validation failed (expected type is ${this.validationOptions.fileType})`;
   }
 
-  isValid(file: FileType): boolean {
+  isValid(file: Partial<FileType> | FileType): boolean {
     if (!this.validationOptions) {
       return true;
     }

diff --git a/packages/common/pipes/file/file-validator.interface.ts b/packages/common/pipes/file/file-validator.interface.ts
@@ -1,4 +1,4 @@
-import { FileType } from "./interfaces";
+import { FileType } from './interfaces';
 
 /**
  * Interface describing FileValidators, which can be added to a {@link ParseFilePipe}.
@@ -12,7 +12,9 @@ export abstract class FileValidator<TValidationOptions = Record<string, any>> {
    * Indicates if this file should be considered valid, according to the options passed in the constructor.
    * @param file the file from the request object
    */
-  abstract isValid(file?: FileType): boolean | Promise<boolean>;
+  abstract isValid(
+    file?: FileType | Partial<FileType>,
+  ): boolean | Promise<boolean>;
 
   /**
    * Builds an error message in case the validation fails.

diff --git a/packages/common/pipes/file/max-file-size.validator.ts b/packages/common/pipes/file/max-file-size.validator.ts
@@ -17,7 +17,7 @@ export class MaxFileSizeValidator extends FileValidator<MaxFileSizeValidatorOpti
     return `Validation failed (expected size is less than ${this.validationOptions.maxSize})`;
   }
 
-  public isValid(file: FileType): boolean {
+  public isValid(file: Pick<FileType, 'size'>): boolean {
     if (!this.validationOptions) {
       return true;
     }