diff options
author | Peter Hutterer <peter.hutterer@who-t.net> | 2023-03-22 16:15:23 +1000 |
---|---|---|
committer | Peter Hutterer <peter.hutterer@who-t.net> | 2023-03-22 16:16:59 +1000 |
commit | 9422d72f97cef778ce464f957c8edd3d53d93d72 (patch) | |
tree | 2651a461a35a61ad80938a285683de2850a8a726 | |
parent | c491e3b94de8c6117ddb3a8ac30476ed26d62853 (diff) | |
download | libinput-9422d72f97cef778ce464f957c8edd3d53d93d72.tar.gz |
test: comment two tap tests
This is confusing without any explanation - the first set of touches is
within the edge zone, the second set of touches is explicitly within the
software button area but not inside the edge zone. Let's add comments so
the intention is clear.
Same-ish for the clickfinger test.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
-rw-r--r-- | test/test-touchpad.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/test/test-touchpad.c b/test/test-touchpad.c index 7c0587f0..c02cb01c 100644 --- a/test/test-touchpad.c +++ b/test/test-touchpad.c @@ -1602,6 +1602,8 @@ START_TEST(touchpad_palm_detect_tap_softbuttons) litest_disable_hold_gestures(dev->libinput_device); litest_drain_events(li); + /* Two touches in the software button area, but inside + * the palm detection edge zone -> expect palm detection */ litest_touch_down(dev, 0, 99, 99); litest_touch_up(dev, 0); libinput_dispatch(li); @@ -1616,6 +1618,8 @@ START_TEST(touchpad_palm_detect_tap_softbuttons) litest_timeout_tap(); litest_assert_empty_queue(li); + /* Two touches in the software button area, but + * not in the palm detection edge zone -> expect taps */ litest_touch_down(dev, 0, 10, 99); litest_touch_up(dev, 0); libinput_dispatch(li); @@ -1657,6 +1661,8 @@ START_TEST(touchpad_palm_detect_tap_clickfinger) litest_disable_hold_gestures(dev->libinput_device); litest_drain_events(li); + /* Taps in each of the 4 corners of the touchpad, all + * inside the palm detection edge zone*/ litest_touch_down(dev, 0, 95, 5); litest_touch_up(dev, 0); libinput_dispatch(li); |