Re: [PATCH v3 0/4] MR47: kernel32/tests: Add tests for CancelSynchronousIo.

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about dlls/ntdll/tests/pipe.c:
>  /* bogus values */
>  res = pNtCancelSynchronousIoFile((HANDLE)0xdeadbeef, NULL, &iosb);
> - todo_wine
> ok(res == STATUS_INVALID_HANDLE, "NtCancelSynchronousIoFile returned %lx\n", res);
> res = pNtCancelSynchronousIoFile(GetCurrentThread(), NULL, NULL);
> - todo_wine
> ok(res == STATUS_ACCESS_VIOLATION, "NtCancelSynchronousIoFile returned %lx\n", res);
> res = pNtCancelSynchronousIoFile(GetCurrentThread(), NULL, (IO_STATUS_BLOCK*)0xdeadbeef);
> - todo_wine
> ok(res == STATUS_ACCESS_VIOLATION, "NtCancelSynchronousIoFile returned %lx\n", res);
> memset(&iosb, 0x55, sizeof(iosb));
> res = pNtCancelSynchronousIoFile(GetCurrentThread(), (HANDLE)0xdeadbeef, &iosb);
> - todo_wine
> - ok(res == STATUS_ACCESS_VIOLATION || broken(res == STATUS_NOT_FOUND), /* Win<10 */
> - ok(res == STATUS_ACCESS_VIOLATION || res == STATUS_NOT_FOUND, /* Wine & Win < 10 */

This should be better left as `todo_wine`.

```suggestion:-0+0
    todo_wine_if(res == STATUS_NOT_FOUND)
    ok(res == STATUS_ACCESS_VIOLATION || broken(res == STATUS_NOT_FOUND), /* Win<10 */
```