tests: attach the surface's buffer when moving it
move_client() needs to attach the buffer, even if it was attached already,
because since 184df50 configure() will be called only on newly attached
surfaces, but the one that sets the test surface position is the configure
function.
diff --git a/tests/weston-test-client-helper.c b/tests/weston-test-client-helper.c
index 1234c62..264dc0d 100644
--- a/tests/weston-test-client-helper.c
+++ b/tests/weston-test-client-helper.c
@@ -84,6 +84,10 @@
client->surface->y = y;
wl_test_move_surface(client->test->wl_test, surface->wl_surface,
surface->x, surface->y);
+ /* The attach here is necessary because commit() will call congfigure
+ * only on surfaces newly attached, and the one that sets the surface
+ * position is the configure. */
+ wl_surface_attach(surface->wl_surface, surface->wl_buffer, 0, 0);
wl_surface_damage(surface->wl_surface, 0, 0, surface->width,
surface->height);
@@ -516,7 +520,6 @@
&surface->data);
memset(surface->data, 64, width * height * 4);
- wl_surface_attach(surface->wl_surface, surface->wl_buffer, 0, 0);
move_client(client, x, y);