test: make proxy tests runnable when there is a system proxy

Some client-proxy tests set HTTP_PROXY variable in the environment.
It's OK in general, however, in some specific cases, the user might
have already defined his own http_proxy setting in the environment.
The lowercase user setting takes precedence and leads to a test
failure. The fix cleans up the environment to avoid
an interaction between user and test proxy settings.

PR-URL: #61473
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>

Author: batrla

test/common/proxy-server.js

@@ -156,17 +156,22 @@ function spawnPromisified(...args) {
   });
 }
 
+function spawnOptions(envExtension) {
+  const env = { ...process.env };
+  // Cleanup the environment to avoid interference with client proxy tests.
+  for (const key of ['http_proxy', 'https_proxy', 'no_proxy']) {
+    delete env[key];
+    delete env[key.toUpperCase()];
+  }
+  return { env: { ...env, ...envExtension } };
+}
+
 async function checkProxied(type, envExtension, expectation, cliArgsExtension = []) {
   const script = type === 'fetch' ? fixtures.path('fetch-and-log.mjs') : fixtures.path('request-and-log.js');
   const { code, signal, stdout, stderr } = await spawnPromisified(
     process.execPath,
-    [...cliArgsExtension, script], {
-      env: {
-        NO_LOG_REQUEST: '1',
-        ...process.env,
-        ...envExtension,
-      },
-    });
+    [...cliArgsExtension, script],
+    spawnOptions({ ...envExtension, NO_LOG_REQUEST: '1' }));
 
   assert.deepStrictEqual({
     stderr: stderr.trim(),
@@ -193,24 +198,16 @@ exports.runProxiedRequest = async function(envExtension, cliArgsExtension = [])
   const fixtures = require('./fixtures');
   return spawnPromisified(
     process.execPath,
-    [...cliArgsExtension, fixtures.path('request-and-log.js')], {
-      env: {
-        ...process.env,
-        ...envExtension,
-      },
-    });
+    [...cliArgsExtension, fixtures.path('request-and-log.js')],
+    spawnOptions(envExtension));
 };
 
 exports.runProxiedPOST = async function(envExtension) {
   const fixtures = require('./fixtures');
   return spawnPromisified(
     process.execPath,
-    [fixtures.path('post-resource-and-log.js')], {
-      env: {
-        ...process.env,
-        ...envExtension,
-      },
-    });
+    [fixtures.path('post-resource-and-log.js')],
+    spawnOptions(envExtension));
 };
 
 exports.startTestServers = async function(options = {}) {

test/parallel/test-http2-allow-http1-upgrade-ws.js

@@ -25,7 +25,8 @@ const WebSocketServer = require('../common/websocket-server');
   await new Promise((resolve, reject) => {
     const ws = new WebSocket(`wss://localhost:${server.address().port}`, {
       dispatcher: new undici.EnvHttpProxyAgent({
-        connect: { rejectUnauthorized: false }
+        connect: { rejectUnauthorized: false },
+        noProxy: '*',
       })
     });
     ws.addEventListener('open', common.mustCall(() => {

test/parallel/test-inspector-network-fetch.js

@@ -18,6 +18,7 @@ undici.setGlobalDispatcher(new undici.EnvHttpProxyAgent({
   connect: {
     rejectUnauthorized: false,
   },
+  noProxy: '*',
 }));
 
 const session = new inspector.Session();