shell32/tests: Add test for SHGetPathFromIDListA (try 3)
Nikolay Sivov
bunglehead at gmail.com
Wed Jan 8 04:15:49 CST 2014
On 1/8/2014 12:47, Zhenbo Li wrote:
> try 3:
> Add 'static' in my code
> I forgot to call test_SHGetPathFromIDListA() in try2
> Check GetLastError() as well
> try 2:
> Move code to string.c to avoid creating a new file.
> Avoid malloc() and free().
>
> I'm working on Bug 33750, so I'll add test cases for
> SHGetPathFromIDListA gradually.
> ---
> dlls/shell32/tests/string.c | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
> +static void test_SHGetPathFromIDListA(void)
> +{
> + BOOL ret;
> + char pszPath[MAX_PATH];
> +
> + ret = SHGetPathFromIDListA(NULL, pszPath);
> + ok(!ret, "SHGetPathFromIDListA should fail.\n");
> + ok(GetLastError() == ERROR_PROC_NOT_FOUND,
> + "Bad GetLastError = %d\n", GetLastError());
> +}
> +
You need to set last error to something like 0xdeadbeef, your test
suggest that this call doesn't set it.
I guess you're trying to test return value only, but it could be
interesting to test what happens to path buffer too.
P.S. please avoid psz* style prefixes in names.
More information about the wine-devel
mailing list