admin/openhuman
1
0
Fork 0
mirror of https://github.com/tinyhumansai/openhuman.git synced 2026-07-27 21:08:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

test(proxy): drop env-leaking runtime_proxy assertion in clear test (#1362)

Browse Source
This commit is contained in:
obchain
2026-05-08 19:02:48 -07:00
committed by GitHub
parent 3953a13ecc
commit 44417543dd
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/openhuman/tools/impl/system/proxy_config_tests.rs
+4 -1
View File
@@ -102,8 +102,11 @@ async fn set_null_proxy_url_clears_existing_value() {
let get_result = tool.execute(json!({"action": "get"})).await.unwrap();
assert!(!get_result.is_error);
let parsed: Value = serde_json::from_str(&get_result.output()).unwrap();
// Only assert the *configured* proxy is cleared. `runtime_proxy.http_proxy`
// resolves through the process env (HTTP_PROXY / http_proxy) when the
// configured value is null, so on runners with those vars set the resolved
// field is non-null and unrelated to whether `set` cleared the config.
assert!(parsed["proxy"]["http_proxy"].is_null());
assert!(parsed["runtime_proxy"]["http_proxy"].is_null());
}
// ── parse_scope ──────────────────────────────────────────────────