[PATCH 1/3] winhttp/tests: Add timeout tests for WinHttp{Set, Query}Option

Matijn Woudt tijnema at gmail.com
Fri Jan 29 09:23:13 CST 2010


-------------- next part --------------
From 8e29a8ead3935060db18bcfdb3722aa9d63aa897 Mon Sep 17 00:00:00 2001
From: Matijn Woudt <tijnema at gmail.com>
Date: Fri, 29 Jan 2010 01:40:00 +0100
Subject: [PATCH 1/3] winhttp/tests: Add timeout tests for WinHttp{Set,Query}Option

Tests for WINHTTP_OPTION_{CONNECT,SEND,RECEIVE}_TIMEOUT
---
 dlls/winhttp/tests/winhttp.c |  369 ++++++++++++++++++++++++++++++++++++++++++
 1 files changed, 369 insertions(+), 0 deletions(-)

diff --git a/dlls/winhttp/tests/winhttp.c b/dlls/winhttp/tests/winhttp.c
index bddf871..b1f213c 100644
--- a/dlls/winhttp/tests/winhttp.c
+++ b/dlls/winhttp/tests/winhttp.c
@@ -1034,6 +1034,7 @@ static void test_set_default_proxy_config(void)
 static void test_Timeouts (void)
 {
     BOOL ret;
+    DWORD value, size;
     HINTERNET ses, req, con;
     static const WCHAR codeweavers[] = {'c','o','d','e','w','e','a','v','e','r','s','.','c','o','m',0};
 
@@ -1069,9 +1070,128 @@ static void test_Timeouts (void)
     ret = WinHttpSetTimeouts(ses, 0, 0, 0, 0);
     todo_wine ok(ret, "%u\n", GetLastError());
 
+    SetLastError(0xdeadbeef);
+    ret = WinHttpSetTimeouts(ses, 0x0123, 0x4567, 0x89ab, 0xcdef);
+    todo_wine ok(ret, "%u\n", GetLastError());
+
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(ses, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0x4567, "Expected 0x4567, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(ses, WINHTTP_OPTION_SEND_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0x89ab, "Expected 0x89ab, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(ses, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xcdef, "Expected 0xcdef, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0;
+    ret = WinHttpSetOption(ses, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, sizeof(value));
+    todo_wine ok(ret, "%u\n", GetLastError());
+
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(ses, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0, "Expected 0, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0;
+    ret = WinHttpSetOption(ses, WINHTTP_OPTION_SEND_TIMEOUT, &value, sizeof(value));
+    todo_wine ok(ret, "%u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(ses, WINHTTP_OPTION_SEND_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0, "Expected 0, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0;
+    ret = WinHttpSetOption(ses, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, sizeof(value));
+    todo_wine ok(ret, "%u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(ses, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0, "Expected 0, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xbeefdead;
+    ret = WinHttpSetOption(ses, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, sizeof(value));
+    todo_wine ok(ret, "%u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(ses, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xbeefdead, "Expected 0xbeefdead, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xbeefdead;
+    ret = WinHttpSetOption(ses, WINHTTP_OPTION_SEND_TIMEOUT, &value, sizeof(value));
+    todo_wine ok(ret, "%u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(ses, WINHTTP_OPTION_SEND_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xbeefdead, "Expected 0xbeefdead, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xbeefdead;
+    ret = WinHttpSetOption(ses, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, sizeof(value));
+    todo_wine ok(ret, "%u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(ses, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xbeefdead, "Expected 0xbeefdead, got %u\n", value);
+
     con = WinHttpConnect(ses, codeweavers, 0, 0);
     ok(con != NULL, "failed to open a connection %u\n", GetLastError());
 
+    /* Timeout values should match the last one set for session */
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(con, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xbeefdead, "Expected 0xbeefdead, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(con, WINHTTP_OPTION_SEND_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xbeefdead, "Expected 0xbeefdead, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(con, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xbeefdead, "Expected 0xbeefdead, got %u\n", value);
+
     SetLastError(0xdeadbeef);
     ret = WinHttpSetTimeouts(con, -2, 0, 0, 0);
     ok(!ret && GetLastError() == ERROR_INVALID_PARAMETER,
@@ -1101,10 +1221,87 @@ static void test_Timeouts (void)
     ret = WinHttpSetTimeouts(con, 0, 0, 0, 0);
     ok(!ret && GetLastError() == ERROR_WINHTTP_INCORRECT_HANDLE_TYPE,
        "expected ERROR_WINHTTP_INVALID_TYPE, got %u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0;
+    ret = WinHttpSetOption(con, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, sizeof(value));
+    ok(!ret && GetLastError() == ERROR_WINHTTP_INCORRECT_HANDLE_TYPE,
+       "expected ERROR_WINHTTP_INVALID_TYPE, got %u\n", GetLastError());
+
+    SetLastError(0xdeadbeef);
+    value = 0;
+    ret = WinHttpSetOption(con, WINHTTP_OPTION_SEND_TIMEOUT, &value, sizeof(value));
+    ok(!ret && GetLastError() == ERROR_WINHTTP_INCORRECT_HANDLE_TYPE,
+       "expected ERROR_WINHTTP_INVALID_TYPE, got %u\n", GetLastError());
+
+    SetLastError(0xdeadbeef);
+    value = 0;
+    ret = WinHttpSetOption(con, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, sizeof(value));
+    ok(!ret && GetLastError() == ERROR_WINHTTP_INCORRECT_HANDLE_TYPE,
+       "expected ERROR_WINHTTP_INVALID_TYPE, got %u\n", GetLastError());
+
+    /* Changing timeout values for session should affect the values for connection */
+    SetLastError(0xdeadbeef);
+    value = 0xdead;
+    ret = WinHttpSetOption(ses, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, sizeof(value));
+    todo_wine ok(ret, "%u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(con, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xdead, "Expected 0xdead, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xdead;
+    ret = WinHttpSetOption(ses, WINHTTP_OPTION_SEND_TIMEOUT, &value, sizeof(value));
+    todo_wine ok(ret, "%u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(con, WINHTTP_OPTION_SEND_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xdead, "Expected 0xdead, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xdead;
+    ret = WinHttpSetOption(ses, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, sizeof(value));
+    todo_wine ok(ret, "%u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(con, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xdead, "Expected 0xdead, got %u\n", value);
 
     req = WinHttpOpenRequest(con, NULL, NULL, NULL, NULL, NULL, 0);
     ok(req != NULL, "failed to open a request %u\n", GetLastError());
 
+    /* Timeout values should match the last one set for session */
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(req, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xdead, "Expected 0xdead, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(req, WINHTTP_OPTION_SEND_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xdead, "Expected 0xdead, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(req, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xdead, "Expected 0xdead, got %u\n", value);
+
     SetLastError(0xdeadbeef);
     ret = WinHttpSetTimeouts(req, -2, 0, 0, 0);
     ok(!ret && GetLastError() == ERROR_INVALID_PARAMETER,
@@ -1133,6 +1330,178 @@ static void test_Timeouts (void)
     ret = WinHttpSetTimeouts(req, 0, 0, 0, 0);
     ok(ret, "%u\n", GetLastError());
 
+    SetLastError(0xdeadbeef);
+    ret = WinHttpSetTimeouts(req, 0xcdef, 0x89ab, 0x4567, 0x0123);
+    ok(ret, "%u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(req, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0x89ab, "Expected 0x89ab, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(req, WINHTTP_OPTION_SEND_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0x4567, "Expected 0x4567, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(req, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0x0123, "Expected 0x0123, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0;
+    ret = WinHttpSetOption(req, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, sizeof(value));
+    ok(ret, "%u\n", GetLastError());
+
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(req, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0, "Expected 0, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0;
+    ret = WinHttpSetOption(req, WINHTTP_OPTION_SEND_TIMEOUT, &value, sizeof(value));
+    ok(ret, "%u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(req, WINHTTP_OPTION_SEND_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0, "Expected 0, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0;
+    ret = WinHttpSetOption(req, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, sizeof(value));
+    ok(ret, "%u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(req, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0, "Expected 0, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xbeefdead;
+    ret = WinHttpSetOption(req, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, sizeof(value));
+    ok(ret, "%u\n", GetLastError());
+
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(req, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xbeefdead, "Expected 0xbeefdead, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xbeefdead;
+    ret = WinHttpSetOption(req, WINHTTP_OPTION_SEND_TIMEOUT, &value, sizeof(value));
+    ok(ret, "%u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(req, WINHTTP_OPTION_SEND_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xbeefdead, "Expected 0xbeefdead, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xbeefdead;
+    ret = WinHttpSetOption(req, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, sizeof(value));
+    ok(ret, "%u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(req, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xbeefdead, "Expected 0xbeefdead, got %u\n", value);
+
+    /* Changing timeout values for session should not affect the values for a request,
+     * neither should the other way around.
+     */
+    SetLastError(0xdeadbeef);
+    value = 0xbeefdead;
+    ret = WinHttpSetOption(req, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, sizeof(value));
+    ok(ret, "%u\n", GetLastError());
+
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(ses, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xdead, "Expected 0xdead, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xbeefdead;
+    ret = WinHttpSetOption(req, WINHTTP_OPTION_SEND_TIMEOUT, &value, sizeof(value));
+    ok(ret, "%u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(ses, WINHTTP_OPTION_SEND_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xdead, "Expected 0xdead, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xbeefdead;
+    ret = WinHttpSetOption(req, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, sizeof(value));
+    ok(ret, "%u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(ses, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xdead, "Expected 0xdead, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xbeef;
+    ret = WinHttpSetOption(ses, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, sizeof(value));
+    todo_wine ok(ret, "%u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(req, WINHTTP_OPTION_CONNECT_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xbeefdead, "Expected 0xbeefdead, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xbeef;
+    ret = WinHttpSetOption(ses, WINHTTP_OPTION_SEND_TIMEOUT, &value, sizeof(value));
+    todo_wine ok(ret, "%u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(req, WINHTTP_OPTION_SEND_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xbeefdead, "Expected 0xbeefdead, got %u\n", value);
+
+    SetLastError(0xdeadbeef);
+    value = 0xbeef;
+    ret = WinHttpSetOption(ses, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, sizeof(value));
+    todo_wine ok(ret, "%u\n", GetLastError());
+    
+    SetLastError(0xdeadbeef);
+    value = 0xdeadbeef;
+    size  = sizeof(DWORD);
+    ret = WinHttpQueryOption(req, WINHTTP_OPTION_RECEIVE_TIMEOUT, &value, &size);
+    todo_wine ok(ret, "%u\n", GetLastError());
+    todo_wine ok(value == 0xbeefdead, "Expected 0xbeefdead, got %u\n", value);
+
     WinHttpCloseHandle(req);
     WinHttpCloseHandle(con);
     WinHttpCloseHandle(ses);
-- 
1.6.3.3


More information about the wine-patches mailing list