Re: kernel32: add a test for CreateFileA()

[Nikolay Sivov]

On 7/18/2010 10:17, Austin English wrote:
> Passed all the vm's on wtb.

Hi, Austin. This looks strange:
> -
> - hFile = CreateFileA("c:\*.*", GENERIC_READ,FILE_SHARE_READ,NULL,OPEN_EXISTING,FILE_ATTRIBUTE_NORMAL,NULL );
> - ok(GetLastError() == ERROR_INVALID_NAME || broken(GetLastError() == ERROR_FILE_NOT_FOUND) || broken(GetLastError() == ERROR_PATH_NOT_FOUND),
> -
>   "CreateFileA should have returned ERROR_INVALID_NAME on %s, but got %u\n", filename, GetLastError());
> - CloseHandle( hFile );

If a call failed then you should test for null hFile I suppose and 
remove CloseHandle().  Also when testing for last error a common rule is 
to set it first before the call.

Does anything depend on this having 3 possible return values?