[PATCH 1/5] iphlpapi/tests: Add tests for GetBestInterface.

[Jinoh Kang]

On 1/24/22 17:24, Huw Davies wrote:
> On Sun, Jan 16, 2022 at 01:32:31AM +0900, Jinoh Kang wrote:
>> Signed-off-by: Jinoh Kang <jinoh.kang.kr at gmail.com>
>> ---
>>  dlls/iphlpapi/tests/iphlpapi.c | 26 +++++++++++++++++++++++++-
>>  1 file changed, 25 insertions(+), 1 deletion(-)
>>
>> diff --git a/dlls/iphlpapi/tests/iphlpapi.c b/dlls/iphlpapi/tests/iphlpapi.c
>> index 12da629c30c..f9044c685c5 100644
>> --- a/dlls/iphlpapi/tests/iphlpapi.c
>> +++ b/dlls/iphlpapi/tests/iphlpapi.c
>> @@ -1456,9 +1456,32 @@ static void testGetNetworkParams(void)
>>      }
>>  }
> 
> Hi Jinoh,
> 
> Are you intending to work on the implementations of these functions

I'm willing to, but I'm actually stuck at how I could effectively test
for my changes. They are dependent on system configuration, and mocking
them would be a lot of work, I guess.

> or is this series all there is?

TL;DR: So yes, it's all there is.

> I ask because if you have patches for
> the impementations, it may be better to send them along with the tests.

I do have tests and implementation for GetBestRoute2.
Now that the code freeze is over, I'll send them over too.

> If these are just stand-alone tests I'll review them as is.

Thanks a lot!

> 
> Thanks,
> Huw.
> 


-- 
Sincerely,
Jinoh Kang