X-Git-Url: https://git.sur5r.net/?a=blobdiff_plain;f=testcases%2Ft%2F124-move.t;h=88a36a5b333fc67d38da8019247a4fa5662b3b4a;hb=f62eb9f533a32622fa1cfe13526c33e4e35eb872;hp=ae989f065d0727eff7ede111aa971616312d0bc1;hpb=c7ea8c42b679be191e843f653f5024846ce42fe9;p=i3%2Fi3

diff --git a/testcases/t/124-move.t b/testcases/t/124-move.t
index ae989f06..88a36a5b 100644
--- a/testcases/t/124-move.t
+++ b/testcases/t/124-move.t
@@ -1,6 +1,19 @@
 #!perl
 # vim:ts=4:sw=4:expandtab
 #
+# Please read the following documents before working on tests:
+# • http://build.i3wm.org/docs/testsuite.html
+#   (or docs/testsuite)
+#
+# • http://build.i3wm.org/docs/lib-i3test.html
+#   (alternatively: perldoc ./testcases/lib/i3test.pm)
+#
+# • http://build.i3wm.org/docs/ipc.html
+#   (or docs/ipc)
+#
+# • http://onyxneon.com/books/modern_perl/modern_perl_a4.pdf
+#   (unless you are already familiar with Perl)
+#
 # Tests moving. Basically, there are four different code-paths:
 # 1) move a container which cannot be moved (single container on a workspace)
 # 2) move a container before another single container
@@ -100,14 +113,12 @@ is($nodes->[1]->{id}, $second, 'second container on bottom');
 
 # move it outside again
 cmd 'move left';
-$content = get_ws_content($tmp);
-is(@{$content}, 3, 'three nodes on this workspace');
+is_num_children($tmp, 3, 'three containers after moving left');
 
 # due to automatic flattening/cleanup, the remaining split container
 # will be replaced by the con itself, so we will still have 3 nodes
 cmd 'move right';
-$content = get_ws_content($tmp);
-is(@{$content}, 2, 'two nodes on this workspace');
+is_num_children($tmp, 2, 'two containers after moving right (flattening)');
 
 ######################################################################
 # 4) We create two v-split containers on the workspace, then we move
@@ -128,8 +139,7 @@ cmd "move right";
 cmd 'focus left';
 cmd "move right";
 
-$content = get_ws_content($otmp);
-is(@{$content}, 1, 'only one nodes on this workspace');
+is_num_children($otmp, 1, 'only one node on this workspace');
 
 ######################################################################
 # 5) test moving floating containers.
@@ -141,6 +151,8 @@ my ($absolute_before, $top_before) = $floatwin->rect;
 
 cmd 'move left';
 
+sync_with_i3;
+
 my ($absolute, $top) = $floatwin->rect;
 
 is($absolute->x, ($absolute_before->x - 10), 'moved 10 px to the left');
@@ -153,6 +165,8 @@ $top_before = $top;
 
 cmd 'move right';
 
+sync_with_i3;
+
 ($absolute, $top) = $floatwin->rect;
 
 is($absolute->x, ($absolute_before->x + 10), 'moved 10 px to the right');
@@ -165,6 +179,8 @@ $top_before = $top;
 
 cmd 'move up';
 
+sync_with_i3;
+
 ($absolute, $top) = $floatwin->rect;
 
 is($absolute->x, $absolute_before->x, 'x not changed');
@@ -177,6 +193,8 @@ $top_before = $top;
 
 cmd 'move down';
 
+sync_with_i3;
+
 ($absolute, $top) = $floatwin->rect;
 
 is($absolute->x, $absolute_before->x, 'x not changed');
@@ -193,6 +211,8 @@ $top_before = $top;
 
 cmd 'move left 20 px';
 
+sync_with_i3;
+
 ($absolute, $top) = $floatwin->rect;
 
 is($absolute->x, ($absolute_before->x - 20), 'moved 20 px to the left');
@@ -200,6 +220,45 @@ is($absolute->y, $absolute_before->y, 'y not changed');
 is($absolute->width, $absolute_before->width, 'width not changed');
 is($absolute->height, $absolute_before->height, 'height not changed');
 
+######################################################################
+# 6) test moving floating window to a specified position
+#    and to absolute center
+######################################################################
+
+$tmp = fresh_workspace;
+open_floating_window; my @floatcon;
+
+cmd 'move position 5 px 15 px';
+
+@floatcon = @{get_ws($tmp)->{floating_nodes}};
+
+is($floatcon[0]->{rect}->{x}, 5, 'moved to position 5 x');
+is($floatcon[0]->{rect}->{y}, 15, 'moved to position 15 y');
+
+cmd 'move absolute position center';
+
+@floatcon = @{get_ws($tmp)->{floating_nodes}};
+
+my $center_x = int($x->root->rect->width/2) - int($floatcon[0]->{rect}->{width}/2);
+my $center_y = int($x->root->rect->height/2) - int($floatcon[0]->{rect}->{height}/2);
+
+is($floatcon[0]->{rect}->{x}, $center_x, "moved to center at position $center_x x");
+is($floatcon[0]->{rect}->{y}, $center_y, "moved to center at position $center_y y");
+
+# Make sure the command works with criteria
+open_floating_window;
+
+@floatcon = @{get_ws($tmp)->{floating_nodes}};
+
+cmd '[con_id="' . $floatcon[0]->{nodes}[0]->{id} . '"] move position 25 px 30 px';
+cmd '[con_id="' . $floatcon[1]->{nodes}[0]->{id} . '"] move position 35 px 40 px';
+
+@floatcon = @{get_ws($tmp)->{floating_nodes}};
+
+is($floatcon[0]->{rect}->{x}, 25, 'moved to position 25 x with criteria');
+is($floatcon[0]->{rect}->{y}, 30, 'moved to position 30 y with criteria');
 
+is($floatcon[1]->{rect}->{x}, 35, 'moved to position 35 x with criteria');
+is($floatcon[1]->{rect}->{y}, 40, 'moved to position 40 y with criteria');
 
 done_testing;