fix a few timing bugs in tests

mojolicious · Apr 21, 2016 · f49a33a · f49a33a
1 parent 8aa8fed
commit f49a33a
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/Changes b/Changes
@@ -1,9 +1,10 @@
 
-6.59  2016-04-20
+6.59  2016-04-21
   - Improved HTML Living Standard compliance of Mojo::DOM::HTML.
   - Improved Mojo::Server::Prefork to log when the manager process has been
     stopped.
   - Fixed a few tests to keep up with breaking changes in IO::Socket::SSL.
+  - Fixed a few timing bugs in tests.
 
 6.58  2016-04-09
   - Removed deprecated build_frame and parse_frame methods from

diff --git a/t/mojo/reactor_ev.t b/t/mojo/reactor_ev.t
@@ -129,13 +129,16 @@ ok $writable, 'handle is writable again';
 ok !$timer,     'timer was not triggered';
 ok !$recurring, 'recurring was not triggered again';
 ($readable, $writable, $timer, $recurring) = ();
+my $next_tick;
+is $reactor->next_tick(sub { $next_tick++ }), undef, 'returned undef';
 $id = $reactor->recurring(0 => sub { $recurring++ });
-is $reactor->next_tick(sub { shift->stop }), undef, 'returned undef';
+$reactor->timer(0.025 => sub { shift->stop });
 $reactor->start;
 ok $readable, 'handle is readable again';
 ok $writable, 'handle is writable again';
 ok !$timer, 'timer was not triggered';
 ok $recurring, 'recurring was triggered again';
+ok $next_tick, 'next tick was triggered';
 
 # Reset
 $reactor->next_tick(sub { die 'Reset failed' });

diff --git a/t/mojo/reactor_poll.t b/t/mojo/reactor_poll.t
@@ -126,13 +126,16 @@ ok $writable, 'handle is writable again';
 ok !$timer,     'timer was not triggered';
 ok !$recurring, 'recurring was not triggered again';
 ($readable, $writable, $timer, $recurring) = ();
+my $next_tick;
+is $reactor->next_tick(sub { $next_tick++ }), undef, 'returned undef';
 $id = $reactor->recurring(0 => sub { $recurring++ });
-is $reactor->next_tick(sub { shift->stop }), undef, 'returned undef';
+$reactor->timer(0.025 => sub { shift->stop });
 $reactor->start;
 ok $readable, 'handle is readable again';
 ok $writable, 'handle is writable again';
 ok !$timer, 'timer was not triggered';
 ok $recurring, 'recurring was triggered again';
+ok $next_tick, 'next tick was triggered';
 
 # Reset
 $reactor->next_tick(sub { die 'Reset failed' });