Commit 3b8a1c7f authored by Kristian Høgsberg's avatar Kristian Høgsberg

resources-test: Don't send invalid event

Even if nothing receives the even, the arguments still need to be valid.
The test is sending out event 0 from the wl_display interface, which is
the error event.  This requires arg 0 to be a valid object and arg 2 to
be a non-null string.  The test just leaves that undefined, causing
intermittent test failures.

As it is, the resource destroy test doesn't need to send an event to
validate the various resource destroy hooks, so we can just remove the
call to wl_resource_post_event() alltogether.

Thanks to Matt Turner <mattst88@gmail.com> for pointing out the failure.
parent 47bbc6bb
......@@ -107,9 +107,6 @@ TEST(destroy_res_tst)
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);
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment