Re: [PATCH v3 0/2] MR4140: uxtheme/tests: Add some tests for OpenThemeData() with various app/class names.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/uxtheme/tests/system.c:
> - for (i = 0; i < ARRAY_SIZE(app_class); i++)
> - {
> -
>    htheme = OpenThemeData(hwnd, app_class[i]);
> -
>    todo_wine_if(i == 3 || i == 6 || i == 12 || i == 15 || i == 16 || i == 21)
> -
>    ok(htheme != NULL, "OpenThemeData(%s) error %#lx.\n", wine_dbgstr_w(app_class[i]), GetLastError());
> -
>    CloseThemeData(htheme);
> - }
> -
> - hr = SetWindowTheme(hwnd, L"explorer", NULL);
> - ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr);
> - SetLastError(0xdeadbeef);
> - htheme = OpenThemeData(hwnd, L"explorer::listview");
> - ok(!htheme, "OpenThemeData() should fail.\n");
> - ok(GetLastError() == E_PROP_ID_UNSUPPORTED, "Got unexpected %#lx.\n", GetLastError());
> - SetWindowTheme(hwnd, NULL, NULL);

Let's test that OpenThemeData(hwnd, L"abc:def;explorer::treeview") succeeds.