]> git.sur5r.net Git - i3/i3/commitdiff
testcases: drop sync_with_i3()s $x parameter, use global
authorMaik Fischer <maikf@qu.cx>
Tue, 22 Nov 2011 00:13:37 +0000 (01:13 +0100)
committerMichael Stapelberg <michael@stapelberg.de>
Wed, 23 Nov 2011 00:02:50 +0000 (00:02 +0000)
26 files changed:
testcases/lib/i3test.pm
testcases/t/002-i3-sync.t
testcases/t/003-ipc.t
testcases/t/005-floating.t
testcases/t/100-fullscreen.t
testcases/t/101-focus.t
testcases/t/102-dock.t
testcases/t/111-goto.t
testcases/t/112-floating-resize.t
testcases/t/113-urgent.t
testcases/t/119-match.t
testcases/t/133-size-hints.t
testcases/t/135-floating-focus.t
testcases/t/137-floating-unmap.t
testcases/t/140-focus-lost.t
testcases/t/141-resize.t
testcases/t/153-floating-originalsize.t
testcases/t/156-fullscreen-focus.t
testcases/t/162-regress-dock-urgent.t
testcases/t/163-wm-state.t
testcases/t/164-kill-win-vs-client.t
testcases/t/165-for_window.t
testcases/t/166-assign.t
testcases/t/167-workspace_layout.t
testcases/t/170-force_focus_wrapping.t
testcases/t/175-startup-notification.t

index d3d8ad04576fc4456f09d4d0a4296e29a2f144fd..d1a17865eb877146e0c4a24bbcc61156b85e0e6b 100644 (file)
@@ -139,7 +139,7 @@ sub wait_for_unmap {
     wait_for_event 2, sub {
         $_[0]->{response_type} == UNMAP_NOTIFY # and $_[0]->{window} == $id
     };
-    sync_with_i3($x);
+    sync_with_i3();
 }
 
 #
@@ -174,7 +174,7 @@ 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($x);
+    sync_with_i3();
     return $window;
 }
 
