hacky type casting for console args

apps/server/src/modules/deletion-console/deletion-execution.console.spec.ts

@@ -6,6 +6,7 @@ import { DeletionExecutionConsole } from './deletion-execution.console';
 import { DeletionExecutionTriggerResultBuilder, TriggerDeletionExecutionOptionsBuilder } from './builder';
 import { DeletionExecutionUc } from './uc';
 import { DeletionConsoleModule } from './deletion-console.app.module';
+import { TriggerDeletionExecutionOptions } from './interface';
 
 describe(DeletionExecutionConsole.name, () => {
 	let module: TestingModule;
@@ -92,5 +93,29 @@ describe(DeletionExecutionConsole.name, () => {
 				expect(spy).toHaveBeenCalledWith(err);
 			});
 		});
+
+		describe('when runFailed is given as string "true"', () => {
+			it('should convert runFailed to boolean', async () => {
+				const options = { limit: 1000, runFailed: 'true' } as unknown as TriggerDeletionExecutionOptions;
+
+				const spy = jest.spyOn(deletionExecutionUc, 'triggerDeletionExecution').mockResolvedValueOnce(undefined);
+
+				await console.triggerDeletionExecution(options);
+
+				expect(spy).toHaveBeenCalledWith(1000, true);
+			});
+		});
+
+		describe('when limit is a string "5"', () => {
+			it('should convert limit to number', async () => {
+				const options = { limit: '5', runFailed: false } as unknown as TriggerDeletionExecutionOptions;
+
+				const spy = jest.spyOn(deletionExecutionUc, 'triggerDeletionExecution').mockResolvedValueOnce(undefined);
+
+				await console.triggerDeletionExecution(options);
+
+				expect(spy).toHaveBeenCalledWith(5, false);
+			});
+		});
 	});
 });

apps/server/src/modules/deletion-console/deletion-execution.console.ts

@@ -14,26 +14,34 @@ export class DeletionExecutionConsole {
 		options: [
 			{
 				flags: '-l, --limit <value>',
-				/* istanbul ignore next */
-				fn: (value: string) => (value ? Number(value) : undefined), // NOSONAR
+				// TODO implemnt fn with proper command testing
+				// fn: (value: string) => (value ? Number(value) : undefined),
 				description: 'Limit of the requested deletion executions that should be performed.',
 				required: false,
 			},
 			{
 				flags: '-f, --runFailed <value>',
-				/* istanbul ignore next */
-				fn: (value: string) => /^(true|yes|1)$/i.test(value), // NOSONAR
+				// TODO implemnt fn with proper command testing
+				// fn: (value: string) => /^(true|yes|1)$/i.test(value),
 				description: 'Limit of the requested deletion executions that should be performed.',
 				required: false,
 			},
 		],
 	})
-	async triggerDeletionExecution(options: TriggerDeletionExecutionOptions): Promise<void> {
+	public async triggerDeletionExecution(options: TriggerDeletionExecutionOptions): Promise<void> {
 		// Try to trigger the deletion execution(s) via Deletion API client,
 		// return successful status in case of a success, otherwise return
 		// a result with a failure status and a proper error message.
 		let result: DeletionExecutionTriggerResult;
 
+		if (typeof options.limit === 'string') {
+			options.limit = Number(options.limit);
+		}
+
+		if (typeof options.runFailed === 'string') {
+			options.runFailed = /^(true|yes|1)$/i.test(options.runFailed);
+		}
+
 		try {
 			await this.deletionExecutionUc.triggerDeletionExecution(options.limit, options.runFailed);
 			result = DeletionExecutionTriggerResultBuilder.buildSuccess();

apps/server/src/modules/deletion-console/interface/trigger-deletion-execution-options.interface.ts

@@ -1,4 +1,4 @@
 export interface TriggerDeletionExecutionOptions {
-	limit: number;
-	runFailed: boolean;
+	limit: string | number;
+	runFailed: string | boolean;
 }