Re: [PATCH 1/2] kernel32/tests: Add test for FormatMessage with HRESULT

[Sebastian Lackner]

On 20.10.2014 19:05, Catalin Patulea wrote:
> +    /* Test HRESULT. It's not documented but in practice _com_error::ErrorMessage relies on this. */
> +    ret = FormatMessageA(FORMAT_MESSAGE_FROM_SYSTEM | FORMAT_MESSAGE_FROM_HMODULE, h, 0x80070005 /* E_ACCESSDENIED */,
> +                         MAKELANGID(LANG_NEUTRAL, SUBLANG_NEUTRAL), out, sizeof(out)/sizeof(CHAR), NULL);
> +    ok(ret != 0, "FormatMessageA returned 0\n");
> +

As this test is failing on Wine you should mark it with "todo_wine", to avoid introducing test failures. Patch 2 which fixes the issue should remove the "todo_wine" again.