From 44417543dd023f58bc00d19b5e4290824d31df5c Mon Sep 17 00:00:00 2001
From: obchain <167975049+obchain@users.noreply.github.com>
Date: Sat, 9 May 2026 07:32:48 +0530
Subject: [PATCH] test(proxy): drop env-leaking runtime_proxy assertion in
 clear test (#1362)

---
 src/openhuman/tools/impl/system/proxy_config_tests.rs | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/openhuman/tools/impl/system/proxy_config_tests.rs b/src/openhuman/tools/impl/system/proxy_config_tests.rs
index 73f0340ca..52314affc 100644
--- a/src/openhuman/tools/impl/system/proxy_config_tests.rs
+++ b/src/openhuman/tools/impl/system/proxy_config_tests.rs
@@ -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 ──────────────────────────────────────────────────