[PATCH 1/3] event-test: more aggressive event testing

U. Artie Eoff ullysses.a.eoff at intel.com
Fri Sep 28 06:39:30 PDT 2012


From: "U. Artie Eoff" <ullysses.a.eoff at intel.com>

Test surface pointer enter/leave/motion and surface leave/enter
events more aggressively.

Signed-off-by: U. Artie Eoff <ullysses.a.eoff at intel.com>
---
 tests/event-test.c  | 302 ++++++++++++++++++++++++++++++++++++++++++++++++----
 tests/test-client.c |  71 ++++++++++--
 tests/test-runner.c |   8 +-
 3 files changed, 349 insertions(+), 32 deletions(-)

diff --git a/tests/event-test.c b/tests/event-test.c
index 2cbfc2d..59e6970 100644
--- a/tests/event-test.c
+++ b/tests/event-test.c
@@ -25,50 +25,313 @@
 #include <sys/socket.h>
 #include <assert.h>
 #include <unistd.h>
-
 #include <string.h>
 
 #include "test-runner.h"
 
+struct state {
+	int px; /* pointer x */
+	int py; /* pointer y */
+	int sx; /* surface x */
+	int sy; /* surface y */
+	int sw; /* surface width */
+	int sh; /* surface height */
+};
+
+static size_t state_size = sizeof(struct state);
+
+struct context {
+	struct weston_layer *layer;
+	struct weston_seat *seat;
+	struct weston_surface *surface;
+	int pointer_x; /* server pointer x */
+	int pointer_y; /* server pointer y */
+	size_t index;
+	struct wl_array states;
+};
+
+static void
+resize(struct context *context, int w, int h)
+{
+	/* resize the surface only if the width or height is different */
+	if (context->surface->geometry.width != w ||
+	    context->surface->geometry.height != h) {
+
+		weston_surface_configure(context->surface,
+					 context->surface->geometry.x,
+					 context->surface->geometry.y,
+					 w, h);
+		weston_surface_update_transform(context->surface);
+		weston_surface_damage(context->surface);
+
+		fprintf(stderr, "resize surface: %d %d\n",
+			context->surface->geometry.width,
+			context->surface->geometry.height);
+	}
+}
+
+static void
+move(struct context *context, int x, int y)
+{
+	/* move the surface only if x or y is different */
+	if (context->surface->geometry.x != x ||
+	    context->surface->geometry.y != y) {
+
+		weston_surface_configure(context->surface,
+					 x, y,
+					 context->surface->geometry.width,
+					 context->surface->geometry.height);
+		weston_surface_update_transform(context->surface);
+		weston_surface_damage(context->surface);
+
+		fprintf(stderr, "move surface: %f %f\n",
+			context->surface->geometry.x,
+			context->surface->geometry.y);
+	}
+}
+
+static int
+contains(struct context *context, int x, int y)
+{
+	/* test whether a global x,y point is contained in the surface */
+	int sx = context->surface->geometry.x;
+	int sy = context->surface->geometry.y;
+	int sw = context->surface->geometry.width;
+	int sh = context->surface->geometry.height;
+	return x >= sx && y >= sy && x < sx + sw && y < sy + sh;
+}
+
+static void
+move_pointer(struct context *context, int x, int y)
+{
+	if (contains(context, context->pointer_x, context->pointer_y)) {
+		/* pointer is currently on the surface */
+		notify_motion(context->seat, 100,
+			      wl_fixed_from_int(x), wl_fixed_from_int(y));
+	} else {
+		/* pointer is not currently on the surface */
+		notify_pointer_focus(context->seat, context->surface->output,
+				     wl_fixed_from_int(x),
+				     wl_fixed_from_int(y));
+	}
+
+	/* update server expected pointer location */
+	context->pointer_x = x;
+	context->pointer_y = y;
+
+	fprintf(stderr, "move pointer: %d %d\n", x, y);
+}
+
+static void
+check_pointer(struct context *context, int cx, int cy)
+{
+	/*
+	 * Check whether the client reported pointer position matches
+	 * the server expected pointer position.  The client
+	 * reports -1,-1 when the pointer is not on its surface and
+	 * a surface relative x,y otherwise.
+	 */
+	int gx = context->surface->geometry.x + cx;
+	int gy = context->surface->geometry.y + cy;
+	if (!contains(context, gx, gy)) {
+		assert(!contains(context, context->pointer_x,
+				 context->pointer_y));
+	} else {
+		assert(gx == context->pointer_x);
+		assert(gy == context->pointer_y);
+	}
+}
+
+static void
+check_visible(struct context *context, int visible)
+{
+	/*
+	 * Check whether the client reported surface visibility matches
+	 * the servers expected surface visibility
+	 */
+	int ow = context->surface->output->width;
+	int oh = context->surface->output->height;
+	int sx = context->surface->geometry.x;
+	int sy = context->surface->geometry.y;
+	int sw = context->surface->geometry.width;
+	int sh = context->surface->geometry.height;
+
+	const int expect = sx < ow && sy < oh && sx + sw > 0 && sy + sh > 0;
+
+	assert(visible == expect);
+}
+
+static void
+handle_state(struct test_client *);
+
+static void
+set_state(struct test_client *client)
+{
+	struct state* state;
+	struct context *context = client->data;
+
+	if (context->index < context->states.size) {
+		state = context->states.data + context->index;
+		resize(context, state->sw, state->sh);
+		move(context, state->sx, state->sy);
+		move_pointer(context, state->px, state->py);
+		context->index += state_size;
+
+		test_client_send(client, "send-state\n");
+		client->handle = handle_state;
+	} else {
+		test_client_send(client, "bye\n");
+		client->handle = NULL;
+	}
+}
+
+static void
+handle_state(struct test_client *client)
+{
+	struct context *context = client->data;
+	wl_fixed_t x, y;
+	int visible;
+
+	assert(sscanf(client->buf, "%d %d %d", &x, &y, &visible) == 3);
+
+	check_pointer(context, wl_fixed_to_int(x), wl_fixed_to_int(y));
+	check_visible(context, visible);
+
+	set_state(client);
+}
+
+static void
+add_state(struct context *context, int px, int py, int sx, int sy,
+	  int sw, int sh)
+{
+	struct state *state = wl_array_add(&context->states,
+					   sizeof(struct state));
+
+	assert(state);
+
+	state->px = px;
+	state->py = py;
+	state->sx = sx;
+	state->sy = sy;
+	state->sw = sw;
+	state->sh = sh;
+}
+
+static void
+initialize_states(struct test_client *client)
+{
+	struct context *context = client->data;
+	struct weston_surface *surface = context->surface;
+
+	int x = surface->geometry.x;
+	int y = surface->geometry.y;
+	int w = surface->geometry.width;
+	int h = surface->geometry.height;
+
+	wl_array_init(&context->states);
+
+	/* move pointer outside top left */
+	add_state(context, x - 1, y - 1, x, y, w, h);
+	/* move pointer on top left */
+	add_state(context, x, y, x, y, w, h);
+	/* move pointer outside bottom left */
+	add_state(context, x - 1, y + h, x, y, w, h);
+	/* move pointer on bottom left */
+	add_state(context, x, y + h - 1, x, y, w, h);
+	/* move pointer outside top right */
+	add_state(context, x + w, y - 1, x, y, w, h);
+	/* move pointer on top right */
+	add_state(context, x + w - 1, y, x, y, w, h);
+	/* move pointer outside bottom right */
+	add_state(context, x + w, y + h, x, y, w, h);
+	/* move pointer on bottom right */
+	add_state(context, x + w - 1, y + h - 1, x, y, w, h);
+
+	/* move pointer outside top center */
+	add_state(context, x + w/2, y - 1, x, y, w, h);
+	/* move pointer on top center */
+	add_state(context, x + w/2, y, x, y, w, h);
+	/* move pointer outside bottom center */
+	add_state(context, x + w/2, y + h, x, y, w, h);
+	/* move pointer on bottom center */
+	add_state(context, x + w/2, y + h - 1, x, y, w, h);
+	/* move pointer outside left center */
+	add_state(context, x - 1, y + h/2, x, y, w, h);
+	/* move pointer on left center */
+	add_state(context, x, y + h/2, x, y, w, h);
+	/* move pointer outside right center */
+	add_state(context, x + w, y + h/2, x, y, w, h);
+	/* move pointer on right center */
+	add_state(context, x + w - 1, y + h/2, x, y, w, h);
+
+	/* move pointer outside of client */
+	add_state(context, 50, 50, x, y, w, h);
+	/* move client center to pointer */
+	add_state(context, 50, 50, 0, 0, w, h);
+
+	/* not visible */
+	add_state(context, 0, 0, 0, -h, w, h);
+	/* visible */
+	add_state(context, 0, 0, 0, -h+1, w, h);
+	/* not visible */
+	add_state(context, 0, 0, 0, context->surface->output->height, w, h);
+	/* visible */
+	add_state(context, 0, 0, 0, context->surface->output->height - 1, w, h);
+	/* not visible */
+	add_state(context, 0, 0, -w, 0, w, h);
+	/* visible */
+	add_state(context, 0, 0, -w+1, 0, w, h);
+	/* not visible */
+	add_state(context, 0, 0, context->surface->output->width, 0, w, h);
+	/* visible */
+	add_state(context, 0, 0, context->surface->output->width - 1, 0, w, h);
+
+	set_state(client);
+}
+
 static void
 handle_surface(struct test_client *client)
 {
 	uint32_t id;
+	struct context *context = client->data;
 	struct wl_resource *resource;
-	struct weston_surface *surface;
-	struct weston_layer *layer = client->data;
 	struct wl_list *seat_list;
-	struct weston_seat *seat;
 
 	assert(sscanf(client->buf, "surface %u", &id) == 1);
-	fprintf(stderr, "got surface id %u\n", id);
+	fprintf(stderr, "server: got surface id %u\n", id);
 	resource = wl_client_get_object(client->client, id);
 	assert(resource);
 	assert(strcmp(resource->object.interface->name, "wl_surface") == 0);
 
-	surface = (struct weston_surface *) resource;
+	context->surface = (struct weston_surface *) resource;
+	weston_surface_set_color(context->surface, 0.0, 0.0, 0.0, 1.0);
 
-	weston_surface_configure(surface, 100, 100, 200, 200);
-	weston_surface_update_transform(surface);
-	weston_surface_set_color(surface, 0.0, 0.0, 0.0, 1.0);
-	wl_list_insert(&layer->surface_list, &surface->layer_link);
-	weston_surface_damage(surface);
+	context->layer = malloc(sizeof *context->layer);
+	assert(context->layer);
+	weston_layer_init(context->layer,
+			  &client->compositor->cursor_layer.link);
+	wl_list_insert(&context->layer->surface_list,
+		       &context->surface->layer_link);
 
 	seat_list = &client->compositor->seat_list;
 	assert(wl_list_length(seat_list) == 1);
-	seat = container_of(seat_list->next, struct weston_seat, link);
+	context->seat = container_of(seat_list->next, struct weston_seat, link);
+
 	client->compositor->focus = 1; /* Make it work even if pointer is
 					* outside X window. */
-	notify_motion(seat, 100,
-		      wl_fixed_from_int(150), wl_fixed_from_int(150));
 
-	test_client_send(client, "bye\n");
+	resize(context, 100, 100);
+	move(context, 100, 100);
+	move_pointer(context, 150, 150);
+
+	test_client_send(client, "send-state\n");
+	client->handle = initialize_states;
 }
 
 TEST(event_test)
 {
+	struct context *context;
 	struct test_client *client;
-	struct weston_layer *layer;
 
 	client = test_client_launch(compositor, "test-client");
 	client->terminate = 1;
@@ -76,8 +339,7 @@ TEST(event_test)
 	test_client_send(client, "create-surface\n");
 	client->handle = handle_surface;
 
-	layer = malloc(sizeof *layer);
-	assert(layer);
-	weston_layer_init(layer, &compositor->cursor_layer.link);
-	client->data = layer;
+	context = calloc(1, sizeof *context);
+	assert(context);
+	client->data = context;
 }
