Re: [Linuxwacom-devel] [PATCH] test: Fix tests for Capacity removal

[Chris B. <ch...@cn...>]

Probably I should write a checklist on wiki for all files to consider
when adding or removing properties.  I totally forgot this part as
well.  Thanks.

Reviewed-by: Chris Bagwell <ch...@cn...>

On Sun, Jul 17, 2011 at 6:50 PM, Peter Hutterer
<pet...@wh...> wrote:
> Removal of capacity caused a few tests to fail. Fix this.
>
> Introduced in xf86-input-wacom-0.11.0-36-g6c9c05e
>
> CC: Chris Bagwell <ch...@cn...>
> Signed-off-by: Peter Hutterer <pet...@wh...>
> ---
>  test/wacom-tests.c |    1 -
>  tools/xsetwacom.c  |    4 ++--
>  2 files changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/test/wacom-tests.c b/test/wacom-tests.c
> index a08b7b9..d91d9c3 100644
> --- a/test/wacom-tests.c
> +++ b/test/wacom-tests.c
> @@ -263,7 +263,6 @@ test_suppress(void)
>        new.field = old.field;
>
>        test_above_suppress(pressure);
> -       test_above_suppress(capacity);
>        test_above_suppress(throttle);
>        test_above_suppress(rotation);
>        test_above_suppress(abswheel);
> diff --git a/tools/xsetwacom.c b/tools/xsetwacom.c
> index 360776e..365bad4 100644
> --- a/tools/xsetwacom.c
> +++ b/tools/xsetwacom.c
> @@ -2437,8 +2437,8 @@ static void test_parameter_number(void)
>         * deprecated them.
>         * Numbers include trailing NULL entry.
>         */
> -       assert(ArrayLength(parameters) == 35);
> -       assert(ArrayLength(deprecated_parameters) == 16);
> +       assert(ArrayLength(parameters) == 34);
> +       assert(ArrayLength(deprecated_parameters) == 17);
>  }
>
>  /**
> --
> 1.7.6
>