Re-enable socks timeout test

TooTallNate · May 4, 2023 · 2ce0d20 · 2ce0d20
1 parent 9e84559
commit 2ce0d20
Showing 1 changed file with 4 additions and 10 deletions.
diff --git a/packages/socks-proxy-agent/test/test.ts b/packages/socks-proxy-agent/test/test.ts
@@ -75,23 +75,17 @@ describe('SocksProxyAgent', () => {
 			assert.equal(socksServerUrl.hostname, agent.proxy.host);
 			assert.equal(+socksServerUrl.port, agent.proxy.port);
 		});
-		it.skip('setup timeout', async () => {
-			httpServer.once('request', function (req, res) {
-				assert.equal('/timeout', req.url);
-				res.statusCode = 200;
-				setTimeout(() => res.end('Written after 500'), 500);
-			});
-
-			const agent = new SocksProxyAgent(socksServerUrl, { timeout: 50 });
+		it('should respect `timeout` option during connection to socks server', async () => {
+			const agent = new SocksProxyAgent(socksServerUrl, { timeout: 1 });
 
 			let err: Error | undefined;
 			try {
-				await req(new URL('/timeout', httpServerUrl), { agent });
+				await req('http://example.com', { agent });
 			} catch (_err) {
 				err = _err as Error;
 			}
 			assert(err);
-			assert.equal(err.message, 'socket hang up');
+			assert.equal(err.message, 'Proxy connection timed out');
 		});
 	});