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>

diff --git a/test/test-touchpad.c b/test/test-touchpad.c
index 7c0587f09ae543b43aa68abb730d36ead07f77af..c02cb01c74383f7d04a734115b596575cab5473a 100644 (file)
--- 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);