From df0ec0f771b56f51c1bf5767aeae932b2c712292 Mon Sep 17 00:00:00 2001 From: Michael Stapelberg Date: Wed, 25 May 2011 20:16:14 +0200 Subject: [PATCH] tests: explicitly declare done_testing in subtest --- testcases/t/58-wm_take_focus.t | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/testcases/t/58-wm_take_focus.t b/testcases/t/58-wm_take_focus.t index a7552c24..eca3cef2 100644 --- a/testcases/t/58-wm_take_focus.t +++ b/testcases/t/58-wm_take_focus.t @@ -4,12 +4,12 @@ # Tests if the WM_TAKE_FOCUS protocol is correctly handled by i3 # use X11::XCB qw(:all); -use EV; -use AnyEvent; use i3test; use v5.10; BEGIN { + use_ok('EV'); + use_ok('AnyEvent'); use_ok('X11::XCB::Window'); use_ok('X11::XCB::Event::Generic'); use_ok('X11::XCB::Event::MapNotify'); @@ -59,6 +59,8 @@ subtest 'Window without WM_TAKE_FOCUS', sub { my $result = $cv->recv; ok($result, 'cv result'); + + done_testing; }; subtest 'Window with WM_TAKE_FOCUS', sub { @@ -105,6 +107,8 @@ subtest 'Window with WM_TAKE_FOCUS', sub { my ($data, $time) = unpack("L2", $result); is($data, $x->atom(name => 'WM_TAKE_FOCUS')->id, 'first uint32_t contains WM_TAKE_FOCUS atom'); } + + done_testing; }; -- 2.39.5