test: fix some checks using ints to compare doubles
authorPeter Hutterer <peter.hutterer@who-t.net>
Mon, 14 Oct 2024 05:26:57 +0000 (15:26 +1000)
committerPeter Hutterer <peter.hutterer@who-t.net>
Tue, 15 Oct 2024 02:44:27 +0000 (12:44 +1000)
Theoretically we should be using ck_assert_double_eq here for
consistency but this patch  is part of a series eventually
replacing those calls, so let's jump to litest_assert_double
directly to avoid further rebase conflicts.

Part-of: <https://gitlab.freedesktop.org/libinput/libinput/-/merge_requests/1059>

test/test-pointer.c
test/test-touch.c
test/test-touchpad.c

index 4aeb84555d88a1bf561b35e9618613e1e51d7581..2d33e05fe7cb6ed3f758cbdd853bc2f155ad4134 100644 (file)
@@ -304,10 +304,10 @@ START_TEST(pointer_absolute_initial_state)
                p1 = libinput_event_get_pointer_event(ev1);
                p2 = libinput_event_get_pointer_event(ev2);
 
-               ck_assert_int_eq(libinput_event_pointer_get_absolute_x(p1),
-                                libinput_event_pointer_get_absolute_x(p2));
-               ck_assert_int_eq(libinput_event_pointer_get_absolute_y(p1),
-                                libinput_event_pointer_get_absolute_y(p2));
+               litest_assert_double_eq(libinput_event_pointer_get_absolute_x(p1),
+                                       libinput_event_pointer_get_absolute_x(p2));
+               litest_assert_double_eq(libinput_event_pointer_get_absolute_y(p1),
+                                       libinput_event_pointer_get_absolute_y(p2));
 
                libinput_event_destroy(ev1);
                libinput_event_destroy(ev2);
index d5d6fae803044b7ae42d5049f9f15843c2a1993b..c6802850e09127b351d976d9c2d4a59b979dbdac 100644 (file)
@@ -865,10 +865,10 @@ START_TEST(touch_initial_state)
                    libinput_event_get_type(ev1) == LIBINPUT_EVENT_TOUCH_FRAME)
                        break;
 
-               ck_assert_int_eq(libinput_event_touch_get_x(t1),
-                                libinput_event_touch_get_x(t2));
-               ck_assert_int_eq(libinput_event_touch_get_y(t1),
-                                libinput_event_touch_get_y(t2));
+               litest_assert_double_eq(libinput_event_touch_get_x(t1),
+                                       libinput_event_touch_get_x(t2));
+               litest_assert_double_eq(libinput_event_touch_get_y(t1),
+                                       libinput_event_touch_get_y(t2));
 
                libinput_event_destroy(ev1);
                libinput_event_destroy(ev2);
index 27aa59b7c495a5a9727fe35bb5e364f6df58e8a8..17a9d05440726e230c6b11c3305d0d24004df7b5 100644 (file)
@@ -76,8 +76,8 @@ START_TEST(touchpad_1fg_motion)
                struct libinput_event_pointer *ptrev;
 
                ptrev = litest_is_motion_event(event);
-               ck_assert_int_ge(libinput_event_pointer_get_dx(ptrev), 0);
-               ck_assert_int_eq(libinput_event_pointer_get_dy(ptrev), 0);
+               litest_assert_double_ge(libinput_event_pointer_get_dx(ptrev), 0);
+               litest_assert_double_eq(libinput_event_pointer_get_dy(ptrev), 0);
                libinput_event_destroy(event);
                event = libinput_get_event(li);
        }
@@ -3663,10 +3663,10 @@ START_TEST(touchpad_initial_state)
                ck_assert_int_eq(libinput_event_get_type(ev1),
                                 libinput_event_get_type(ev2));
 
-               ck_assert_int_eq(libinput_event_pointer_get_dx(p1),
-                                libinput_event_pointer_get_dx(p2));
-               ck_assert_int_eq(libinput_event_pointer_get_dy(p1),
-                                libinput_event_pointer_get_dy(p2));
+               litest_assert_double_eq(libinput_event_pointer_get_dx(p1),
+                                       libinput_event_pointer_get_dx(p2));
+               litest_assert_double_eq(libinput_event_pointer_get_dy(p1),
+                                       libinput_event_pointer_get_dy(p2));
                libinput_event_destroy(ev1);
                libinput_event_destroy(ev2);
        }