From 42b417aa20d829f1eaded40e3500373af904ee79 Mon Sep 17 00:00:00 2001
From: Casper Overholm Elkrog <support.open-source@elkrog.me>
Date: Sun, 16 Apr 2023 20:47:25 +0200
Subject: [PATCH] #208 Added simple validation for ensuring that times and
 interval cannot be negative values

---
 .../executor/executor-yargs-generator.js         | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/bin/execution/executor/executor-yargs-generator.js b/bin/execution/executor/executor-yargs-generator.js
index 503238e..7fed5ad 100644
--- a/bin/execution/executor/executor-yargs-generator.js
+++ b/bin/execution/executor/executor-yargs-generator.js
@@ -99,13 +99,25 @@ export default new class {
                             alias: 't',
                             describe: 'Number of times to run action (0 - infinite)',
                             type: 'number',
-                            default: 0
+                            default: 0,
+                            coerce: (value) => {
+                                if (value < 0) {
+                                    throw new Error('Option --times, -t must be positive value');
+                                }
+                                return value;
+                            }
                         })
                         .option('interval', {
                             alias: 'i',
                             describe: 'Interval between runs (seconds)',
                             type: 'number',
-                            demandOption: true
+                            demandOption: true,
+                            coerce: (value) => {
+                                if (value < 0) {
+                                    throw new Error('Option --interval, -i must be positive value');
+                                }
+                                return value;
+                            }
                         });
 
                     customOptionsYargsCreator.addToYargs(yargs, segment.actions)