testcases: replace sleep with sync_with_i3 where appropiate

diff --git a/testcases/t/001-tile.t b/testcases/t/001-tile.t
index 460cbd9042f75251595d49e61c89f7cd392d5018..65af8d744220d13c15c7bd9fada0db645333cc46 100644 (file)
--- a/testcases/t/001-tile.t
+++ b/testcases/t/001-tile.t
@@ -18,7 +18,7 @@ is_deeply($window->rect, $original_rect, "rect unmodified before mapping");
 
 $window->map;
 
-sleep(0.25);
+sleep(0.5);
 
 my $new_rect = $window->rect;
 ok(!eq_hash($new_rect, $original_rect), "Window got repositioned");

diff --git a/testcases/t/127-regress-floating-parent.t b/testcases/t/127-regress-floating-parent.t
index 52b8b9c0624693e1ae246fd9e22b8d805b64900c..c83c0809e3c50942704be11a7b04c6f3037c4f1f 100644 (file)
--- a/testcases/t/127-regress-floating-parent.t
+++ b/testcases/t/127-regress-floating-parent.t
@@ -16,8 +16,7 @@ cmd 'open';
 my $floating = get_focused($tmp);
 diag("focused floating: " . get_focused($tmp));
 cmd 'mode toggle';
-# TODO: eliminate this race conditition
-sleep 1;
+sync_with_i3;
 
 # kill old container
 cmd qq|[con_id="$old"] focus|;
@@ -33,7 +32,7 @@ cmd 'kill';
 cmd qq|[con_id="$floating"] focus|;
 is(get_focused($tmp), $floating, 'floating window focused');
 
-sleep 1;
+sync_with_i3;
 cmd 'mode toggle';
 
 does_i3_live;

diff --git a/testcases/t/129-focus-after-close.t b/testcases/t/129-focus-after-close.t
index 3913f4c6d1eb1221ae664c2f849af03a42976d0b..5fc3786e02292bfdbf988053aad25a56fc95a5a0 100644 (file)
--- a/testcases/t/129-focus-after-close.t
+++ b/testcases/t/129-focus-after-close.t
@@ -42,9 +42,8 @@ isnt(get_focused($tmp), $second, 'different container focused');
 ##############################################################
 
 cmd 'kill';
-# TODO: this testcase sometimes has different outcomes when the
-# sleep is missing. why?
-sleep 0.25;
+sync_with_i3;
+
 ($nodes, $focus) = get_ws_content($tmp);
 is($nodes->[1]->{nodes}->[0]->{id}, $second, 'second container found');
 ok($nodes->[1]->{nodes}->[0]->{focused}, 'second container focused');
@@ -103,8 +102,7 @@ my $win = open_window({ background_color => '#00ff00' });
 
 cmd qq|[con_id="$middle"] focus|;
 $win->destroy;
-
-sleep 0.25;
+sync_with_i3;
 
 is(get_focused($tmp), $middle, 'middle container focused');
 

diff --git a/testcases/t/147-regress-floatingmove.t b/testcases/t/147-regress-floatingmove.t
index ed85b57b0f75b16a1e517336703bc3c1e0e49fac..5a2c87d3f4c1e357fe900bc847121af56fa8c8eb 100644 (file)
--- a/testcases/t/147-regress-floatingmove.t
+++ b/testcases/t/147-regress-floatingmove.t
@@ -6,6 +6,11 @@
 #
 use i3test;
 
+sub sync_cmd {
+    cmd @_;
+    sync_with_i3;
+}
+
 my $tmp = fresh_workspace;
 
 my $left = open_window;
@@ -13,23 +18,19 @@ my $mid = open_window;
 my $right = open_window;
 
 # go to workspace level
-cmd 'level up';
-sleep 0.25;
+sync_cmd 'level up';
 
 # make it floating
-cmd 'mode toggle';
-sleep 0.25;
+sync_cmd 'mode toggle';
 
 # move the con outside the floating con
-cmd 'move up';
-sleep 0.25;
+sync_cmd 'move up';
 
 does_i3_live;
 
 # move another con outside
-cmd '[id="' . $mid->id . '"] focus';
-cmd 'move up';
-sleep 0.25;
+sync_cmd '[id="' . $mid->id . '"] focus';
+sync_cmd 'move up';
 
 does_i3_live;
 

diff --git a/testcases/t/148-regress-floatingmovews.t b/testcases/t/148-regress-floatingmovews.t
index 44d5a4454a8a6a00952259f7bd4f37fca89799a8..3d71b5003387355344c443f2def994da406644de 100644 (file)
--- a/testcases/t/148-regress-floatingmovews.t
+++ b/testcases/t/148-regress-floatingmovews.t
@@ -10,20 +10,18 @@ my $tmp = fresh_workspace;
 
 # open a tiling window on the first workspace
 open_window;
-#sleep 0.25;
 my $first = get_focused($tmp);
 
 # on a different ws, open a floating window
 my $otmp = fresh_workspace;
 open_window;
-#sleep 0.25;
 my $float = get_focused($otmp);
 cmd 'mode toggle';
-#sleep 0.25;
+sync_with_i3;
 
 # move the floating con to first workspace
 cmd "move workspace $tmp";
-#sleep 0.25;
+sync_with_i3;
 
 # switch to the first ws and check focus
 is(get_focused($tmp), $float, 'floating client correctly focused');

diff --git a/testcases/t/168-regress-fullscreen-restart.t b/testcases/t/168-regress-fullscreen-restart.t
index 74d284dba63a5a277ebb94f0f9e08d98fc9e420b..27812565f86026cca1000703cd603a7dce0d8e3b 100644 (file)
--- a/testcases/t/168-regress-fullscreen-restart.t
+++ b/testcases/t/168-regress-fullscreen-restart.t
@@ -11,10 +11,10 @@ open_window;
 open_window;
 
 cmd 'layout stacking';
-sleep 1;
+sync_with_i3;
 
 cmd 'fullscreen';
-sleep 1;
+sync_with_i3;
 
 cmd 'restart';
 sleep 1;