test: add e2e tests for --update-env option

test/e2e/cli/env-refresh.sh

@@ -88,3 +88,98 @@ $pm2 kill
 $pm2 l
 NODE_PATH='/test2' $pm2 start local_require.js -i 1
 should 'should have loaded the right globalPaths' 'restart_time: 0' 1
+
+#
+# Ensuring that environment update works correctly when reloading with JSON config.
+#
+# Related issue:
+# https://github.com/Unitech/pm2/issues/3192
+#
+
+# start with config
+SH=shell_initial SH_PM=shell_initial $pm2 start update-env.config.js --env initial
+>out-env.log
+
+sleep 0.5
+grep "SH=shell_initial PM=pm2_initial SH_PM=pm2_initial" out-env.log &> /dev/null
+spec "should inject shell environment, then inject config environment on start with config"
+
+# restart config without --update-env
+$pm2 delete all
+SH=shell_initial SH_PM=shell_initial $pm2 start update-env.config.js --env initial
+SH=shell_updated SH_PM=shell_updated $pm2 restart update-env.config.js --env updated
+>out-env.log
+
+sleep 0.5
+grep "SH=shell_updated PM=pm2_updated SH_PM=pm2_updated" out-env.log &> /dev/null
+spec "should inject shell environment, then inject config environment on restart with config and without --update-env option"
+
+# reload config without --update-env
+$pm2 delete all
+SH=shell_initial SH_PM=shell_initial $pm2 start update-env.config.js --env initial
+SH=shell_updated SH_PM=shell_updated $pm2 reload update-env.config.js --env updated
+>out-env.log
+
+sleep 0.5
+grep "SH=shell_updated PM=pm2_updated SH_PM=pm2_updated" out-env.log &> /dev/null
+spec "should inject shell environment, then inject config environment on reload with config and without --update-env option"
+
+# restart config with --update-env
+$pm2 delete all
+SH=shell_initial SH_PM=shell_initial $pm2 start update-env.config.js --env initial
+SH=shell_updated SH_PM=shell_updated $pm2 restart update-env.config.js --env updated --update-env
+>out-env.log
+
+sleep 0.5
+grep "SH=shell_updated PM=pm2_updated SH_PM=pm2_updated" out-env.log &> /dev/null
+spec "should inject shell environment, then inject config environment on restart with config and with --update-env option"
+
+# reload config with --update-env
+$pm2 delete all
+SH=shell_initial SH_PM=shell_initial $pm2 start update-env.config.js --env initial
+SH=shell_updated SH_PM=shell_updated $pm2 reload update-env.config.js --env updated --update-env
+>out-env.log
+
+sleep 0.5
+grep "SH=shell_updated PM=pm2_updated SH_PM=pm2_updated" out-env.log &> /dev/null
+spec "should inject shell environment, then inject config environment on reload with config and with --update-env option"
+
+# restart pid without --update-env
+$pm2 delete all
+SH=shell_initial SH_PM=shell_initial $pm2 start update-env.config.js --env initial
+SH=shell_updated SH_PM=shell_updated $pm2 restart update_env_app
+>out-env.log
+
+sleep 0.5
+grep "SH=shell_initial PM=pm2_initial SH_PM=pm2_initial" out-env.log &> /dev/null
+spec "should keep environment on restart with pid and without --update-env option"
+
+# reload pid without --update-env
+$pm2 delete all
+SH=shell_initial SH_PM=shell_initial $pm2 start update-env.config.js --env initial
+SH=shell_updated SH_PM=shell_updated $pm2 reload update_env_app
+>out-env.log
+
+sleep 0.5
+grep "SH=shell_initial PM=pm2_initial SH_PM=pm2_initial" out-env.log &> /dev/null
+spec "should keep environment on reload with pid and without --update-env option"
+
+# restart pid with --update-env
+$pm2 delete all
+SH=shell_initial SH_PM=shell_initial $pm2 start update-env.config.js --env initial
+SH=shell_updated SH_PM=shell_updated $pm2 restart update_env_app --update-env
+>out-env.log
+
+sleep 0.5
+grep "SH=shell_updated PM=pm2_initial SH_PM=shell_updated" out-env.log &> /dev/null
+spec "should inject shell environment on restart with pid and with --update-env option"
+
+# reload pid with --update-env
+$pm2 delete all
+SH=shell_initial SH_PM=shell_initial $pm2 start update-env.config.js --env initial
+SH=shell_updated SH_PM=shell_updated $pm2 reload update_env_app --update-env
+>out-env.log
+
+sleep 0.5
+grep "SH=shell_updated PM=pm2_initial SH_PM=shell_updated" out-env.log &> /dev/null
+spec "should inject shell environment on reload with pid and with --update-env option"

test/fixtures/update-env.config.js

@@ -0,0 +1,22 @@
+module.exports = {
+  apps: [
+    {
+      name: 'update_env_app',
+      script: './update-env.js',
+      instances: 2,
+      exec_mode: 'cluster',
+      out_file: 'out-env.log',
+      merge_logs: true,
+      env_initial: {
+        NODE_ENV: 'test',
+        PM: 'pm2_initial',
+        SH_PM: 'pm2_initial',
+      },
+      env_updated: {
+        NODE_ENV: 'test',
+        PM: 'pm2_updated',
+        SH_PM: 'pm2_updated',
+      },
+    },
+  ],
+};

test/fixtures/update-env.js

@@ -0,0 +1,5 @@
+setInterval(() => {
+  const { SH, PM, SH_PM } = process.env;
+
+  console.log(`SH=${SH} PM=${PM} SH_PM=${SH_PM}`);
+}, 100);

test/programmatic/env_switching.js

@@ -131,7 +131,7 @@ describe('PM2 programmatic calls', function() {
   });
 
   /**
-   * Ensuring that environment update works correct when reloading with JSON config.
+   * Ensuring that environment update works correctly when reloading with JSON config.
    *
    * Related issue:
    * https://github.com/Unitech/pm2/issues/3192