Re: [PATCH 0/3] MR330: Tests for properties.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/d2d1/tests/d2d1.c:
>  }
>  effect_tests[] =
>  {
> -
>    {&CLSID_D2D12DAffineTransform,       1, 1, 1, 1},
> -
>    {&CLSID_D2D13DPerspectiveTransform,  1, 1, 1, 1},
> -
>    {&CLSID_D2D1Composite,               1, 2, 1, 0xffffffff},
> -
>    {&CLSID_D2D1Crop,                    1, 1, 1, 1},
> -
>    {&CLSID_D2D1Shadow,                  1, 1, 1, 1},
> -
>    {&CLSID_D2D1Grayscale,               3, 1, 1, 1},
> -
>    {&CLSID_D2D12DAffineTransform,       1, 4, 1, 1, 1},
> -
>    {&CLSID_D2D13DPerspectiveTransform,  1, 8, 1, 1, 1},
> -
>    {&CLSID_D2D1Composite,               1, 1, 2, 1, 0xffffffff},
> -
>    {&CLSID_D2D1Crop,                    1, 2, 1, 1, 1},
> -
>    {&CLSID_D2D1Shadow,                  1, 3, 1, 1, 1},
> -
>    {&CLSID_D2D1Grayscale,               3, 0, 1, 1, 1},
> - };

I don't think it's very convenient to test a bunch of builtin effects at once. We won't have them implemented at the same time, so that will require more skips/todos. It's better to have that tested in per-effect functions, like test_effect_2d_affine().