[PATCH libinput 9/9] test: add seat changing tests
Hans de Goede
hdegoede at redhat.com
Mon Nov 24 01:02:55 PST 2014
Hi,
On 11/24/2014 01:46 AM, Peter Hutterer wrote:
> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
Looks good:
Reviewed-by: Hans de Goede <hdegoede at redhat.com>
Regards,
Hans
> ---
> test/path.c | 71 ++++++++++++++++++++++++++++++++++++++++++++++++++++++-
> test/udev.c | 78 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-
> 2 files changed, 147 insertions(+), 2 deletions(-)
>
> diff --git a/test/path.c b/test/path.c
> index ecb7839..3a2bf2f 100644
> --- a/test/path.c
> +++ b/test/path.c
> @@ -162,6 +162,74 @@ START_TEST(path_added_seat)
> }
> END_TEST
>
> +START_TEST(path_seat_change)
> +{
> + struct litest_device *dev = litest_current_device();
> + struct libinput *li = dev->libinput;
> + struct libinput_event *event;
> + struct libinput_device *device;
> + struct libinput_seat *seat1, *seat2;
> + const char *seat1_name;
> + const char *seat2_name = "new seat";
> + int rc;
> +
> + libinput_dispatch(li);
> +
> + event = libinput_get_event(li);
> + ck_assert_int_eq(libinput_event_get_type(event),
> + LIBINPUT_EVENT_DEVICE_ADDED);
> +
> + device = libinput_event_get_device(event);
> + libinput_device_ref(device);
> +
> + seat1 = libinput_device_get_seat(device);
> + libinput_seat_ref(seat1);
> +
> + seat1_name = libinput_seat_get_logical_name(seat1);
> + libinput_event_destroy(event);
> +
> + litest_drain_events(li);
> +
> + rc = libinput_device_set_seat_logical_name(device,
> + seat2_name);
> + ck_assert_int_eq(rc, 0);
> +
> + libinput_dispatch(li);
> +
> + event = libinput_get_event(li);
> + ck_assert(event != NULL);
> +
> + ck_assert_int_eq(libinput_event_get_type(event),
> + LIBINPUT_EVENT_DEVICE_REMOVED);
> +
> + ck_assert(libinput_event_get_device(event) == device);
> + libinput_event_destroy(event);
> +
> + event = libinput_get_event(li);
> + ck_assert(event != NULL);
> + ck_assert_int_eq(libinput_event_get_type(event),
> + LIBINPUT_EVENT_DEVICE_ADDED);
> + ck_assert(libinput_event_get_device(event) != device);
> + libinput_device_unref(device);
> +
> + device = libinput_event_get_device(event);
> + seat2 = libinput_device_get_seat(device);
> +
> + ck_assert_str_ne(libinput_seat_get_logical_name(seat2),
> + seat1_name);
> + ck_assert_str_eq(libinput_seat_get_logical_name(seat2),
> + seat2_name);
> + libinput_event_destroy(event);
> +
> + libinput_seat_unref(seat1);
> +
> + /* litest: swap the new device in, so cleanup works */
> + libinput_device_unref(dev->libinput_device);
> + libinput_device_ref(device);
> + dev->libinput_device = device;
> +}
> +END_TEST
> +
> START_TEST(path_added_device)
> {
> struct litest_device *dev = litest_current_device();
> @@ -805,7 +873,8 @@ main(int argc, char **argv)
> litest_add_no_device("path:suspend", path_add_device_suspend_resume);
> litest_add_no_device("path:suspend", path_add_device_suspend_resume_fail);
> litest_add_no_device("path:suspend", path_add_device_suspend_resume_remove_device);
> - litest_add_for_device("path:seat events", path_added_seat, LITEST_SYNAPTICS_CLICKPAD);
> + litest_add_for_device("path:seat", path_added_seat, LITEST_SYNAPTICS_CLICKPAD);
> + litest_add_for_device("path:seat", path_seat_change, LITEST_SYNAPTICS_CLICKPAD);
> litest_add("path:device events", path_added_device, LITEST_ANY, LITEST_ANY);
> litest_add("path:device events", path_device_sysname, LITEST_ANY, LITEST_ANY);
> litest_add_for_device("path:device events", path_add_device, LITEST_SYNAPTICS_CLICKPAD);
> diff --git a/test/udev.c b/test/udev.c
> index 9520663..f5d2c88 100644
> --- a/test/udev.c
> +++ b/test/udev.c
> @@ -175,6 +175,81 @@ START_TEST(udev_added_seat_default)
> }
> END_TEST
>
> +/**
> + * This test only works if there's at least one device in the system that is
> + * assigned the default seat. Should cover the 99% case.
> + */
> +START_TEST(udev_change_seat)
> +{
> + struct libinput *li;
> + struct udev *udev;
> + struct libinput_event *event;
> + struct libinput_device *device;
> + struct libinput_seat *seat1, *seat2;
> + const char *seat1_name;
> + const char *seat2_name = "new seat";
> + int rc;
> +
> + udev = udev_new();
> + ck_assert(udev != NULL);
> +
> + li = libinput_udev_create_context(&simple_interface, NULL, udev);
> + ck_assert(li != NULL);
> + ck_assert_int_eq(libinput_udev_assign_seat(li, "seat0"), 0);
> + libinput_dispatch(li);
> +
> + event = libinput_get_event(li);
> + ck_assert(event != NULL);
> +
> + ck_assert_int_eq(libinput_event_get_type(event),
> + LIBINPUT_EVENT_DEVICE_ADDED);
> +
> + device = libinput_event_get_device(event);
> + libinput_device_ref(device);
> +
> + seat1 = libinput_device_get_seat(device);
> + libinput_seat_ref(seat1);
> +
> + seat1_name = libinput_seat_get_logical_name(seat1);
> + libinput_event_destroy(event);
> +
> + litest_drain_events(li);
> +
> + rc = libinput_device_set_seat_logical_name(device,
> + seat2_name);
> + ck_assert_int_eq(rc, 0);
> +
> + libinput_dispatch(li);
> +
> + event = libinput_get_event(li);
> + ck_assert_int_eq(libinput_event_get_type(event),
> + LIBINPUT_EVENT_DEVICE_REMOVED);
> +
> + ck_assert(libinput_event_get_device(event) == device);
> + libinput_event_destroy(event);
> +
> + event = libinput_get_event(li);
> + ck_assert_int_eq(libinput_event_get_type(event),
> + LIBINPUT_EVENT_DEVICE_ADDED);
> + ck_assert(libinput_event_get_device(event) != device);
> + libinput_device_unref(device);
> +
> + device = libinput_event_get_device(event);
> + seat2 = libinput_device_get_seat(device);
> +
> + ck_assert_str_ne(libinput_seat_get_logical_name(seat2),
> + seat1_name);
> + ck_assert_str_eq(libinput_seat_get_logical_name(seat2),
> + seat2_name);
> + libinput_event_destroy(event);
> +
> + libinput_seat_unref(seat1);
> +
> + libinput_unref(li);
> + udev_unref(udev);
> +}
> +END_TEST
> +
> START_TEST(udev_double_suspend)
> {
> struct libinput *li;
> @@ -414,7 +489,8 @@ main(int argc, char **argv)
> litest_add_no_device("udev:create", udev_create_seat0);
> litest_add_no_device("udev:create", udev_create_empty_seat);
>
> - litest_add_no_device("udev:seat events", udev_added_seat_default);
> + litest_add_no_device("udev:seat", udev_added_seat_default);
> + litest_add_no_device("udev:seat", udev_change_seat);
>
> litest_add_for_device("udev:suspend", udev_double_suspend, LITEST_SYNAPTICS_CLICKPAD);
> litest_add_for_device("udev:suspend", udev_double_resume, LITEST_SYNAPTICS_CLICKPAD);
>
More information about the wayland-devel
mailing list