[PATCH libinput 4/5] test: fix some tablet tests' unreached code
Peter Hutterer
peter.hutterer at who-t.net
Fri Jan 6 06:30:21 UTC 2017
These conditions were never triggered by our test suite, so let's tighten up
the tests to match what we expect.
Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
---
test/tablet.c | 42 +++++++-----------------------------------
1 file changed, 7 insertions(+), 35 deletions(-)
diff --git a/test/tablet.c b/test/tablet.c
index 212ef0c..149b058 100644
--- a/test/tablet.c
+++ b/test/tablet.c
@@ -700,16 +700,7 @@ START_TEST(proximity_in_out)
ck_assert(have_proximity_out);
/* Proximity out must not emit axis events */
- litest_tablet_proximity_out(dev);
- libinput_dispatch(li);
-
- while ((event = libinput_get_event(li))) {
- enum libinput_event_type type = libinput_event_get_type(event);
-
- ck_assert(type != LIBINPUT_EVENT_TABLET_TOOL_AXIS);
-
- libinput_event_destroy(event);
- }
+ litest_assert_empty_queue(li);
}
END_TEST
@@ -1601,16 +1592,8 @@ START_TEST(motion_event_state)
libinput_dispatch(li);
- while ((event = libinput_get_event(li))) {
- if (libinput_event_get_type(event) == LIBINPUT_EVENT_TABLET_TOOL_AXIS)
- break;
- libinput_event_destroy(event);
- }
-
- /* pop the first event off */
- ck_assert_notnull(event);
- tablet_event = libinput_event_get_tablet_tool_event(event);
- ck_assert_notnull(tablet_event);
+ event = libinput_get_event(li);
+ tablet_event = litest_is_tablet_event(event, LIBINPUT_EVENT_TABLET_TOOL_AXIS);
last_x = libinput_event_tablet_tool_get_x(tablet_event);
last_y = libinput_event_tablet_tool_get_y(tablet_event);
@@ -2022,7 +2005,6 @@ START_TEST(pad_buttons_ignored)
{
struct litest_device *dev = litest_current_device();
struct libinput *li = dev->libinput;
- struct libinput_event *event;
struct axis_replacement axes[] = {
{ ABS_DISTANCE, 10 },
{ ABS_PRESSURE, 0 },
@@ -2040,15 +2022,12 @@ START_TEST(pad_buttons_ignored)
libinput_dispatch(li);
}
- while ((event = libinput_get_event(li))) {
- ck_assert_int_ne(libinput_event_get_type(event),
- LIBINPUT_EVENT_TABLET_TOOL_BUTTON);
- libinput_event_destroy(event);
- libinput_dispatch(li);
- }
+ litest_assert_empty_queue(li);
/* same thing while in prox */
litest_tablet_proximity_in(dev, 10, 10, axes);
+ litest_drain_events(li);
+
for (button = BTN_0; button < BTN_MOUSE; button++) {
litest_event(dev, EV_KEY, button, 1);
litest_event(dev, EV_SYN, SYN_REPORT, 0);
@@ -2056,15 +2035,8 @@ START_TEST(pad_buttons_ignored)
litest_event(dev, EV_SYN, SYN_REPORT, 0);
libinput_dispatch(li);
}
- litest_tablet_proximity_out(dev);
- libinput_dispatch(li);
- while ((event = libinput_get_event(li))) {
- ck_assert_int_ne(libinput_event_get_type(event),
- LIBINPUT_EVENT_TABLET_TOOL_BUTTON);
- libinput_event_destroy(event);
- libinput_dispatch(li);
- }
+ litest_assert_empty_queue(li);
}
END_TEST
--
2.9.3
More information about the wayland-devel
mailing list