Re: [PATCH] winhttp/tests: Added multi Authenticate header test

[Hans Leidekker]

On Tue, 2017-04-04 at 23:39 +0000, Alistair Leslie-Hughes wrote:
> +static void test_multi_authentication(int port)
> +{
> +    static const WCHAR multiauthW[] = {'/','m','u','l','t','i','a','u','t','h',0};
> +    static const WCHAR getW[] = {'G','E','T',0};
> +    HINTERNET ses, con, req;
> +    DWORD supported, first, target = 8;
> +    BOOL ret;
> +
> +    ses = WinHttpOpen(test_useragent, WINHTTP_ACCESS_TYPE_NO_PROXY, NULL, NULL, 0);
> +    ok(ses != NULL, "failed to open session %u\n", GetLastError());
> +
> +    con = WinHttpConnect(ses, localhostW, port, 0);
> +    ok(con != NULL, "failed to open a connection %u\n", GetLastError());
> +
> +    req = WinHttpOpenRequest(con, getW, multiauthW, NULL, NULL, NULL, 0);
> +    ok(req != NULL, "failed to open a request %u\n", GetLastError());
> +
> +    ret = WinHttpSendRequest(req, WINHTTP_NO_ADDITIONAL_HEADERS, 0,
> +                             WINHTTP_NO_REQUEST_DATA,0, 0, 0 );
> +    ok(ret, "expected success\n");
> +
> +    ret = WinHttpReceiveResponse(req, NULL);
> +    ok(ret, "expected success\n");
> +
> +    ret = WinHttpQueryAuthSchemes(req, &supported, &first, &target);
> +    todo_wine ok(ret, "expected success\n");
> +    todo_wine ok(supported == (WINHTTP_AUTH_SCHEME_BASIC | WINHTTP_AUTH_SCHEME_NTLM), "got %x\n", supported);
> +    todo_wine ok(target == WINHTTP_AUTH_TARGET_SERVER, "got %x\n", target);

You need to initialize 'supported'. Maybe test 'first' too?