[PATCH 3/3] wininet/test: Improve tests for InternetGetConnectedStateEx

Bruno Jesus 00cpxxx at gmail.com
Mon Mar 6 08:32:19 CST 2017


On Mon, Mar 6, 2017 at 11:24 AM, Jacek Caban <jacek at codeweavers.com> wrote:
> Hi Bruno,

Hi, thanks for the review ;-)

> On 06.03.2017 05:24, Bruno Jesus wrote:
>> -    ok(sz - 1 == lstrlenW(buffer), "Expected %u bytes, got %u\n", sz - 1, lstrlenW(buffer));
>> +    if (flags & INTERNET_CONNECTION_MODEM)
>> +        ok(lstrlenW(buffer) == 0, "Expected 0 bytes, got %u\n", lstrlenW(buffer));
>> +    else
>> +        ok(sz - 1 == lstrlenW(buffer), "Expected %u bytes, got %u\n", sz - 1, lstrlenW(buffer));
>
>
> You're dropping sz value tests in case is set. What is it set to in this
> case? 0?
>
> As a side note, you don't need strlen() to check if buffer is empty,
> something like !*buffer will do the trick.

You are right, I will make the change to test both !buffer[0] and sz
== 0 in the three cases.



More information about the wine-devel mailing list