diff --git a/tests/test-client.c b/tests/test-client.c
index 0009a8e..fd54867 100644
--- a/tests/test-client.c
+++ b/tests/test-client.c
@@ -35,6 +35,7 @@ struct display {
 	struct wl_compositor *compositor;
 	struct input *input;
 	struct output *output;
+	struct surface *surface;
 };
 
 struct input {
@@ -68,6 +69,8 @@ pointer_handle_enter(void *data, struct wl_pointer *pointer,
 	input->pointer_focus = wl_surface_get_user_data(surface);
 	input->x = wl_fixed_to_double(x);
 	input->y = wl_fixed_to_double(y);
+	fprintf(stderr, "test-client: got pointer enter %f %f, surface %p\n",
+		input->x, input->y, surface);
 }
 
 static void
@@ -77,6 +80,9 @@ pointer_handle_leave(void *data, struct wl_pointer *pointer,
 	struct input *input = data;
 
 	input->pointer_focus = NULL;
+	
+	fprintf(stderr, "test-client: got pointer leave, surface %p\n",
+		surface);
 }
 
 static void
@@ -87,6 +93,9 @@ pointer_handle_motion(void *data, struct wl_pointer *pointer,
 
 	input->x = wl_fixed_to_double(x);
 	input->y = wl_fixed_to_double(y);
+	
+	fprintf(stderr, "test-client: got pointer motion %f %f\n",
+		input->x, input->y);
 }
 
 static void
@@ -103,12 +112,15 @@ pointer_handle_button(void *data, struct wl_pointer *pointer,
 		input->button_mask |= bit;
 	else
 		input->button_mask &= ~bit;
+	fprintf(stderr, "test-client: got pointer button %u %u\n",
+		button, state_w);
 }
 
 static void
 pointer_handle_axis(void *data, struct wl_pointer *pointer,
 		    uint32_t time, uint32_t axis, wl_fixed_t value)
 {
+	fprintf(stderr, "test-client: got pointer axis %u %d\n", axis, value);
 }
 
 static void
@@ -116,6 +128,7 @@ keyboard_handle_keymap(void *data, struct wl_keyboard *keyboard,
 		       uint32_t format, int fd, uint32_t size)
 {
 	close(fd);
+	fprintf(stderr, "test-client: got keyboard keymap\n");
 }
 
 static void
@@ -126,6 +139,8 @@ keyboard_handle_enter(void *data, struct wl_keyboard *keyboard,
 	struct input *input = data;
 
 	input->keyboard_focus = wl_surface_get_user_data(surface);
+	fprintf(stderr, "test-client: got keyboard enter, surface %p\n",
+		surface);
 }
 
 static void
@@ -135,6 +150,8 @@ keyboard_handle_leave(void *data, struct wl_keyboard *keyboard,
 	struct input *input = data;
 
 	input->keyboard_focus = NULL;
+	fprintf(stderr, "test-client: got keyboard leave, surface %p\n",
+		surface);
 }
 
 static void
@@ -142,6 +159,7 @@ keyboard_handle_key(void *data, struct wl_keyboard *keyboard,
 		    uint32_t serial, uint32_t time, uint32_t key,
 		    uint32_t state)
 {
+	fprintf(stderr, "test-client: got keyboard key %u %u\n", key, state);
 }
 
 static void
@@ -150,6 +168,7 @@ keyboard_handle_modifiers(void *data, struct wl_keyboard *keyboard,
 			  uint32_t mods_latched, uint32_t mods_locked,
 			  uint32_t group)
 {
+	fprintf(stderr, "test-client: got keyboard modifier\n");
 }
 
 static const struct wl_pointer_listener pointer_listener = {
@@ -266,7 +285,8 @@ handle_global(struct wl_display *_display, uint32_t id,
 				       &output_listener, output);
 		display->output = output;
 
-		fprintf(stderr, "created output global %p\n", display->output);
+		fprintf(stderr, "test-client: created output global %p\n",
+			display->output);
 	}
 }
 
@@ -278,7 +298,8 @@ surface_enter(void *data,
 
 	surface->output = wl_output_get_user_data(output);
 
-	fprintf(stderr, "got surface enter, output %p\n", surface->output);
+	fprintf(stderr, "test-client: got surface enter, output %p\n",
+		surface->output);
 }
 
 static void
@@ -288,6 +309,9 @@ surface_leave(void *data,
 	struct surface *surface = data;
 
 	surface->output = NULL;
+
+	fprintf(stderr, "test-client: got surface leave, output %p\n",
+		wl_output_get_user_data(output));
 }
 
 static const struct wl_surface_listener surface_listener = {
@@ -296,6 +320,33 @@ static const struct wl_surface_listener surface_listener = {
 };
 
 static void
+send_state(int fd, struct display* display)
+{
+	char buf[64];
+	int len;
+	int visible = display->surface->output != NULL;
+	wl_fixed_t x = wl_fixed_from_int(-1);
+	wl_fixed_t y = wl_fixed_from_int(-1);
+
+	if (display->input->pointer_focus == display->surface) {
+		x = wl_fixed_from_double(display->input->x);
+		y = wl_fixed_from_double(display->input->y);
+	}
+
+	if (visible) {
+		/* FIXME: this fails on multi-display setup */
+		/* assert(display->surface->output == display->output); */
+	}
+
+	wl_display_flush(display->display);
+
+	len = snprintf(buf, sizeof buf, "%d %d %d\n", x, y, visible);
+	assert(write(fd, buf, len) == len);
+
+	wl_display_roundtrip(display->display);
+}
+
+static void
 create_surface(int fd, struct display *display)
 {
 	struct surface *surface;
@@ -304,8 +355,10 @@ create_surface(int fd, struct display *display)
 
 	surface = malloc(sizeof *surface);
 	assert(surface);
+	display->surface = surface;
 	surface->surface = wl_compositor_create_surface(display->compositor);
 	wl_surface_add_listener(surface->surface, &surface_listener, surface);
+
 	wl_display_flush(display->display);
 
 	len = snprintf(buf, sizeof buf, "surface %d\n",
@@ -313,12 +366,8 @@ create_surface(int fd, struct display *display)
 	assert(write(fd, buf, len) == len);
 
 	poll(NULL, 0, 100); /* Wait for next frame where we'll get events. */
-	wl_display_roundtrip(display->display);
 
-	assert(surface->output == display->output);
-	assert(display->input->pointer_focus == surface);
-	assert(display->input->x == 50);
-	assert(display->input->y == 50);
+	wl_display_roundtrip(display->display);
 }
 
 int main(int argc, char *argv[])
@@ -346,7 +395,8 @@ int main(int argc, char *argv[])
 	while (1) {
 		ret = read(fd, buf, sizeof buf);
 		if (ret == -1) {
-			fprintf(stderr, "read error: fd %d, %m\n", fd);
+			fprintf(stderr, "test-client: read error: fd %d, %m\n",
+				fd);
 			return -1;
 		}
 
@@ -356,8 +406,11 @@ int main(int argc, char *argv[])
 			return 0;
 		} else if (strncmp(buf, "create-surface\n", ret) == 0) {
 			create_surface(fd, display);
+		} else if (strncmp(buf, "send-state\n", ret) == 0) {
+			send_state(fd, display);
 		} else {
-			fprintf(stderr, "unknown command %.*s\n", ret, buf);
+			fprintf(stderr, "test-client: unknown command %.*s\n",
+				ret, buf);
 			return -1;
 		}
 	}
diff --git a/tests/test-runner.c b/tests/test-runner.c
index 09c2b1f..6ca087d 100644
--- a/tests/test-runner.c
+++ b/tests/test-runner.c
@@ -40,7 +40,7 @@ test_client_cleanup(struct weston_process *proc, int status)
 	struct test_client *client =
 		container_of(proc, struct test_client, proc);
 
-	fprintf(stderr, "test client exited, status %d\n", status);
+	fprintf(stderr, "server: test client exited, status %d\n", status);
 
 	client->status = status;
 	client->done = 1;
@@ -60,7 +60,7 @@ test_client_data(int fd, uint32_t mask, void *data)
 
 	len = read(client->fd, client->buf, sizeof client->buf);
 	assert(len >= 0);
-	fprintf(stderr, "got %.*s from client\n", len - 1, client->buf);
+	fprintf(stderr, "server: got %.*s from client\n", len - 1, client->buf);
 	assert(client->buf[len - 1] == '\n');
 	client->buf[len - 1] = '\0';
 
@@ -88,7 +88,7 @@ test_client_launch(struct weston_compositor *compositor, const char *file_name)
 	snprintf(buf, sizeof buf, "%d", client_fd);
 	setenv("TEST_SOCKET", buf, 1);
 	snprintf(buf, sizeof buf, "%s/%s", getenv("abs_builddir"), file_name);
-	fprintf(stderr, "launching %s\n", buf);
+	fprintf(stderr, "server: launching %s\n", buf);
 
 	client->terminate = 0;
 	client->compositor = compositor;
@@ -117,6 +117,8 @@ test_client_send(struct test_client *client, const char *fmt, ...)
 	len = vsnprintf(buf, sizeof buf, fmt, ap);
 	va_end(ap);
 
+	fprintf(stderr, "server: sending %s", buf);
+
 	assert(write(client->fd, buf, len) == len);
 }
 
-- 
1.7.11.2



More information about the wayland-devel mailing list