fix code review

Author: SevenWaysDP

src/infra/configuration/configuration.decorator.ts

@@ -35,7 +35,6 @@ export function Configuration() {
 					},
 				});
 
-				// eslint-disable-next-line @typescript-eslint/no-unsafe-return
 				return proxyInstance;
 			}
 

src/infra/configuration/configuration.module.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-process-env */
 import { DynamicModule, Module } from '@nestjs/common';
 import { ConfigModule, ConfigModuleOptions, ConfigService } from '@nestjs/config';
 import { ConfigurationFactory } from './configuration.factory.js';

src/shared/guard/type.guard.spec.ts

@@ -370,6 +370,50 @@ describe('TypeGuard', () => {
 		});
 	});
 
+	describe('isArrayOfStrings', () => {
+		describe('when passing type of value is an array of strings', () => {
+			it('should return true', () => {
+				expect(TypeGuard.isArrayOfStrings(['', '', ''])).toBe(true);
+			});
+		});
+
+		describe('when passing type of value is NOT an array of strings', () => {
+			it('should return false', () => {
+				expect(TypeGuard.isArrayOfStrings([1, 2, 3])).toBe(false);
+			});
+
+			it('should return false', () => {
+				expect(TypeGuard.isArrayOfStrings(['a', 'b', 3])).toBe(false);
+			});
+
+			it('should return false', () => {
+				expect(TypeGuard.isArrayOfStrings([{}, {}, {}])).toBe(false);
+			});
+		});
+	});
+
+	describe('checkArrayOfStrings', () => {
+		describe('when passing type of value is an array of strings', () => {
+			it('should return value', () => {
+				expect(TypeGuard.checkArrayOfStrings(['', '', ''])).toEqual(['', '', '']);
+			});
+		});
+
+		describe('when passing type of value is NOT an array of strings', () => {
+			it('should throw an error', () => {
+				expect(() => TypeGuard.checkArrayOfStrings([1, 2, 3])).toThrow('Type is not an array of strings.');
+			});
+
+			it('should throw an error', () => {
+				expect(() => TypeGuard.checkArrayOfStrings(['a', 'b', 3])).toThrow('Type is not an array of strings.');
+			});
+
+			it('should throw an error', () => {
+				expect(() => TypeGuard.checkArrayOfStrings([{}, {}, {}])).toThrow('Type is not an array of strings.');
+			});
+		});
+	});
+
 	describe('isArrayWithElements', () => {
 		describe('when passing type of value is an array with elements', () => {
 			it('should return true', () => {
@@ -687,6 +731,18 @@ describe('TypeGuard', () => {
 			});
 		});
 
+		describe('checkKeysInObject', () => {
+			it('should be return the key value', () => {
+				const result = TypeGuard.checkKeysInObject({ x1: 'abc', x2: 'bcd' }, ['x1', 'x2']);
+				expect(result).toEqual({ x1: 'abc', x2: 'bcd' });
+			});
+			it('should throw an error', () => {
+				expect(() => TypeGuard.checkKeysInObject({ x1: 'abc', x2: 'bcd' }, ['b1'])).toThrow(
+					'Object has missing key. Required are: [\"b1\"]. Get object keys: [\"x1\",\"x2\"]',
+				);
+			});
+		});
+
 		describe('when param contract is not fullfilled', () => {
 			it('should be throw an error', () => {
 				// @ts-expect-error test-case

src/shared/transformer/comma-separated-string-to-array.transformer.spec.ts

@@ -4,7 +4,7 @@ import { CommaSeparatedStringToArray } from './comma-separated-string-to-array.t
 describe('CommaSeparatedStringToArray()', () => {
 	class TestDto {
 		@CommaSeparatedStringToArray()
-		items!: string[];
+		public items!: string[];
 	}
 
 	it('should transform a comma separated string to an array of trimmed strings', () => {

src/shared/transformer/comma-separated-string-to-array.transformer.ts

@@ -11,6 +11,7 @@ export function CommaSeparatedStringToArray(): PropertyDecorator {
 			if (params.value.trim() === '') {
 				return [];
 			}
+
 			return params.value.split(',').map((item: string) => item.trim());
 		}
 

src/shared/transformer/string-to-boolean.transformer.spec.ts

@@ -6,10 +6,10 @@ describe('ToBooleanTransformer Decorator', () => {
 	describe('when transform a string to boolean', () => {
 		class WithBooleanDto {
 			@StringToBoolean()
-			booleanProp!: boolean;
+			public booleanProp!: boolean;
 
 			@StringToBoolean()
-			optionalBooleanProp?: boolean;
+			public optionalBooleanProp?: boolean;
 		}
 
 		it('should ignore undefined and null values', () => {

src/shared/transformer/string-to-number.transformer.spec.ts

@@ -5,7 +5,7 @@ describe('ToNumberTransformer Decorator', () => {
 	describe('when transform a string to number', () => {
 		class WithNumberDto {
 			@StringToNumber()
-			booleanProp!: boolean;
+			public booleanProp!: boolean;
 		}
 
 		it('should transform from string to number', () => {