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:
> static void test_theme(void)
>  {
> - static const WCHAR * const app_class[] =
> - {
> -
>    L"button", L"combobox", L"edit", L"explorerbar", L"header",
> -
>    L"listview", L"explorer::listview", L"menu", L"progress",
> -
>    L"rebar", L"scrollbar", L"spin", L"startpanel", L"status",
> -
>    L"tab", L"taskband", L"taskbar", L"toolbar", L"tooltip",
> -
>    L"trackbar", L"treeview", L"explorer::treeview", L"window"

Now that I come to think of it. You don't really need to test all these parts. All you need is to test "explorer::treeview" for the OpenThemeData() function and you can move the OpenThemeData() tests to test_OpenThemeData(). test_theme() is for testing whether a part exists or part properties in the theme files. If you still want to test these extra theme parts, you can keep them in test_theme() and make them in a separate commit.