Fix tests due to bump in class-validator

GnaneshKunal · GnaneshKunal · commit cbf26195f313 · 2023-07-20T15:38:05.000+05:30
diff --git a/redisinsight/api/src/common/models/client-metadata.ts b/redisinsight/api/src/common/models/client-metadata.ts
@@ -21,7 +21,11 @@ export class ClientMetadata {
   databaseId: string;
 
   @IsNotEmpty()
-  @IsEnum(ClientContext)
+  @IsEnum(ClientContext, {
+    message: `context must be a valid enum value. Valid values: ${Object.values(
+      ClientContext,
+    )}.`,
+  })
   context: ClientContext;
 
   @IsOptional()
diff --git a/redisinsight/api/src/modules/browser/dto/browser-history/create.browser-history.dto.ts b/redisinsight/api/src/modules/browser/dto/browser-history/create.browser-history.dto.ts
@@ -20,6 +20,10 @@ export class CreateBrowserHistoryDto {
     example: BrowserHistoryMode.Pattern,
   })
   @IsOptional()
-  @IsEnum(BrowserHistoryMode)
+  @IsEnum(BrowserHistoryMode, {
+    message: `mode must be a valid enum value. Valid values: ${Object.values(
+      BrowserHistoryMode,
+    )}.`,
+  })
   mode?: BrowserHistoryMode = BrowserHistoryMode.Pattern;
 }
diff --git a/redisinsight/api/src/modules/browser/dto/browser-history/get.browser-history.dto.ts b/redisinsight/api/src/modules/browser/dto/browser-history/get.browser-history.dto.ts
@@ -12,7 +12,11 @@ export class ScanFilter {
   })
   @IsOptional()
   @Expose()
-  @IsEnum(RedisDataType)
+  @IsEnum(RedisDataType, {
+    message: `type must be a valid enum value. Valid values: ${Object.values(
+      RedisDataType,
+    )}.`,
+  })
   type?: RedisDataType = null;
 
   @ApiProperty({
diff --git a/redisinsight/api/src/modules/browser/dto/redisearch.ts b/redisinsight/api/src/modules/browser/dto/redisearch.ts
@@ -43,7 +43,11 @@ export class CreateRedisearchIndexFieldDto {
     enum: RedisearchIndexDataType,
   })
   @IsDefined()
-  @IsEnum(RedisearchIndexDataType)
+  @IsEnum(RedisearchIndexDataType, {
+    message: `type must be a valid enum value. Valid values: ${Object.values(
+      RedisearchIndexDataType,
+    )}.`,
+  })
   type: RedisearchIndexDataType;
 }
 
@@ -62,7 +66,11 @@ export class CreateRedisearchIndexDto {
     enum: RedisearchIndexKeyType,
   })
   @IsDefined()
-  @IsEnum(RedisearchIndexKeyType)
+  @IsEnum(RedisearchIndexKeyType, {
+    message: `type must be a valid enum value. Valid values: ${Object.values(
+      RedisearchIndexKeyType,
+    )}.`,
+  })
   type: RedisearchIndexKeyType;
 
   @ApiPropertyOptional({
diff --git a/redisinsight/api/src/modules/bulk-actions/dto/create-bulk-action.dto.ts b/redisinsight/api/src/modules/bulk-actions/dto/create-bulk-action.dto.ts
@@ -12,7 +12,11 @@ export class CreateBulkActionDto extends BulkActionIdDto {
   databaseId: string;
 
   @IsNotEmpty()
-  @IsEnum(BulkActionType)
+  @IsEnum(BulkActionType, {
+    message: `type must be a valid enum value. Valid values: ${Object.values(
+      BulkActionType,
+    )}.`,
+  })
   type: BulkActionType;
 
   @IsNotEmpty()
diff --git a/redisinsight/api/src/modules/bulk-actions/models/bulk-action-filter.ts b/redisinsight/api/src/modules/bulk-actions/models/bulk-action-filter.ts
@@ -6,7 +6,11 @@ import { IBulkActionFilterOverview } from 'src/modules/bulk-actions/interfaces/b
 
 export class BulkActionFilter {
   @IsOptional()
-  @IsEnum(RedisDataType)
+  @IsEnum(RedisDataType, {
+    message: `type must be a valid enum value. Valid values: ${Object.values(
+      RedisDataType,
+    )}.`,
+  })
   type?: RedisDataType = null;
 
   @IsOptional()
diff --git a/redisinsight/api/src/modules/cloud/autodiscovery/dto/add-cloud-database.dto.ts b/redisinsight/api/src/modules/cloud/autodiscovery/dto/add-cloud-database.dto.ts
@@ -14,7 +14,11 @@ export class AddCloudDatabaseDto {
   @IsInt({ always: true })
   subscriptionId: number;
 
-  @IsEnum(CloudSubscriptionType)
+  @IsEnum(CloudSubscriptionType, {
+    message: `subscriptionType must be a valid enum value. Valid values: ${Object.values(
+      CloudSubscriptionType,
+    )}.`,
+  })
   @IsNotEmpty()
   subscriptionType: CloudSubscriptionType;
 
diff --git a/redisinsight/api/src/modules/cloud/autodiscovery/dto/get-cloud-subscription-database.dto.ts b/redisinsight/api/src/modules/cloud/autodiscovery/dto/get-cloud-subscription-database.dto.ts
@@ -16,7 +16,11 @@ export class GetCloudSubscriptionDatabaseDto {
   @Type(() => Number)
   subscriptionId: number;
 
-  @IsEnum(CloudSubscriptionType)
+  @IsEnum(CloudSubscriptionType, {
+    message: `subscriptionType must be a valid enum value. Valid values: ${Object.values(
+      CloudSubscriptionType,
+    )}.`,
+  })
   @IsNotEmpty()
   subscriptionType: CloudSubscriptionType;
 
diff --git a/redisinsight/api/src/modules/cloud/autodiscovery/dto/get-cloud-subscription-databases.dto.ts b/redisinsight/api/src/modules/cloud/autodiscovery/dto/get-cloud-subscription-databases.dto.ts
@@ -20,7 +20,11 @@ export class GetCloudSubscriptionDatabasesDto {
     description: 'Subscription Id',
     enum: CloudSubscriptionType,
   })
-  @IsEnum(CloudSubscriptionType)
+  @IsEnum(CloudSubscriptionType, {
+    message: `subscriptionType must be a valid enum value. Valid values: ${Object.values(
+      CloudSubscriptionType,
+    )}.`,
+  })
   @IsNotEmpty()
   subscriptionType: CloudSubscriptionType;
 }
diff --git a/redisinsight/api/src/modules/cloud/autodiscovery/models/cloud-database-details.ts b/redisinsight/api/src/modules/cloud/autodiscovery/models/cloud-database-details.ts
@@ -26,7 +26,11 @@ export class CloudDatabaseDetails {
   })
   @Expose()
   @IsNotEmpty()
-  @IsEnum(CloudSubscriptionType)
+  @IsEnum(CloudSubscriptionType, {
+    message: `subscriptionType must be a valid enum value. Valid values: ${Object.values(
+      CloudSubscriptionType,
+    )}.`,
+  })
   subscriptionType: CloudSubscriptionType;
 
   @ApiPropertyOptional({
diff --git a/redisinsight/api/src/modules/custom-tutorial/models/custom-tutorial.manifest.ts b/redisinsight/api/src/modules/custom-tutorial/models/custom-tutorial.manifest.ts
@@ -50,7 +50,11 @@ export class CustomTutorialManifest {
 
   @ApiProperty({ enum: CustomTutorialManifestType })
   @Expose()
-  @IsEnum(CustomTutorialManifestType)
+  @IsEnum(CustomTutorialManifestType, {
+    message: `type must be a valid enum value. Valid values: ${Object.values(
+      CustomTutorialManifestType,
+    )}.`,
+  })
   type: CustomTutorialManifestType;
 
   @ApiProperty({ type: String })
diff --git a/redisinsight/api/src/modules/database-analysis/models/scan-filter.ts b/redisinsight/api/src/modules/database-analysis/models/scan-filter.ts
@@ -13,7 +13,11 @@ export class ScanFilter {
   })
   @IsOptional()
   @Expose()
-  @IsEnum(RedisDataType)
+  @IsEnum(RedisDataType, {
+    message: `type must be a valid enum value. Valid values: ${Object.values(
+      RedisDataType,
+    )}.`,
+  })
   type?: RedisDataType = null;
 
   @ApiProperty({
diff --git a/redisinsight/api/src/modules/database-recommendation/dto/modify.database-recommendation.dto.ts b/redisinsight/api/src/modules/database-recommendation/dto/modify.database-recommendation.dto.ts
@@ -9,7 +9,11 @@ export class ModifyDatabaseRecommendationDto {
     enum: Vote,
   })
   @IsOptional()
-  @IsEnum(Vote)
+  @IsEnum(Vote, {
+    message: `vote must be a valid enum value. Valid values: ${Object.values(
+      Vote,
+    )}.`,
+  })
   vote?: Vote;
 
   @ApiPropertyOptional({
diff --git a/redisinsight/api/src/modules/database/dto/update.database.dto.ts b/redisinsight/api/src/modules/database/dto/update.database.dto.ts
@@ -166,7 +166,11 @@ export class UpdateDatabaseDto extends CreateDatabaseDto {
     enum: Compressor,
   })
   @Expose()
-  @IsEnum(Compressor)
+  @IsEnum(Compressor, {
+    message: `compressor must be a valid enum value. Valid values: ${Object.values(
+      Compressor,
+    )}.`,
+  })
   @IsOptional()
   compressor?: Compressor = Compressor.NONE;
 }
diff --git a/redisinsight/api/src/modules/database/models/database.ts b/redisinsight/api/src/modules/database/models/database.ts
@@ -274,7 +274,11 @@ export class Database {
     enum: Compressor,
   })
   @Expose()
-  @IsEnum(Compressor)
+  @IsEnum(Compressor, {
+    message: `compressor must be a valid enum value. Valid values: ${Object.values(
+      Compressor,
+    )}.`,
+  })
   @IsOptional()
   compressor?: Compressor = Compressor.NONE;
 
diff --git a/redisinsight/api/src/modules/pub-sub/dto/subscription.dto.ts b/redisinsight/api/src/modules/pub-sub/dto/subscription.dto.ts
@@ -7,6 +7,10 @@ export class SubscriptionDto {
   channel: string;
 
   @IsNotEmpty()
-  @IsEnum(SubscriptionType)
+  @IsEnum(SubscriptionType, {
+    message: `type must be a valid enum value. Valid values: ${Object.values(
+      SubscriptionType,
+    )}.`,
+  })
   type: SubscriptionType;
 }
diff --git a/redisinsight/api/test/helpers/test.ts b/redisinsight/api/test/helpers/test.ts
@@ -131,7 +131,7 @@ export const checkResponseBody = (body, expected) => {
 
 const defaultValidationErrorMessages = {
   'any.required': '{#label} should not be null or undefined',
-  'any.only': '{#label} must be one of the following values',
+  'any.only': '{#label} must be a valid enum value',
   'array.base': '{#label} must be an array',
   'string.base': `{#label} must be a string`,
   'string.empty': `{#label} should not be null or undefined`,