@@ -325,8 +325,6 @@ sub focused_ws {
 # See also docs/testsuite for a long explanation
 #
 sub sync_with_i3 {
-    my ($x) = @_;
-
     # Since we need a (mapped) window for receiving a ClientMessage, we create
     # one on the first call of sync_with_i3. It will be re-used in all
     # subsequent calls.
@@ -470,7 +468,7 @@ use parent 'X11::XCB::Connection';
 
 sub input_focus {
     my $self = shift;
-    i3test::sync_with_i3($self);
+    i3test::sync_with_i3();
 
     return $self->SUPER::input_focus(@_);
 }
index 75871f9deaa190850c93aab1b09a898c92407311..1377ee942407f8c615273212ea82e563b1134441 100644 (file)
@@ -5,7 +5,7 @@
 #
 use i3test;
 
-my $result = sync_with_i3($x);
+my $result = sync_with_i3;
 ok($result, 'syncing was successful');
 
 done_testing;
index 477b7163a64f3bebdeb677d342af59dd675b6e82..34359f20a7e28410378666bbd0f2c3665fa12295 100644 (file)
@@ -17,7 +17,7 @@ my $focus = $x->input_focus;
 # Switch to another workspace
 fresh_workspace;
 
-sync_with_i3($x);
+sync_with_i3;
 my $new_focus = $x->input_focus;
 isnt($focus, $new_focus, "Focus changed");
 
index ac6623f2b383f8108b0a339170397bbab4208dd1..6de3ea8060e1bb5e444f69f717ee4f641ff59a49 100644 (file)
@@ -76,7 +76,7 @@ $window->map;
 wait_for_map $window;
 
 cmd 'floating enable';
-sync_with_i3($x);
+sync_with_i3;
 
 ($absolute, $top) = $window->rect;
 
index f5fb66932d3f66c246fc7d706ec1925da3a3f177..dc7eec188a01a0f619906e3e6bc6c7fc97bddd54 100644 (file)
@@ -56,7 +56,7 @@ $original_rect = $new_rect;
 
 $window->fullscreen(1);
 
-sync_with_i3($x);
+sync_with_i3;
 
 $new_rect = $window->rect;
 ok(!eq_deeply($new_rect, $original_rect), "Window got repositioned after fullscreen");
@@ -123,7 +123,7 @@ my $swindow = $x->root->create_child(
 
 $swindow->map;
 
-sync_with_i3($x);
+sync_with_i3;
 
 ok(!$swindow->mapped, 'window not mapped while fullscreen window active');
 
@@ -131,12 +131,12 @@ $new_rect = $swindow->rect;
 ok(!eq_deeply($new_rect, $original_rect), "Window got repositioned");
 
 $swindow->fullscreen(1);
-sync_with_i3($x);
+sync_with_i3;
 
 is(fullscreen_windows(), 1, 'amount of fullscreen windows');
 
 $window->fullscreen(0);
-sync_with_i3($x);
+sync_with_i3;
 is(fullscreen_windows(), 0, 'amount of fullscreen windows');
 
 ok($swindow->mapped, 'window mapped after other fullscreen ended');
@@ -148,7 +148,7 @@ ok($swindow->mapped, 'window mapped after other fullscreen ended');
 ###########################################################################
 
 $swindow->fullscreen(0);
-sync_with_i3($x);
+sync_with_i3;
 
 is(fullscreen_windows(), 0, 'amount of fullscreen windows after disabling');
 
index 545b9dd05907b948eae7dc4d60ccd9fcff18565c..8f6038e6c0e91940b50bb32197ae20a6f2fae220 100644 (file)
@@ -25,7 +25,7 @@ sub focus_after {
     my $msg = shift;
 
     cmd $msg;
-    sync_with_i3 $x;
+    sync_with_i3;
     return $x->input_focus;
 }
 
index ce33f052166fdf55e86e3617e34a430ad4b03dfe..20acf49e033ded8426b45b9f54671b06006c618d 100644 (file)
@@ -52,7 +52,7 @@ is($docknode->{rect}->{height}, 30, 'dock node has unchanged height');
 
 $window->rect(X11::XCB::Rect->new(x => 0, y => 0, width => 50, height => 40));
 
-sync_with_i3 $x;
+sync_with_i3;
 
 @docked = get_dock_clients('top');
 is(@docked, 1, 'one dock client found');
index 5f4d7762702240a05e67f1b07f648059a4524f8f..fa9aaf3c0a62727023a2b95a9a3a3125f3d4ca98 100644 (file)
@@ -24,7 +24,7 @@ sub focus_after {
     my $msg = shift;
 
     cmd $msg;
-    sync_with_i3($x);
+    sync_with_i3;
     return $x->input_focus;
 }
 
index fe9803e6c1ca7de641805a3faef861a846f60bb8..ff3a1c04e0922cf3b3cd759b278f27894e2f8651 100644 (file)
@@ -15,7 +15,7 @@ my $window = open_floating_window;
 my ($a, $t) = $window->rect;
 $window->rect(X11::XCB::Rect->new(x => $a->x, y => $a->y, width => $a->width, height => $a->height));
 
-sync_with_i3($x);
+sync_with_i3;
 
 my ($na, $nt) = $window->rect;
 is_deeply($na, $a, 'Rects are equal after configurerequest');
@@ -23,7 +23,7 @@ is_deeply($na, $a, 'Rects are equal after configurerequest');
 sub test_resize {
     $window->rect(X11::XCB::Rect->new(x => 0, y => 0, width => 100, height => 100));
 
-    sync_with_i3($x);
+    sync_with_i3;
 
     my ($absolute, $top) = $window->rect;
 
@@ -34,7 +34,7 @@ sub test_resize {
 
     $window->rect(X11::XCB::Rect->new(x => 0, y => 0, width => 300, height => 500));
 
-    sync_with_i3($x);
+    sync_with_i3;
 
     ($absolute, $top) = $window->rect;
 
index fcfd11e7dd318237956297d734b2ba81a0145478..5ee61a4d2b887c34a2776e3920a27743ffefe2df 100644 (file)
@@ -24,7 +24,7 @@ is(@urgent, 0, 'no window got the urgent flag');
 # Add the urgency hint, switch to a different workspace and back again
 #####################################################################
 $top->add_hint('urgency');
-sync_with_i3($x);
+sync_with_i3;
 
 my @content = @{get_ws_content($tmp)};
 @urgent = grep { $_->{urgent} } @content;
@@ -41,7 +41,7 @@ cmd '[id="' . $top->id . '"] focus';
 is(@urgent, 0, 'no window got the urgent flag after focusing');
 
 $top->add_hint('urgency');
-sync_with_i3($x);
+sync_with_i3;
 
 @urgent = grep { $_->{urgent} } @{get_ws_content($tmp)};
 is(@urgent, 0, 'no window got the urgent flag after re-setting urgency hint');
@@ -55,7 +55,7 @@ ok(!$ws->{urgent}, 'urgent flag not set on workspace');
 my $otmp = fresh_workspace;
 
 $top->add_hint('urgency');
-sync_with_i3($x);
+sync_with_i3;
 
 $ws = get_ws($tmp);
 ok($ws->{urgent}, 'urgent flag set on workspace');
index e2e553f439b372c52cabcf3ee5c579c7a47bec1a..2124c02ef7d28a472bf4969af68480a8453b35d0 100644 (file)
@@ -101,7 +101,7 @@ ok(@{$content} == 2, 'two windows opened');
 
 cmd '[class="special" title="left"] kill';
 
-sync_with_i3($x);
+sync_with_i3;
 
 $content = get_ws_content($tmp);
 is(@{$content}, 1, 'one window still there');
index 34b5e4a5be7bd71a47c17034a243c1ef062ee377..d3736e3c35fee0d64db0546b4de63adcdd8da24b 100644 (file)
@@ -23,7 +23,7 @@ wait_for_map $win;
 $win->hints->aspect($aspect);
 $x->flush;
 
-sync_with_i3($x);
+sync_with_i3;
 
 my $rect = $win->rect;
 my $ar = $rect->width / $rect->height;
index f5182e1b0f2c6a31a7cac6dd9f0ed895e18e2dbf..4955618fcb193d6988a3bd6a9f501ea4a4544fe4 100644 (file)
@@ -36,7 +36,7 @@ cmd 'floating enable';
 
 cmd '[id="' . $second->id . '"] focus';
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($x->input_focus, $second->id, 'second con focused');
 
@@ -67,7 +67,7 @@ cmd 'floating enable';
 
 cmd '[id="' . $second->id . '"] focus';
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($x->input_focus, $second->id, 'second con focused');
 
@@ -103,13 +103,13 @@ cmd 'floating enable';
 
 cmd '[id="' . $second->id . '"] focus';
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($x->input_focus, $second->id, 'second con focused');
 
 cmd 'floating enable';
 
-sync_with_i3($x);
+sync_with_i3;
 
 # now kill the second one. focus should fall back to the third one, which is
 # also floating
@@ -132,7 +132,7 @@ $tmp = fresh_workspace;
 $first = open_window({ background_color => '#ff0000' });    # window 8
 $second = open_window({ background_color => '#00ff00' });   # window 9
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($x->input_focus, $second->id, 'second container focused');
 
@@ -142,31 +142,31 @@ is($x->input_focus, $second->id, 'second container focused');
 
 cmd 'focus tiling';
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($x->input_focus, $first->id, 'first (tiling) container focused');
 
 cmd 'focus floating';
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($x->input_focus, $second->id, 'second (floating) container focused');
 
 cmd 'focus floating';
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($x->input_focus, $second->id, 'second (floating) container still focused');
 
 cmd 'focus mode_toggle';
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($x->input_focus, $first->id, 'first (tiling) container focused');
 
 cmd 'focus mode_toggle';
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($x->input_focus, $second->id, 'second (floating) container focused');
 
@@ -180,37 +180,37 @@ $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($x);
+sync_with_i3;
 
 is($x->input_focus, $third->id, 'third container focused');
 
 cmd 'focus left';
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($x->input_focus, $second->id, 'second container focused');
 
 cmd 'focus left';
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($x->input_focus, $first->id, 'first container focused');
 
 cmd 'focus left';
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($x->input_focus, $third->id, 'focus wrapped to third container');
 
 cmd 'focus right';
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($x->input_focus, $first->id, 'focus wrapped to first container');
 
 cmd 'focus right';
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($x->input_focus, $second->id, 'focus on second container');
 
index 0d126feab0d267cae5200887de8aec04d2b25e42..e91870bc35e0ee1fd6bd1a203803f96c1355635a 100644 (file)
@@ -21,7 +21,7 @@ ok($window->mapped, 'Window is mapped');
 
 my $otmp = fresh_workspace;
 
-sync_with_i3($x);
+sync_with_i3;
 
 ok(!$window->mapped, 'Window is not mapped after switching ws');
 
index 042afcfefd9e59e21d7c26a628e94c471301e06b..e7388fd9bac8317058f2a3a1da93b0ef426c609e 100644 (file)
@@ -22,7 +22,7 @@ my $left = open_window;
 my $mid = open_window;
 my $right = open_window;
 
-sync_with_i3($x);
+sync_with_i3;
 
 diag("left = " . $left->id . ", mid = " . $mid->id . ", right = " . $right->id);
 
index b2c7d0f36226befb269722f141478dd35e74e359..3f332ce10f488251abfd76b60a890365ca1e9e69 100644 (file)
@@ -10,7 +10,7 @@ cmd 'split v';
 my $top = open_window;
 my $bottom = open_window;
 
-sync_with_i3($x);
+sync_with_i3;
 
 diag("top = " . $top->id . ", bottom = " . $bottom->id);
 
index dcd1144e76dede059bd0d9f8bb02153548f7ffc4..83f3e85d58bf7cac5d9fe823d0c6ed07b57bda1c 100644 (file)
@@ -17,7 +17,7 @@ cmp_ok($absolute->{width}, '>', 400, 'i3 raised the width');
 cmp_ok($absolute->{height}, '>', 150, 'i3 raised the height');
 
 cmd 'floating toggle';
-sync_with_i3($x);
+sync_with_i3;
 
 ($absolute, $top) = $window->rect;
 
index 78323990e14c85774174722fb8e18fbdf607b1f9..75133cae3973149f00f3e85cddb2e27cbdc96c2a 100644 (file)
@@ -46,7 +46,7 @@ my $third = open_window({
 
 $third->map;
 
-sync_with_i3 $x;
+sync_with_i3;
 
 diag("third = " . $third->id);
 
@@ -58,7 +58,7 @@ cmd "move workspace $tmp2";
 
 # verify that the third window has the focus
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($x->input_focus, $third->id, 'third window focused');
 
index 39e4eac4bfb12c042cb76e850effd4065196303c..7a0ac48788de3258c0f9788530ca37590ee828ac 100644 (file)
@@ -47,7 +47,7 @@ is($docknode->{rect}->{height}, 30, 'dock node has unchanged height');
 
 $window->add_hint('urgency');
 
-sync_with_i3($x);
+sync_with_i3;
 
 does_i3_live;
 
index 468499d502c6cd0a8f9248b3582b02a098afc87b..b14dd5b5c925b6cd9ec5106f7ffc218e71576495 100644 (file)
@@ -9,7 +9,7 @@ use X11::XCB qw(ICCCM_WM_STATE_NORMAL ICCCM_WM_STATE_WITHDRAWN);
 
 my $window = open_window;
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($window->state, ICCCM_WM_STATE_NORMAL, 'WM_STATE normal');
 
index 20dd50ca349aa40596ef7e72098900cc702dcb6c..5c11234109e3764221446bb79aeb1888e1c83dce 100644 (file)
@@ -14,7 +14,7 @@ sub two_windows {
     my $first = open_window;
     my $second = open_window;
 
-    sync_with_i3 $x;
+    sync_with_i3;
 
     is($x->input_focus, $second->id, 'second window focused');
     ok(@{get_ws_content($tmp)} == 2, 'two containers opened');
index 281f840f6658eb71ceee7b0bf914d10402eb18e8..c7601809da764b6e2b99584ab8deca9d6bad536b 100644 (file)
@@ -124,13 +124,13 @@ cmp_ok(@content, '==', 1, 'one node on this workspace now');
 is($content[0]->{border}, 'normal', 'normal border');
 
 $window->name('special borderless title');
-sync_with_i3 $x;
+sync_with_i3;
 
 @content = @{get_ws_content($tmp)};
 is($content[0]->{border}, 'none', 'no border');
 
 $window->name('special title');
-sync_with_i3 $x;
+sync_with_i3;
 
 cmd 'border normal';
 
@@ -138,7 +138,7 @@ cmd 'border normal';
 is($content[0]->{border}, 'normal', 'border reset to normal');
 
 $window->name('special borderless title');
-sync_with_i3 $x;
+sync_with_i3;
 
 @content = @{get_ws_content($tmp)};
 is($content[0]->{border}, 'normal', 'still normal border');
@@ -235,7 +235,7 @@ wait_for_unmap $window;
 $window->destroy;
 
 # give i3 a chance to delete the window from its tree
-sync_with_i3 $x;
+sync_with_i3;
 
 @content = @{get_ws_content($tmp)};
 cmp_ok(@content, '==', 0, 'no nodes on this workspace now');
@@ -457,7 +457,7 @@ $x->change_property(
 
 $x->flush;
 
-sync_with_i3 $x;
+sync_with_i3;
 
 @content = @{get_ws_content($tmp)};
 cmp_ok(@content, '==', 1, 'one node on this workspace now');
index f42962f3ace0f2632ad5b703e5d626c7aa7c1570..a10516a61254db6eb50a56f5705ab476350a292f 100644 (file)
@@ -135,7 +135,7 @@ $window->map;
 # We use sync_with_i3 instead of wait_for_map here because i3 will not actually
 # map the window -- it will be assigned to a different workspace and will only
 # be mapped once you switch to that workspace
-sync_with_i3 $x;
+sync_with_i3;
 
 ok(@{get_ws_content($tmp)} == 0, 'still no containers');
 ok(@{get_ws_content('targetws')} == 2, 'two containers on targetws');
index d5a02ca626302bcf355242a3791cac8526940d98..e6704d2406f33aaf1d2e962934e2480280f0168b 100644 (file)
@@ -26,7 +26,7 @@ ok(@{get_ws_content($tmp)} == 0, 'no containers yet');
 my $first = open_window;
 my $second = open_window;
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($x->input_focus, $second->id, 'second window focused');
 my @content = @{get_ws_content($tmp)};
@@ -56,7 +56,7 @@ ok(@{get_ws_content($tmp)} == 0, 'no containers yet');
 $first = open_window;
 $second = open_window;
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($x->input_focus, $second->id, 'second window focused');
 @content = @{get_ws_content($tmp)};
index bf66c44e8b45b61f289907eb638413fa018617ac..69d146882ba9e1c7b795c5a23b09a17b442be71e 100644 (file)
@@ -70,7 +70,7 @@ cmd 'focus parent';
 
 $third = open_window;
 
-sync_with_i3($x);
+sync_with_i3;
 
 is($x->input_focus, $third->id, 'third window focused');
 
index a794f70ae5d8b8afc6bcab90d96d590d6c334beb..3a4dbc81f3ed5d4a305f725acaab5cc516c3bacd 100644 (file)
@@ -103,7 +103,7 @@ $win->map;
 # We don’t use wait_for_map because the window will not get mapped -- it is on
 # a different workspace.
 # We sync with i3 here to make sure $x->input_focus is updated.
-sync_with_i3($x);
+sync_with_i3;
 
 is(@{get_ws_content($second_ws)}, 0, 'still no containers on the second workspace');
 is(@{get_ws_content($first_ws)}, 1, 'one container on the first workspace');
@@ -117,7 +117,7 @@ mark_window($leader->id);
 
 $win = open_window({ dont_map => 1, client_leader => $leader });
 $win->map;
-sync_with_i3($x);
+sync_with_i3;
 
 is(@{get_ws_content($second_ws)}, 0, 'still no containers on the second workspace');
 is(@{get_ws_content($first_ws)}, 2, 'two containers on the first workspace');
@@ -128,7 +128,7 @@ is(@{get_ws_content($first_ws)}, 2, 'two containers on the first workspace');
 ######################################################################
 
 complete_startup();
-sync_with_i3($x);
+sync_with_i3;
 
 my $otherwin = open_window;
 is(@{get_ws_content($second_ws)}, 1, 'one container on the second workspace');