Re: [PATCH v5 0/3] MR6904: ntdll.dll: Update NtQueryDirectoryFile to align with current Windows behaviour

[eric pouech (＠epo)]

eric pouech (@epo) commented about dlls/ntdll/tests/directory.c:
> -
>    /* Print the query parameters and the result */
> -
>    if (status == 0) {
> -
>        dir_info = (FILE_DIRECTORY_INFORMATION *)data;
> -
>        name = dir_info->FileName;
> -
>        name_len = dir_info->FileNameLength / sizeof(WCHAR);
> -
> -
>        ok( name_len == mask_len, "unexpected filename length %lu, expected %lu\n", name_len, mask_len );
> -
>        ok( !memcmp(name, mask.Buffer, mask_len), "unexpected filename %s, expected %s\n",
> -
>            wine_dbgstr_wn(name, name_len), wine_dbgstr_wn(mask.Buffer, mask_len) );
> -
>    }
> - }
> - else
> - {
> -
>    if (validate_only && status == STATUS_SUCCESS) return FALSE;
> -
>    /* Print the query parameters and the result */
> -
>    if (status == 0) {

(I thought I commented this on previous iteration, but couldn't find it back)
If I follow the logic of the test, you should here replace this whole `if (status == 0){}` block by `ok(status == STATUS_NO_MORE_FILES`