fix(env): Do not clobber defined 'env' script

If an env script is already defined, run that
instead of the default.

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

1 files changed, 65 insertions, 2 deletions

diff --git a/test/lib/run-script.js b/test/lib/run-script.js
index bad8a63c0..070f766b4 100644
--- a/test/lib/run-script.js
+++ b/test/lib/run-script.js
@@ -126,12 +126,14 @@ t.test('default env, start, and restart scripts', async t => {
         scriptShell: undefined,
         stdio: 'inherit',
         stdioString: true,
-        pkg: { name: 'x',
+        pkg: {
+          name: 'x',
           version: '1.2.3',
           _id: 'x@1.2.3',
           scripts: {
             env: 'env',
-          } },
+          },
+        },
         event: 'env',
       },
     ])
@@ -185,6 +187,67 @@ t.test('default env, start, and restart scripts', async t => {
   RUN_SCRIPTS.length = 0
 })
 
+t.test('non-default env script', async t => {
+  npm.localPrefix = t.testdir({
+    'package.json': JSON.stringify({
+      name: 'x',
+      version: '1.2.3',
+      scripts: {
+        env: 'hello',
+      },
+    }),
+  })
+
+  await runScript(['env'], er => {
+    if (er)
+      throw er
+
+    t.match(RUN_SCRIPTS, [
+      {
+        path: npm.localPrefix,
+        args: [],
+        scriptShell: undefined,
+        stdio: 'inherit',
+        stdioString: true,
+        pkg: {
+          name: 'x',
+          version: '1.2.3',
+          _id: 'x@1.2.3',
+          scripts: {
+            env: 'hello',
+          },
+        },
+        event: 'env',
+      },
+    ])
+  })
+  RUN_SCRIPTS.length = 0
+
+  await runScriptWin(['env'], er => {
+    if (er)
+      throw er
+
+    t.match(RUN_SCRIPTS, [
+      {
+        path: npm.localPrefix,
+        args: [],
+        scriptShell: undefined,
+        stdio: 'inherit',
+        stdioString: true,
+        pkg: { name: 'x',
+          version: '1.2.3',
+          _id: 'x@1.2.3',
+          scripts: {
+            env: 'hello',
+          },
+        },
+        event: 'env',
+      },
+    ])
+  })
+  RUN_SCRIPTS.length = 0
+})
+
 t.test('try to run missing script', t => {
   npm.localPrefix = t.testdir({
     'package.json': JSON.stringify({