fix: bundle dotenv files on Azure

Remove explicit check for AWS Lambda from BOM so that dotenv files get bundled by default on all platforms other than AWS ECS.
artilleryio · Aug 19, 2024 · 6e3c5d1 · 6e3c5d1
1 parent 99a7df7
commit 6e3c5d1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/packages/artillery/lib/cmds/run.js b/packages/artillery/lib/cmds/run.js
@@ -44,7 +44,7 @@ class RunCommand extends Command {
   async run() {
     const { flags, argv, args } = await this.parse(RunCommand);
 
-    if (flags.platform === 'aws:fargate') {
+    if (flags.platform === 'aws:ecs') {
       // Delegate to existing implementation
       const RunFargateCommand = require('./run-fargate');
       return await RunFargateCommand.run(argv);

diff --git a/packages/artillery/lib/platform/aws-ecs/legacy/bom.js b/packages/artillery/lib/platform/aws-ecs/legacy/bom.js
@@ -379,8 +379,7 @@ function getExtraFiles(context, next) {
 
 function getDotEnv(context, next) {
   const flags = context.opts.flags;
-  //TODO: For now only enabled on Lambda. Enable this for Fargate after refactoring to allow for it
-  if (!flags.dotenv || flags.platform !== 'aws:lambda') {
+  if (!flags.dotenv || flags.platform === 'aws:ecs') {
     return next(null, context);
   }