feat(cli): expose --variables flag in fargate (#2391)

* feat(cli): expose --variables flag in fargate * refactor(cli): move --variables to common flags
artilleryio · Jan 2, 2024 · 7f1d779 · 7f1d779
1 parent aec0b89
commit 7f1d779
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 10 deletions.
diff --git a/packages/artillery/lib/cli/common-flags.js b/packages/artillery/lib/cli/common-flags.js
@@ -25,6 +25,11 @@ const CommonRunFlags = {
   dotenv: Flags.string({
     description: 'Path to a dotenv file to load environment variables from'
   }),
+  variables: Flags.string({
+    char: 'v',
+    description:
+      'Set variables available to vusers during the test; a JSON object'
+  }),
   overrides: Flags.string({
     description: 'Dynamically override values in the test script; a JSON object'
   }),

diff --git a/packages/artillery/lib/cmds/run-lambda.js b/packages/artillery/lib/cmds/run-lambda.js
@@ -52,11 +52,6 @@ Examples:
 `;
 RunLambdaCommand.flags = {
   ...CommonRunFlags,
-  variables: Flags.string({
-    char: 'v',
-    description:
-      'Set variables available to vusers during the test; a JSON object'
-  }),
   payload: Flags.string({
     char: 'p',
     description: 'Specify a CSV file for dynamic data'

diff --git a/packages/artillery/lib/cmds/run.js b/packages/artillery/lib/cmds/run.js
@@ -80,11 +80,6 @@ RunCommand.flags = {
     char: 'q',
     description: 'Quiet mode'
   }),
-  variables: Flags.string({
-    char: 'v',
-    description:
-      'Set variables available to vusers during the test; a JSON object'
-  }),
   // TODO: Deprecation notices for commands below:
   payload: Flags.string({
     char: 'p',

diff --git a/packages/artillery/lib/platform/aws-ecs/legacy/run-cluster.js b/packages/artillery/lib/platform/aws-ecs/legacy/run-cluster.js
@@ -1365,6 +1365,9 @@ async function generateTaskOverrides(context) {
     context.cliOptions.overrides
       ? ['--overrides', context.cliOptions.overrides]
       : [],
+    context.cliOptions.variables
+      ? ['--variables', context.cliOptions.variables]
+      : [],
     context.configPath ? ['--config', context.configPath] : []
   );
   // NOTE: This MUST come last: