diff options
author | Alexandros Frantzis <alexandros.frantzis@collabora.com> | 2017-12-13 13:27:57 +0200 |
---|---|---|
committer | Pekka Paalanen <pekka.paalanen@collabora.co.uk> | 2017-12-18 11:46:25 +0200 |
commit | dae224c6dc8054d1cf3ce4596588f922ff029fd2 (patch) | |
tree | 54f3ec01e75f8318f7769131428054bc45f54b46 /tests | |
parent | 2180858592bdfa67ed75fb91cde1c18ee17270a2 (diff) |
tests: Add test for keyboard key event timestamps
Add test to verify that the server correctly sets the timestamps of
keyboard key events. This requires updating the weston-test protocol to
support passing key event timestamps.
simple_keyboard_test now uses the create_client_with_keyboard_focus()
helper function which changes the initial state of the surface to be
focused. This leads to one additional iteration of the test loop when
starting, during which the surface is deactivated, i.e., loses focus.
After this initial iteration the test continues as before.
Furthermore, simple_keyboard_test now uses the send_key() helper
function which performs a roundtrip internally. To account for this, the
client_roundtrip() function is now directly called in the loop only when
it is still required, i.e., when deactivating the surface.
Signed-off-by: Alexandros Frantzis <alexandros.frantzis@collabora.com>
Reviewed-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/keyboard-test.c | 61 | ||||
-rw-r--r-- | tests/weston-test-client-helper.c | 3 | ||||
-rw-r--r-- | tests/weston-test-client-helper.h | 1 | ||||
-rw-r--r-- | tests/weston-test.c | 3 |
4 files changed, 52 insertions, 16 deletions
diff --git a/tests/keyboard-test.c b/tests/keyboard-test.c index 6b4ba19d..722bfd32 100644 --- a/tests/keyboard-test.c +++ b/tests/keyboard-test.c @@ -27,21 +27,45 @@ #include <stdint.h> +#include "shared/timespec-util.h" #include "weston-test-client-helper.h" +static const struct timespec t1 = { .tv_sec = 1, .tv_nsec = 1000001 }; +static const struct timespec t2 = { .tv_sec = 2, .tv_nsec = 2000001 }; + +static struct client * +create_client_with_keyboard_focus(void) +{ + struct client *cl = create_client_and_test_surface(10, 10, 1, 1); + assert(cl); + + weston_test_activate_surface(cl->test->weston_test, + cl->surface->wl_surface); + client_roundtrip(cl); + + return cl; +} + +static void +send_key(struct client *client, const struct timespec *time, + uint32_t key, uint32_t state) +{ + uint32_t tv_sec_hi, tv_sec_lo, tv_nsec; + + timespec_to_proto(time, &tv_sec_hi, &tv_sec_lo, &tv_nsec); + weston_test_send_key(client->test->weston_test, tv_sec_hi, tv_sec_lo, + tv_nsec, key, state); + client_roundtrip(client); +} + TEST(simple_keyboard_test) { - struct client *client; - struct surface *expect_focus = NULL; - struct keyboard *keyboard; + struct client *client = create_client_with_keyboard_focus(); + struct keyboard *keyboard = client->input->keyboard; + struct surface *expect_focus = client->surface; uint32_t expect_key = 0; uint32_t expect_state = 0; - client = create_client_and_test_surface(10, 10, 1, 1); - assert(client); - - keyboard = client->input->keyboard; - while (1) { assert(keyboard->key == expect_key); assert(keyboard->state == expect_state); @@ -49,12 +73,12 @@ TEST(simple_keyboard_test) if (keyboard->state == WL_KEYBOARD_KEY_STATE_PRESSED) { expect_state = WL_KEYBOARD_KEY_STATE_RELEASED; - weston_test_send_key(client->test->weston_test, - expect_key, expect_state); + send_key(client, &t1, expect_key, expect_state); } else if (keyboard->focus) { expect_focus = NULL; weston_test_activate_surface( client->test->weston_test, NULL); + client_roundtrip(client); } else if (expect_key < 10) { expect_key++; expect_focus = client->surface; @@ -62,12 +86,21 @@ TEST(simple_keyboard_test) weston_test_activate_surface( client->test->weston_test, expect_focus->wl_surface); - weston_test_send_key(client->test->weston_test, - expect_key, expect_state); + send_key(client, &t1, expect_key, expect_state); } else { break; } - - client_roundtrip(client); } } + +TEST(keyboard_key_event_time) +{ + struct client *client = create_client_with_keyboard_focus(); + struct keyboard *keyboard = client->input->keyboard; + + send_key(client, &t1, 1, WL_KEYBOARD_KEY_STATE_PRESSED); + assert(keyboard->key_time_msec == timespec_to_msec(&t1)); + + send_key(client, &t2, 1, WL_KEYBOARD_KEY_STATE_RELEASED); + assert(keyboard->key_time_msec == timespec_to_msec(&t2)); +} diff --git a/tests/weston-test-client-helper.c b/tests/weston-test-client-helper.c index 203cd441..695f6912 100644 --- a/tests/weston-test-client-helper.c +++ b/tests/weston-test-client-helper.c @@ -262,13 +262,14 @@ keyboard_handle_leave(void *data, struct wl_keyboard *wl_keyboard, static void keyboard_handle_key(void *data, struct wl_keyboard *wl_keyboard, - uint32_t serial, uint32_t time, uint32_t key, + uint32_t serial, uint32_t time_msec, uint32_t key, uint32_t state) { struct keyboard *keyboard = data; keyboard->key = key; keyboard->state = state; + keyboard->key_time_msec = time_msec; fprintf(stderr, "test-client: got keyboard key %u %u\n", key, state); } diff --git a/tests/weston-test-client-helper.h b/tests/weston-test-client-helper.h index 6f5f9c41..214f6959 100644 --- a/tests/weston-test-client-helper.h +++ b/tests/weston-test-client-helper.h @@ -107,6 +107,7 @@ struct keyboard { int rate; int delay; } repeat_info; + uint32_t key_time_msec; }; struct touch { diff --git a/tests/weston-test.c b/tests/weston-test.c index 1799de92..7dd7a1cf 100644 --- a/tests/weston-test.c +++ b/tests/weston-test.c @@ -208,13 +208,14 @@ activate_surface(struct wl_client *client, struct wl_resource *resource, static void send_key(struct wl_client *client, struct wl_resource *resource, + uint32_t tv_sec_hi, uint32_t tv_sec_lo, uint32_t tv_nsec, uint32_t key, enum wl_keyboard_key_state state) { struct weston_test *test = wl_resource_get_user_data(resource); struct weston_seat *seat = get_seat(test); struct timespec time; - timespec_from_msec(&time, 100); + timespec_from_proto(&time, tv_sec_hi, tv_sec_lo, tv_nsec); notify_key(seat, &time, key, state, STATE_UPDATE_AUTOMATIC); } |