wininet: Relax a notification test.

Paul Vriens paul.vriens.wine at gmail.com
Thu Dec 18 10:23:18 CST 2008


Hans Leidekker wrote:
> diff --git a/dlls/wininet/tests/http.c b/dlls/wininet/tests/http.c
> index 6554680..8b3a8f8 100644
> --- a/dlls/wininet/tests/http.c
> +++ b/dlls/wininet/tests/http.c
> @@ -325,8 +325,8 @@ static void InternetReadFile_test(int flags)
>      SET_EXPECT2(INTERNET_STATUS_REQUEST_SENT, 2);
>      SET_EXPECT2(INTERNET_STATUS_RECEIVING_RESPONSE, 2);
>      SET_EXPECT2(INTERNET_STATUS_RESPONSE_RECEIVED, 2);
> -    SET_EXPECT2(INTERNET_STATUS_CLOSING_CONNECTION, 2);
> -    SET_EXPECT2(INTERNET_STATUS_CONNECTION_CLOSED, 2);
> +    SET_OPTIONAL2(INTERNET_STATUS_CLOSING_CONNECTION, 2);
> +    SET_OPTIONAL2(INTERNET_STATUS_CONNECTION_CLOSED, 2);
>      SET_EXPECT(INTERNET_STATUS_REDIRECT);
>      SET_OPTIONAL(INTERNET_STATUS_CONNECTING_TO_SERVER);
>      SET_OPTIONAL(INTERNET_STATUS_CONNECTED_TO_SERVER);
> 
> 
> 
Hi Hans,

Apparently that's not enough to fix the tests. Although it's set to be optional 
we are still using CHECK_NOTIFIED2 that doesn't make use of the optional parameter.

-- 
Cheers,

Paul.



More information about the wine-devel mailing list