Properly set npm_command environment variable.

Fix: #2015

PR-URL: https://github.com/npm/cli/pull/2016
Credit: @isaacs
Close: #2016
Reviewed-by: @ruyadorno

1 files changed, 9 insertions, 0 deletions

diff --git a/test/lib/npm.js b/test/lib/npm.js
index 296817849..b2be5543f 100644
--- a/test/lib/npm.js
+++ b/test/lib/npm.js
@@ -4,7 +4,16 @@ const fs = require('fs')
 
 // delete this so that we don't have configs from the fact that it
 // is being run by 'npm test'
+const event = process.env.npm_lifecycle_event
 for (const env of Object.keys(process.env).filter(e => /^npm_/.test(e))) {
+  if (env === 'npm_command') {
+    // should only be running this in the 'test' or 'run-script' command!
+    // if the lifecycle event is 'test', then it'll be 'test', otherwise
+    // it should always be run-script.  Of course, it'll be missing if this
+    // test is just run directly, which is also acceptable.
+    const cmd = event === 'test' ? 'test' : 'run-script'
+    t.match(process.env[env], cmd)
+  }
   delete process.env[env]
 }