I'm not sure to get your point.
- do you mean that you'd like to ok() test the return values of all the calls in the helper? I didn't do it as a) the helper can be called with an inexistant file, b) the cases of failure are ok() tested later on (machine not being unknown)
- do you mean the calls in the helper can't fail? (I don't see a reason they wouldn't)
- something else?

Le ven. 10 f������vr. 2023 ������ 19:23, Zebediah Figura (@zfigura) <wine@gitlab.winehq.org> a ������crit������:
Zebediah Figura (@zfigura) commented about dlls/dbghelp/tests/dbghelp.c:
> +������ ������ {
> +������ ������ ������ ������ hMap = CreateFileMappingW(hFile, NULL, PAGE_READONLY, 0, 0, NULL);
> +������ ������ ������ ������ if (hMap != NULL)
> +������ ������ ������ ������ {
> +������ ������ ������ ������ ������ ������ mapping = MapViewOfFile(hMap, FILE_MAP_READ, 0, 0, 0);
> +������ ������ ������ ������ ������ ������ if (mapping != NULL)
> +������ ������ ������ ������ ������ ������ {
> +������ ������ ������ ������ ������ ������ ������ ������ nthdr = RtlImageNtHeader(mapping);
> +������ ������ ������ ������ ������ ������ ������ ������ if (nthdr != NULL) machine = nthdr->FileHeader.Machine;
> +������ ������ ������ ������ ������ ������ ������ ������ UnmapViewOfFile(mapping);
> +������ ������ ������ ������ ������ ������ }
> +������ ������ ������ ������ ������ ������ CloseHandle(hMap);
> +������ ������ ������ ������ }
> +������ ������ ������ ������ CloseHandle(hFile);
> +������ ������ }
>������ ������ ������ return machine;
Why do this? This part doesn't fail.

--
https://gitlab.winehq.org/wine/wine/-/merge_requests/2152#note_23766