mirror of
https://gitlab.freedesktop.org/libinput/libinput.git
synced 2026-01-09 13:20:24 +01:00
test: fix two tablet tests
The serial test was broken, it succeeded even if we never got an event. The second test was fine, but complicated. Make it use some of the newer litest features. Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> Reviewed-by: Benjamin Tissoires <benjamin.tissoires@gmail.com> Reviewed-by: Stephen Chandler Paul <thatslyude@gmail.com>
This commit is contained in:
parent
048fc37ba5
commit
c1e567e4fa
1 changed files with 16 additions and 26 deletions
|
|
@ -761,18 +761,14 @@ START_TEST(tool_serial)
|
|||
litest_event(dev, EV_MSC, MSC_SERIAL, 1000);
|
||||
litest_event(dev, EV_SYN, SYN_REPORT, 0);
|
||||
|
||||
libinput_dispatch(li);
|
||||
while ((event = libinput_get_event(li))) {
|
||||
if (libinput_event_get_type(event) ==
|
||||
LIBINPUT_EVENT_TABLET_PROXIMITY) {
|
||||
tablet_event = libinput_event_get_tablet_event(event);
|
||||
tool = libinput_event_tablet_get_tool(tablet_event);
|
||||
|
||||
ck_assert_uint_eq(libinput_tool_get_serial(tool), 1000);
|
||||
}
|
||||
|
||||
libinput_event_destroy(event);
|
||||
}
|
||||
litest_wait_for_event_of_type(li,
|
||||
LIBINPUT_EVENT_TABLET_PROXIMITY,
|
||||
-1);
|
||||
event = libinput_get_event(li);
|
||||
tablet_event = libinput_event_get_tablet_event(event);
|
||||
tool = libinput_event_tablet_get_tool(tablet_event);
|
||||
ck_assert_uint_eq(libinput_tool_get_serial(tool), 1000);
|
||||
libinput_event_destroy(event);
|
||||
}
|
||||
END_TEST
|
||||
|
||||
|
|
@ -783,7 +779,6 @@ START_TEST(serial_changes_tool)
|
|||
struct libinput_event_tablet *tablet_event;
|
||||
struct libinput_event *event;
|
||||
struct libinput_tool *tool;
|
||||
bool tool_updated = false;
|
||||
|
||||
litest_drain_events(li);
|
||||
|
||||
|
|
@ -798,20 +793,15 @@ START_TEST(serial_changes_tool)
|
|||
litest_event(dev, EV_MSC, MSC_SERIAL, 2000);
|
||||
litest_event(dev, EV_SYN, SYN_REPORT, 0);
|
||||
|
||||
libinput_dispatch(li);
|
||||
while ((event = libinput_get_event(li))) {
|
||||
if (libinput_event_get_type(event) ==
|
||||
LIBINPUT_EVENT_TABLET_PROXIMITY) {
|
||||
tablet_event = libinput_event_get_tablet_event(event);
|
||||
tool = libinput_event_tablet_get_tool(tablet_event);
|
||||
litest_wait_for_event_of_type(li,
|
||||
LIBINPUT_EVENT_TABLET_PROXIMITY,
|
||||
-1);
|
||||
event = libinput_get_event(li);
|
||||
tablet_event = libinput_event_get_tablet_event(event);
|
||||
tool = libinput_event_tablet_get_tool(tablet_event);
|
||||
|
||||
ck_assert_uint_eq(libinput_tool_get_serial(tool), 2000);
|
||||
tool_updated = true;
|
||||
}
|
||||
|
||||
libinput_event_destroy(event);
|
||||
}
|
||||
ck_assert(tool_updated);
|
||||
ck_assert_uint_eq(libinput_tool_get_serial(tool), 2000);
|
||||
libinput_event_destroy(event);
|
||||
}
|
||||
END_TEST
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue