Skip to content

Commit 969cf15

Browse files
authored
test: address review feedback in split api-proxy suites
Agent-Logs-Url: https://github.com/github/gh-aw-firewall/sessions/f7448a41-aca5-41bb-98a4-1519b774e020
1 parent d598295 commit 969cf15

2 files changed

Lines changed: 19 additions & 2 deletions

File tree

containers/api-proxy/server.proxy.test.js

Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,24 @@ const https = require('https');
1010
const tls = require('tls');
1111
const { EventEmitter } = require('events');
1212

13-
const { proxyRequest, proxyWebSocket } = require('./server');
13+
const originalHttpsProxy = process.env.HTTPS_PROXY;
14+
let proxyRequest;
15+
let proxyWebSocket;
16+
17+
beforeAll(() => {
18+
delete process.env.HTTPS_PROXY;
19+
jest.resetModules();
20+
({ proxyRequest, proxyWebSocket } = require('./server'));
21+
});
22+
23+
afterAll(() => {
24+
if (originalHttpsProxy === undefined) {
25+
delete process.env.HTTPS_PROXY;
26+
} else {
27+
process.env.HTTPS_PROXY = originalHttpsProxy;
28+
}
29+
jest.resetModules();
30+
});
1431

1532
// ── Helpers for proxyWebSocket tests ──────────────────────────────────────────
1633

containers/api-proxy/server.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ const https = require('https');
1212
const { EventEmitter } = require('events');
1313

1414
// Core proxy functions that remain in server.js
15-
const { proxyRequest, proxyWebSocket, httpProbe, validateApiKeys, keyValidationResults, resetKeyValidationState, fetchJson, extractModelIds, fetchStartupModels, reflectEndpoints, healthResponse, cachedModels, resetModelCacheState, makeModelBodyTransform, MODEL_ALIASES, buildModelsJson, writeModelsJson, createProviderServer, extractBillingHeaders } = require('./server');
15+
const { httpProbe, validateApiKeys, keyValidationResults, resetKeyValidationState, fetchJson, extractModelIds, fetchStartupModels, reflectEndpoints, healthResponse, cachedModels, resetModelCacheState, makeModelBodyTransform, MODEL_ALIASES, buildModelsJson, writeModelsJson, createProviderServer, extractBillingHeaders } = require('./server');
1616
const { composeBodyTransforms } = require('./proxy-utils');
1717
const { createCopilotAdapter } = require('./providers/copilot');
1818

0 commit comments

Comments
 (0)