Andrew Eikum : wininet: Add Internet{Set, Query}Option tests, implement POLICY option.

Alexandre Julliard julliard at winehq.org
Fri Jan 22 08:26:49 CST 2010


Module: wine
Branch: master
Commit: deca2e5c0ed1ec5eecae4426d871918841b03065
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=deca2e5c0ed1ec5eecae4426d871918841b03065

Author: Andrew Eikum <aeikum at codeweavers.com>
Date:   Wed Jan 20 14:46:44 2010 -0600

wininet: Add Internet{Set,Query}Option tests, implement POLICY option.

---

 dlls/wininet/internet.c       |    6 ++++++
 dlls/wininet/tests/internet.c |   26 ++++++++++++++++++++++++++
 2 files changed, 32 insertions(+), 0 deletions(-)

diff --git a/dlls/wininet/internet.c b/dlls/wininet/internet.c
index 969c079..074bf79 100644
--- a/dlls/wininet/internet.c
+++ b/dlls/wininet/internet.c
@@ -2170,6 +2170,8 @@ DWORD INET_QueryOption(DWORD option, void *buffer, DWORD *size, BOOL unicode)
     }
     case INTERNET_OPTION_USER_AGENT:
         return ERROR_INTERNET_INCORRECT_HANDLE_TYPE;
+    case INTERNET_OPTION_POLICY:
+        return ERROR_INVALID_PARAMETER;
     }
 
     FIXME("Stub for %d\n", option);
@@ -2401,6 +2403,10 @@ BOOL WINAPI InternetSetOptionW(HINTERNET hInternet, DWORD dwOption,
         SetLastError(ERROR_INTERNET_INVALID_OPTION);
         ret = FALSE;
         break;
+    case INTERNET_OPTION_POLICY:
+        SetLastError(ERROR_INVALID_PARAMETER);
+        ret = FALSE;
+        break;
     default:
         FIXME("Option %d STUB\n",dwOption);
         SetLastError(ERROR_INTERNET_INVALID_OPTION);
diff --git a/dlls/wininet/tests/internet.c b/dlls/wininet/tests/internet.c
index 622711a..8708878 100644
--- a/dlls/wininet/tests/internet.c
+++ b/dlls/wininet/tests/internet.c
@@ -824,6 +824,31 @@ static void test_PrivacyGetSetZonePreferenceW(void)
     ok(ret == 0, "expected ret == 0, got %u\n", ret);
 }
 
+static void test_Option_Policy(void)
+{
+    HINTERNET hinet;
+    BOOL ret;
+
+    hinet = InternetOpen(NULL, INTERNET_OPEN_TYPE_DIRECT, NULL, NULL, 0);
+    ok(hinet != 0, "InternetOpen failed: 0x%08x\n", GetLastError());
+
+    SetLastError(0xdeadbeef);
+    ret = InternetSetOptionW(hinet, INTERNET_OPTION_POLICY, NULL, 0);
+    ok(ret == FALSE, "InternetSetOption should've failed\n");
+    ok(GetLastError() == ERROR_INVALID_PARAMETER, "GetLastError should've "
+            "given ERROR_INVALID_PARAMETER, gave: 0x%08x\n", GetLastError());
+
+    SetLastError(0xdeadbeef);
+    ret = InternetQueryOptionW(hinet, INTERNET_OPTION_POLICY, NULL, 0);
+    ok(ret == FALSE, "InternetQueryOption should've failed\n");
+    ok(GetLastError() == ERROR_INVALID_PARAMETER, "GetLastError should've "
+            "given ERROR_INVALID_PARAMETER, gave: 0x%08x\n", GetLastError());
+
+    ret = InternetCloseHandle(hinet);
+    ok(ret == TRUE, "InternetCloseHandle failed: 0x%08x\n", GetLastError());
+}
+
+
 /* ############################### */
 
 START_TEST(internet)
@@ -846,6 +871,7 @@ START_TEST(internet)
     test_complicated_cookie();
     test_version();
     test_null();
+    test_Option_Policy();
 
     if (!pInternetTimeFromSystemTimeA)
         win_skip("skipping the InternetTime tests\n");




More information about the wine-cvs mailing list