testcases: don't sync_with_i3() before $x->input_focus

since input_focus calls it anyways

diff --git a/testcases/lib/i3test.pm b/testcases/lib/i3test.pm
index d1a17865eb877146e0c4a24bbcc61156b85e0e6b..44ce7f2369a290806b77f15d8b03cff475de9faa 100644 (file)
--- a/testcases/lib/i3test.pm
+++ b/testcases/lib/i3test.pm
@@ -173,8 +173,6 @@ sub open_window {
 
     $window->map;
     wait_for_map($window);
-    # We sync with i3 here to make sure $x->input_focus is updated.
-    sync_with_i3();
     return $window;
 }
 

diff --git a/testcases/t/101-focus.t b/testcases/t/101-focus.t
index 8f6038e6c0e91940b50bb32197ae20a6f2fae220..8a795c46c0a3d4f8a555fa08f5700c0be00b9cf0 100644 (file)
--- a/testcases/t/101-focus.t
+++ b/testcases/t/101-focus.t
@@ -25,7 +25,6 @@ sub focus_after {
     my $msg = shift;
 
     cmd $msg;
-    sync_with_i3;
     return $x->input_focus;
 }
 

diff --git a/testcases/t/111-goto.t b/testcases/t/111-goto.t
index fa9aaf3c0a62727023a2b95a9a3a3125f3d4ca98..dd60d9cf200d33964b8471cd306d2e43be3410ae 100644 (file)
--- a/testcases/t/111-goto.t
+++ b/testcases/t/111-goto.t
@@ -24,7 +24,6 @@ sub focus_after {
     my $msg = shift;
 
     cmd $msg;
-    sync_with_i3;
     return $x->input_focus;
 }
 

diff --git a/testcases/t/135-floating-focus.t b/testcases/t/135-floating-focus.t
index 4955618fcb193d6988a3bd6a9f501ea4a4544fe4..c7218130166f082f22a8dcd2a991df3fcc000fc7 100644 (file)
--- a/testcases/t/135-floating-focus.t
+++ b/testcases/t/135-floating-focus.t
@@ -36,8 +36,6 @@ cmd 'floating enable';
 
 cmd '[id="' . $second->id . '"] focus';
 
-sync_with_i3;
-
 is($x->input_focus, $second->id, 'second con focused');
 
 cmd 'floating enable';
@@ -67,8 +65,6 @@ cmd 'floating enable';
 
 cmd '[id="' . $second->id . '"] focus';
 
-sync_with_i3;
-
 is($x->input_focus, $second->id, 'second con focused');
 
 cmd 'floating enable';
@@ -103,8 +99,6 @@ cmd 'floating enable';
 
 cmd '[id="' . $second->id . '"] focus';
 
-sync_with_i3;
-
 is($x->input_focus, $second->id, 'second con focused');
 
 cmd 'floating enable';
@@ -132,8 +126,6 @@ $tmp = fresh_workspace;
 $first = open_window({ background_color => '#ff0000' });    # window 8
 $second = open_window({ background_color => '#00ff00' });   # window 9
 
-sync_with_i3;
-
 is($x->input_focus, $second->id, 'second container focused');
 
 cmd 'floating enable';
@@ -142,32 +134,22 @@ is($x->input_focus, $second->id, 'second container focused');
 
 cmd 'focus tiling';
 
-sync_with_i3;
-
 is($x->input_focus, $first->id, 'first (tiling) container focused');
 
 cmd 'focus floating';
 
-sync_with_i3;
-
 is($x->input_focus, $second->id, 'second (floating) container focused');
 
 cmd 'focus floating';
 
-sync_with_i3;
-
 is($x->input_focus, $second->id, 'second (floating) container still focused');
 
 cmd 'focus mode_toggle';
 
-sync_with_i3;
-
 is($x->input_focus, $first->id, 'first (tiling) container focused');
 
 cmd 'focus mode_toggle';
 
-sync_with_i3;
-
 is($x->input_focus, $second->id, 'second (floating) container focused');
 
 #############################################################################
@@ -180,38 +162,26 @@ $first = open_floating_window({ background_color => '#ff0000' });# window 10
 $second = open_floating_window({ background_color => '#00ff00' }); # window 11
 $third = open_floating_window({ background_color => '#0000ff' }); # window 12
 
-sync_with_i3;
-
 is($x->input_focus, $third->id, 'third container focused');
 
 cmd 'focus left';
 
-sync_with_i3;
-
 is($x->input_focus, $second->id, 'second container focused');
 
 cmd 'focus left';
 
-sync_with_i3;
-
 is($x->input_focus, $first->id, 'first container focused');
 
 cmd 'focus left';
 
-sync_with_i3;
-
 is($x->input_focus, $third->id, 'focus wrapped to third container');
 
 cmd 'focus right';
 
-sync_with_i3;
-
 is($x->input_focus, $first->id, 'focus wrapped to first container');
 
 cmd 'focus right';
 
-sync_with_i3;
-
 is($x->input_focus, $second->id, 'focus on second container');
 
 done_testing;

