[PATCH v2 0/1] MR9964: winhttp/tests: Add broken in test_WinHttpGetProxyForUrl.

[Bernhard Übelacker (＠bernhardu)]

Followup of: 421b5439

Windows 10 1607 and before return `ERROR_INVALID_PARAMETER`. This patch just adds a broken and leaves the function in that case.

[Test pattern page](https://test.winehq.org/data/patterns.html#winhttp:winhttp)

[Testbot run with this patch](https://testbot.winehq.org/JobDetails.pl?Key=161574)

CC: @TobiX

--
  v2: winhttp/tests: Add broken in test_WinHttpGetProxyForUrl.
 
https://gitlab.winehq.org/wine/wine/-/merge_requests/9964

[PATCH v2 1/1] winhttp/tests: Add broken in test_WinHttpGetProxyForUrl.

[Bernhard Übelacker]

From: Bernhard Übelacker <[e-mail]>

Followup of: 421b5439
---
 dlls/winhttp/tests/winhttp.c | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/dlls/winhttp/tests/winhttp.c b/dlls/winhttp/tests/winhttp.c
index 600fbd8f107..2b179871c76 100644
--- a/dlls/winhttp/tests/winhttp.c
+++ b/dlls/winhttp/tests/winhttp.c
@@ -5771,7 +5771,13 @@ static void test_WinHttpGetProxyForUrl(int port)
     options.dwFlags = WINHTTP_AUTOPROXY_ALLOW_STATIC|WINHTTP_AUTOPROXY_AUTO_DETECT;
     options.dwAutoDetectFlags = WINHTTP_AUTO_DETECT_TYPE_DHCP|WINHTTP_AUTO_DETECT_TYPE_DNS_A;
     ret = WinHttpGetProxyForUrl( session, L"http://winehq.com/", &options, &info);
-    ok(ret, "expected success\n" );
+    ok(ret || broken(GetLastError() == ERROR_INVALID_PARAMETER) /* <= Win10 1607 */, "expected success\n" );
+    if (!ret)
+    {
+        win_skip("WINHTTP_AUTOPROXY_ALLOW_STATIC not supported\n");
+        WinHttpCloseHandle( session );
+        return;
+    }
     ok(info.dwAccessType == WINHTTP_ACCESS_TYPE_NO_PROXY,
             "info.dwAccessType = %lu\n", info.dwAccessType);
     ok(!info.lpszProxy, "info.Proxy = %s\n", wine_dbgstr_w(info.lpszProxy));
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/9964

[Bernhard Übelacker (＠bernhardu)]

On Sun Jan 25 14:47:04 2026 +0000, Dmitry Timoshkov wrote:
> Should win_skip() be added in this case?
> Sorry for the delay and thanks for catching this, I guess it should be there, otherwise one cannot see from the logging if it succeeded or if it returned early.

I pushed a v2 adding this win_skip.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/9964#note_128306

[PATCH v2 0/1] MR9964: winhttp/tests: Add broken in test_WinHttpGetProxyForUrl. - approved

[Hans Leidekker (＠hans)]

This merge request was approved by Hans Leidekker.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/9964