[PATCH 3/3] tests: add wl_resource tests

Matt Turner mattst88 at gmail.com
Mon Dec 30 13:48:06 PST 2013


On Wed, Sep 18, 2013 at 8:29 AM, Marek Ch <mchqwerty at gmail.com> wrote:
> ---
>  tests/Makefile.am      |   4 +-
>  tests/resources-test.c | 167 +++++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 170 insertions(+), 1 deletion(-)
>  create mode 100644 tests/resources-test.c
>
> diff --git a/tests/Makefile.am b/tests/Makefile.am
> index 3806cb6..9c673ae 100644
> --- a/tests/Makefile.am
> +++ b/tests/Makefile.am
> @@ -11,7 +11,8 @@ TESTS =                                               \
>         sanity-test                             \
>         socket-test                             \
>         queue-test                              \
> -       signal-test
> +       signal-test                             \
> +       resources-test
>
>  check_PROGRAMS =                               \
>         $(TESTS)                                \
> @@ -34,6 +35,7 @@ sanity_test_SOURCES = sanity-test.c $(test_runner_src)
>  socket_test_SOURCES = socket-test.c $(test_runner_src)
>  queue_test_SOURCES = queue-test.c $(test_runner_src)
>  signal_test_SOURCES = signal-test.c $(test_runner_src)
> +resources_test_SOURCES = resources-test.c $(test_runner_src)
>
>  fixed_benchmark_SOURCES = fixed-benchmark.c
>
> diff --git a/tests/resources-test.c b/tests/resources-test.c
> new file mode 100644
> index 0000000..d7a428a
> --- /dev/null
> +++ b/tests/resources-test.c
> @@ -0,0 +1,167 @@
> +/*
> + * Copyright © 2013 Marek Chalupa
> + *
> + * Permission to use, copy, modify, distribute, and sell this software and its
> + * documentation for any purpose is hereby granted without fee, provided that
> + * the above copyright notice appear in all copies and that both that copyright
> + * notice and this permission notice appear in supporting documentation, and
> + * that the name of the copyright holders not be used in advertising or
> + * publicity pertaining to distribution of the software without specific,
> + * written prior permission.  The copyright holders make no representations
> + * about the suitability of this software for any purpose.  It is provided "as
> + * is" without express or implied warranty.
> + *
> + * THE COPYRIGHT HOLDERS DISCLAIM ALL WARRANTIES WITH REGARD TO THIS SOFTWARE,
> + * INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS, IN NO
> + * EVENT SHALL THE COPYRIGHT HOLDERS BE LIABLE FOR ANY SPECIAL, INDIRECT OR
> + * CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE,
> + * DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER
> + * TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE
> + * OF THIS SOFTWARE.
> + */
> +
> +#include <assert.h>
> +#include <sys/socket.h>
> +#include <unistd.h>
> +
> +#include "wayland-server.h"
> +#include "test-runner.h"
> +
> +TEST(create_resource_tst)
> +{
> +       struct wl_display *display;
> +       struct wl_client *client;
> +       struct wl_resource *res;
> +       struct wl_list *link;
> +       int s[2];
> +       uint32_t id;
> +
> +       assert(socketpair(AF_UNIX, SOCK_STREAM | SOCK_CLOEXEC, 0, s) == 0);
> +       display = wl_display_create();
> +       assert(display);
> +       client = wl_client_create(display, s[0]);
> +       assert(client);
> +
> +       res = wl_resource_create(client, &wl_display_interface, 4, 0);
> +       assert(res);
> +
> +       /* setters/getters */
> +       assert(wl_resource_get_version(res) == 4);
> +
> +       assert(client == wl_resource_get_client(res));
> +       id = wl_resource_get_id(res);
> +       assert(wl_client_get_object(client, id) == res);
> +
> +       link = wl_resource_get_link(res);
> +       assert(link);
> +       assert(wl_resource_from_link(link) == res);
> +
> +       wl_resource_set_user_data(res, (void *) 0xbee);
> +       assert(wl_resource_get_user_data(res) == (void *) 0xbee);
> +
> +       wl_resource_destroy(res);
> +       wl_client_destroy(client);
> +       wl_display_destroy(display);
> +       close(s[1]);
> +}
> +
> +static void
> +res_destroy_func(struct wl_resource *res)
> +{
> +       assert(res);
> +
> +       _Bool *destr = wl_resource_get_user_data(res);
> +       *destr = 1;
> +}
> +
> +static _Bool notify_called = 0;
> +static void
> +destroy_notify(struct wl_listener *l, void *data)
> +{
> +       assert(l && data);
> +       notify_called = 1;
> +}
> +
> +TEST(destroy_res_tst)
> +{
> +       struct wl_display *display;
> +       struct wl_client *client;
> +       struct wl_resource *res;
> +       int s[2];
> +       unsigned id;
> +       struct wl_list *link;
> +
> +       _Bool destroyed = 0;
> +       struct wl_listener destroy_listener = {
> +               .notify = &destroy_notify
> +       };
> +
> +       assert(socketpair(AF_UNIX, SOCK_STREAM | SOCK_CLOEXEC, 0, s) == 0);
> +       display = wl_display_create();
> +       assert(display);
> +       client = wl_client_create(display, s[0]);
> +       assert(client);
> +
> +       res = wl_resource_create(client, &wl_display_interface, 4, 0);
> +       assert(res);
> +       wl_resource_set_implementation(res, NULL, &destroyed, res_destroy_func);
> +       wl_resource_add_destroy_listener(res, &destroy_listener);
> +
> +       /* without implementation this should be ignored .. */
> +       wl_resource_post_event(res, 0);
> +
> +       id = wl_resource_get_id(res);
> +       link = wl_resource_get_link(res);
> +       assert(link);
> +
> +       wl_resource_destroy(res);
> +       assert(destroyed);
> +       assert(notify_called); /* check if signal was emitted */
> +       assert(wl_client_get_object(client, id) == NULL);
> +
> +       res = wl_resource_create(client, &wl_display_interface, 2, 0);
> +       assert(res);
> +       destroyed = 0;
> +       notify_called = 0;
> +       wl_resource_set_destructor(res, res_destroy_func);
> +       wl_resource_set_user_data(res, &destroyed);
> +       wl_resource_add_destroy_listener(res, &destroy_listener);
> +       /* client should destroy the resource upon its destruction */
> +       wl_client_destroy(client);
> +       assert(destroyed);
> +       assert(notify_called);
> +
> +       wl_display_destroy(display);
> +       close(s[1]);
> +}
> +
> +TEST(create_resource_with_same_id)
> +{
> +       struct wl_display *display;
> +       struct wl_client *client;
> +       struct wl_resource *res, *res2;
> +       int s[2];
> +       uint32_t id;
> +
> +       assert(socketpair(AF_UNIX, SOCK_STREAM | SOCK_CLOEXEC, 0, s) == 0);
> +       display = wl_display_create();
> +       assert(display);
> +       client = wl_client_create(display, s[0]);
> +       assert(client);
> +
> +       res = wl_resource_create(client, &wl_display_interface, 2, 0);
> +       assert(res);
> +       id = wl_resource_get_id(res);
> +       assert(wl_client_get_object(client, id) == res);
> +
> +       /* this one should replace the old one */
> +       res2 = wl_resource_create(client, &wl_display_interface, 1, id);
> +       assert(wl_client_get_object(client, id) == res2);
> +
> +       wl_resource_destroy(res2);
> +       wl_resource_destroy(res);
> +
> +       wl_client_destroy(client);
> +       wl_display_destroy(display);
> +       close(s[1]);
> +}
> --
> 1.8.3.1

When I run this test with make check, even from the initial commit
that added it, it fails. How is this test supposed to work?


More information about the wayland-devel mailing list