pjandro/OmniRoute
1
0
Fork 0
mirror of https://github.com/diegosouzapw/OmniRoute.git synced 2026-07-31 20:32:20 +03:00
Code Issues Packages Projects Releases Wiki Activity

test(embeddings): pin seeded connection to direct egress in route-edge-coverage (#5975 collateral)

Browse Source 
#5975 made the embeddings service honor the connection-level proxy. The pre-existing
route-edge-coverage embeddings edge-case tests seed an openai connection while the
settings-proxy suite has left a provider-level proxy (provider.local:8080) in the shared
DATA_DIR that resetStorage() does not clear — inert before #5975, but now the leaked
proxy fast-fails the embedding upstream with PROXY_UNREACHABLE.

These tests do not exercise proxying, so seedOpenAIConnection now pins the connection to
proxyEnabled:false, making resolveProxyForConnection return a direct egress regardless of
leaked global proxyConfig. No assertions weakened; 16/16 in the file pass. Regression
surfaced by the concurrency=1 full-suite run; passes on #5975's parent, red after it.
This commit is contained in:
Diego Rodrigues de Sa e Souza
2026-07-03 16:13:38 -03:00
committed by Diego Rodrigues de Sa e Souza
parent 439f28d456
commit dbc0cdbe1b
1 changed files with 7 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
tests/unit/route-edge-coverage.test.ts
View File

@@ -84,6 +84,13 @@ async function seedOpenAIConnection({
errorCode: "refresh_failed",
rateLimitedUntil,
backoffLevel: 2,
// These embeddings edge-case tests do not exercise proxying. Pin the
// connection to a direct egress (proxy off) so resolveProxyForConnection
// returns "direct" regardless of any global/provider proxyConfig the
// settings-proxy suite left behind in the shared DATA_DIR. Without this,
// #5975 (embeddings now honor the connection-level proxy) makes the leaked
// provider.local:8080 fast-fail the upstream with PROXY_UNREACHABLE.
proxyEnabled: false,
});
}