diff --git a/testcases/t/140-focus-lost.t b/testcases/t/140-focus-lost.t
index e7388fd9bac8317058f2a3a1da93b0ef426c609e..2d3c157013ce8106fb51599ccb51143950b3430b 100644 (file)
--- a/testcases/t/140-focus-lost.t
+++ b/testcases/t/140-focus-lost.t
@@ -22,8 +22,6 @@ my $left = open_window;
 my $mid = open_window;
 my $right = open_window;
 
-sync_with_i3;
-
 diag("left = " . $left->id . ", mid = " . $mid->id . ", right = " . $right->id);
 
 is($x->input_focus, $right->id, 'Right window focused');

diff --git a/testcases/t/141-resize.t b/testcases/t/141-resize.t
index 3f332ce10f488251abfd76b60a890365ca1e9e69..0b429fb0c64e9fb65b55d5852493456e8f09e9f0 100644 (file)
--- a/testcases/t/141-resize.t
+++ b/testcases/t/141-resize.t
@@ -10,8 +10,6 @@ cmd 'split v';
 my $top = open_window;
 my $bottom = open_window;
 
-sync_with_i3;
-
 diag("top = " . $top->id . ", bottom = " . $bottom->id);
 
 is($x->input_focus, $bottom->id, 'Bottom window focused');

diff --git a/testcases/t/156-fullscreen-focus.t b/testcases/t/156-fullscreen-focus.t
index 75133cae3973149f00f3e85cddb2e27cbdc96c2a..c7646e417e7c760c11fdf68c717ea41ced67ee65 100644 (file)
--- a/testcases/t/156-fullscreen-focus.t
+++ b/testcases/t/156-fullscreen-focus.t
@@ -57,9 +57,6 @@ my $tmp2 = get_unused_workspace;
 cmd "move workspace $tmp2";
 
 # verify that the third window has the focus
-
-sync_with_i3;
-
 is($x->input_focus, $third->id, 'third window focused');
 
 done_testing;

diff --git a/testcases/t/163-wm-state.t b/testcases/t/163-wm-state.t
index b14dd5b5c925b6cd9ec5106f7ffc218e71576495..6df2bcbddf0392a5c7d71dec0da1a158818d01f9 100644 (file)
--- a/testcases/t/163-wm-state.t
+++ b/testcases/t/163-wm-state.t
@@ -9,8 +9,6 @@ use X11::XCB qw(ICCCM_WM_STATE_NORMAL ICCCM_WM_STATE_WITHDRAWN);
 
 my $window = open_window;
 
-sync_with_i3;
-
 is($window->state, ICCCM_WM_STATE_NORMAL, 'WM_STATE normal');
 
 $window->unmap;

diff --git a/testcases/t/164-kill-win-vs-client.t b/testcases/t/164-kill-win-vs-client.t
index 5c11234109e3764221446bb79aeb1888e1c83dce..165a32cec0f90438e31d4d6472104534af55cade 100644 (file)
--- a/testcases/t/164-kill-win-vs-client.t
+++ b/testcases/t/164-kill-win-vs-client.t
@@ -14,8 +14,6 @@ sub two_windows {
     my $first = open_window;
     my $second = open_window;
 
-    sync_with_i3;
-
     is($x->input_focus, $second->id, 'second window focused');
     ok(@{get_ws_content($tmp)} == 2, 'two containers opened');
 

diff --git a/testcases/t/167-workspace_layout.t b/testcases/t/167-workspace_layout.t
index e6704d2406f33aaf1d2e962934e2480280f0168b..aeb700bad0a1f945f9571f0e9553ae95d4a53b21 100644 (file)
--- a/testcases/t/167-workspace_layout.t
+++ b/testcases/t/167-workspace_layout.t
@@ -26,8 +26,6 @@ ok(@{get_ws_content($tmp)} == 0, 'no containers yet');
 my $first = open_window;
 my $second = open_window;
 
-sync_with_i3;
-
 is($x->input_focus, $second->id, 'second window focused');
 my @content = @{get_ws_content($tmp)};
 ok(@content == 2, 'two containers opened');
@@ -56,8 +54,6 @@ ok(@{get_ws_content($tmp)} == 0, 'no containers yet');
 $first = open_window;
 $second = open_window;
 
-sync_with_i3;
-
 is($x->input_focus, $second->id, 'second window focused');
 @content = @{get_ws_content($tmp)};
 ok(@content == 1, 'one con at workspace level');

diff --git a/testcases/t/170-force_focus_wrapping.t b/testcases/t/170-force_focus_wrapping.t
index 69d146882ba9e1c7b795c5a23b09a17b442be71e..30458b00a2920503dfe3b1c70882f331a3d7a602 100644 (file)
--- a/testcases/t/170-force_focus_wrapping.t
+++ b/testcases/t/170-force_focus_wrapping.t
@@ -70,8 +70,6 @@ cmd 'focus parent';
 
 $third = open_window;
 
-sync_with_i3;
-
 is($x->input_focus, $third->id, 'third window focused');
 
 cmd 'focus left';