Re: [PATCH v9 0/3] MR8515: setupapi: Support in-built properties for device instances in SetupDiGetDevicePropertyW.

[Connor McAdams (＠cmcadams)]

Connor McAdams (@cmcadams) commented about dlls/setupapi/tests/devinst.c:
> - todo_wine ok(IsEqualGUID(&guid_val, &guid), "Expect buffer %s, got %s\n", debugstr_guid(&guid), debugstr_guid(&guid_val));
> -
> - for (i = 0; i < ARRAY_SIZE(inbuilt_props); i++)
> - {
> -
>    const DEVPROPKEY *key = &inbuilt_props[i].key;
> -
>    DEVPROPTYPE type = inbuilt_props[i].type;
> -
>    const BYTE *val = inbuilt_props[i].value;
> -
>    DWORD size = inbuilt_props[i].size;
> -
>    BYTE *buf;
> -
> -
>    winetest_push_context("inbuilt_props[%lu]", i);
> -
>    SetLastError(0xdeadbeef);
> -
>    ret = pSetupDiSetDevicePropertyW(set, &device_data, key, type, val, size, 0);
> -
>    err = GetLastError();
> -
>    todo_wine ok(!ret, "Expected failure.\n");
> -
>    todo_wine ok(err == ERROR_ACCESS_DENIED, "Expect last error %#x, got %#lx.\n", ERROR_INVALID_ACCESS, err);

```suggestion:-0+0
        todo_wine ok(err == ERROR_ACCESS_DENIED, "Expect last error %#x, got %#lx.\n", ERROR_ACCESS_DENIED, err);
```
You're checking for `err == ERROR_ACCESS_DENIED` here, but printing an expected value of `ERROR_INVALID_ACCESS`.