fix: TypeError: Agent is not a constructor. This happens in the tests due to new https-proxy-agent which changed its default export from a constructor to an object containing a named export

Author: GautamSharda

packages/teeny-request/src/agents.ts

@@ -83,10 +83,16 @@ export function getAgent(
 
   if (proxy && shouldUseProxy) {
     // tslint:disable-next-line variable-name
-    const Agent = isHttp
+    let Agent = isHttp
       ? require('http-proxy-agent')
       : require('https-proxy-agent');
 
+    if (Agent.HttpsProxyAgent) {
+      Agent = Agent.HttpsProxyAgent;
+    } else if (Agent.HttpProxyAgent) {
+      Agent = Agent.HttpProxyAgent;
+    }
+
     const proxyOpts = {...parse(proxy), ...poolOptions};
     return new Agent(proxyOpts);
   }

packages/teeny-request/test/agents.ts

@@ -23,9 +23,16 @@ import * as sinon from 'sinon';
 import {getAgent, pool} from '../src/agents';
 
 // eslint-disable-next-line @typescript-eslint/no-var-requires
-const HttpProxyAgent = require('http-proxy-agent');
+let HttpProxyAgent = require('http-proxy-agent');
 // eslint-disable-next-line @typescript-eslint/no-var-requires
-const HttpsProxyAgent = require('https-proxy-agent');
+let HttpsProxyAgent = require('https-proxy-agent');
+
+if (HttpProxyAgent.HttpProxyAgent) {
+  HttpProxyAgent = HttpProxyAgent.HttpProxyAgent;
+}
+if (HttpsProxyAgent.HttpsProxyAgent) {
+  HttpsProxyAgent = HttpsProxyAgent.HttpsProxyAgent;
+}
 
 describe('agents', () => {
   const httpUri = 'http://example.com';
@@ -91,7 +98,7 @@ describe('agents', () => {
         const proxy = 'https://hello.there:8080';
         const proxyExpected = {
           hostname: 'hello.there',
-          port: 8080,
+          port: '8080',
           protocol: 'https:',
         };