From 6138babc12ebd03d8482542137aac50913be405c Mon Sep 17 00:00:00 2001 From: Peter Hutterer Date: Thu, 18 Dec 2014 13:52:36 +1000 Subject: [PATCH] test: switch touch points around for semi-mt tap-n-drag testing The tests ignored it when motion events never happened - but that's mostly what these tests are about. This only happened for semi-mt devices that use the bounding box only, not separate touch points. Switching the touch points around that the bounding box doesn't interfere causes the test to work as expected. Signed-off-by: Peter Hutterer --- test/touchpad.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/test/touchpad.c b/test/touchpad.c index 84b30804..e6f0eb60 100644 --- a/test/touchpad.c +++ b/test/touchpad.c @@ -211,11 +211,11 @@ START_TEST(touchpad_2fg_tap_n_drag) litest_drain_events(li); - litest_touch_down(dev, 0, 50, 50); + litest_touch_down(dev, 0, 30, 70); litest_touch_up(dev, 0); - litest_touch_down(dev, 0, 50, 50); - litest_touch_down(dev, 1, 60, 50); - litest_touch_move_to(dev, 0, 50, 50, 80, 80, 5, 40); + litest_touch_down(dev, 0, 30, 70); + litest_touch_down(dev, 1, 80, 70); + litest_touch_move_to(dev, 0, 30, 70, 30, 30, 5, 40); libinput_dispatch(li); litest_assert_button_event(li, BTN_LEFT, @@ -250,11 +250,11 @@ START_TEST(touchpad_2fg_tap_n_drag_3fg_btntool) litest_drain_events(li); - litest_touch_down(dev, 0, 50, 50); + litest_touch_down(dev, 0, 30, 70); litest_touch_up(dev, 0); - litest_touch_down(dev, 0, 50, 50); - litest_touch_down(dev, 1, 60, 50); - litest_touch_move_to(dev, 0, 50, 50, 80, 80, 5, 40); + litest_touch_down(dev, 0, 30, 70); + litest_touch_down(dev, 1, 80, 90); + litest_touch_move_to(dev, 0, 30, 70, 30, 30, 5, 40); libinput_dispatch(li); litest_assert_button_event(li, BTN_LEFT,