#RI-3902 add transformers for host, port fields + add tests (resolve pr comments)

Author: Artem

redisinsight/api/src/modules/database-import/dto/database-import.response.ts

@@ -1,4 +1,4 @@
-import { isArray } from 'lodash';
+import { isArray, isString, isNumber } from 'lodash';
 import { ApiProperty, ApiPropertyOptional } from '@nestjs/swagger';
 import { Expose, Transform, Type } from 'class-transformer';
 
@@ -28,13 +28,15 @@ export class DatabaseImportResult {
     type: String,
   })
   @Expose()
+  @Transform((v) => (isString(v) ? v : undefined), { toPlainOnly: true })
   host?: string;
 
   @ApiPropertyOptional({
     description: 'Database port',
     type: Number,
   })
   @Expose()
+  @Transform((v) => (isNumber(v) ? v : undefined), { toPlainOnly: true })
   port?: number;
 
   @ApiPropertyOptional({

redisinsight/api/test/api/database-import/POST-databases-import.test.ts

@@ -68,6 +68,18 @@ describe('POST /databases/import', () => {
             total: 1,
             success: [],
             partial: [],
+          },
+          checkFn: ({ body }) => {
+            expect(body.fail.length).to.eq(1);
+            expect(body.fail[0].status).to.eq('fail');
+            expect(body.fail[0].index).to.eq(0);
+            expect(body.fail[0].error).to.be.a('string');
+            if (body.fail[0].host) {
+              expect(body.fail[0].host).to.be.a('string');
+            }
+            if (body.fail[0].port) {
+              expect(body.fail[0].port).to.be.a('number');
+            }
           }
         }
       })