Fix #25. Selecting panes on load.

Author: tony

CHANGES

@@ -4,6 +4,13 @@ Changelog
 
 Here you can find the recent changes to tmuxp.
 
+0.1-rc3
+-------
+
+- [bug] [tests] `Issue #25`_ - ``focus: true`` not working in panes. Add
+  tests for focusing panes in config.
+- [internal] :meth:`Pane.select_pane()`.
+
 0.1-rc2
 -------
 

TODO

@@ -7,11 +7,10 @@ TODO
 Milestone 0.1
 -------------
 
-- `Issue #25`_ - ``focus: true`` not working in panes.
-
 Done
 """"
 
+- `Issue #25`_ - ``focus: true`` not working in panes.
 - `Issue #15`_ - Pane ordering
 - `Issue #21`_ - Python 2.6 errors with directories in unit tests
 - ``shell_command_before`` for `teamocil`_.

tmuxp/pane.py

@@ -157,6 +157,18 @@ def enter(self):
         """
         self.tmux('send-keys', 'Enter')
 
+    def select_pane(self):
+        """Select pane. Return ``self``.
+
+        To select a window object asynchrously. If a ``pane`` object exists
+        and is no longer longer the current window, ``w.select_pane()``
+        will make ``p`` the current pane.
+
+        :rtype: :class:`pane`
+
+        """
+        return self.window.select_pane(self.get('pane_id'))
+
     def __repr__(self):
         return "%s(%s %s)" % (
             self.__class__.__name__,

tmuxp/testsuite/test_workspacebuilder.py

@@ -121,18 +121,19 @@ class FocusAndPaneIndexTest(TmuxTestCase):
       focus: true
       panes:
       - shell_command:
-        - vim
+        - cd ~
       - shell_command:
-        - echo "hey"
+        - cd /var
+        focus: true
       - shell_command:
+        - cd ~
         - echo "moo"
         - top
-        focus: true
     - window_name: window 2
       panes:
       - shell_command:
-        - vim
-        rocus: true
+        - top
+        1ocus: true
       - shell_command:
         - echo "hey"
       - shell_command:
@@ -171,6 +172,20 @@ def test_split_windows(self):
         pane_indexes_should_be = [pane_base_index + x for x in range(0, 3)]
         self.assertListEqual(pane_indexes_should_be, pane_base_indexes)
 
+        w = self.session.attached_window()
+
+        self.assertNotEqual(w.get('window_name'), 'man')
+
+        pane_path = '/var'
+        for i in range(30):
+            p = w.attached_pane()
+            p.server._update_panes()
+            if p.get('pane_current_path') == pane_path:
+                break
+            time.sleep(.2)
+
+        self.assertEqual(p.get('pane_current_path'), pane_path)
+
 
 class WindowOptions(TmuxTestCase):
 

tmuxp/workspacebuilder.py

@@ -133,18 +133,27 @@ def build(self, session=None):
         assert(isinstance(session, Session))
 
         focus = None
+
         for w, wconf in self.iter_create_windows(session):
             assert(isinstance(w, Window))
-            for p in self.iter_create_panes(w, wconf):
+
+            focus_pane = None
+            for p, pconf in self.iter_create_panes(w, wconf):
                 assert(isinstance(p, Pane))
                 p = p
 
                 if 'layout' in wconf:
                     w.select_layout(wconf['layout'])
 
+                if 'focus' in pconf and pconf['focus']:
+                    focus_pane = p
+
             if 'focus' in wconf and wconf['focus']:
                 focus = w
 
+            if focus_pane:
+                focus_pane.select_pane()
+
         if focus:
             focus.select_window()
 
@@ -157,7 +166,7 @@ def iter_create_windows(self, s):
         Applies ``window_options`` to window.
 
         :param session: :class:`Session` from the config
-        :rtype: :class:`Window`
+        :rtype: tuple(:class:`Window`, ``wconf``)
 
         """
         for i, wconf in enumerate(self.sconf['windows'], start=1):
@@ -202,7 +211,7 @@ def iter_create_panes(self, w, wconf):
         :type w: :class:`Window`
         :param wconf: config section for window
         :type wconf: :py:obj:`dict`
-        :rtype: :class:`Pane`
+        :rtype: tuple(:class:`Pane`, ``pconf``)
 
         """
         assert(isinstance(w, Window))
@@ -233,7 +242,7 @@ def iter_create_panes(self, w, wconf):
 
             w.server._update_panes()
 
-            yield p
+            yield p, pconf
 
 
 def freeze(session):