let's try autorename test. update select_pane to correctly use -l -U -D -L -R or custom target_pane input

Author: tony

tmuxp/testsuite/test_workspacebuilder.py

@@ -4,6 +4,7 @@
 import os
 import unittest
 import logging
+import time
 import kaptan
 from .. import Window, config, exc
 from ..workspacebuilder import WorkspaceBuilder
@@ -24,7 +25,7 @@ class TwoPaneTest(TmuxTestCase):
         - vim
         start_directory: '~'
       - shell_command:
-        - cowsay "hey"
+        - echo "hey"
       window_name: editor
     - panes:
       - shell_command:
@@ -70,9 +71,9 @@ class ThreePaneTest(TmuxTestCase):
         - vim
         start_directory: '~'
       - shell_command:
-        - cowsay "hey"
+        - echo "hey"
       - shell_command:
-        - cowsay "moo"
+        - echo "moo"
     '''
 
     def test_split_windows(self):
@@ -112,9 +113,9 @@ class FocusAndPaneIndexTest(TmuxTestCase):
         - vim
         start_directory: '~'
       - shell_command:
-        - cowsay "hey"
+        - echo "hey"
       - shell_command:
-        - cowsay "moo"
+        - echo "moo"
         - top
         focus: true
     - window_name: window 2
@@ -124,9 +125,9 @@ class FocusAndPaneIndexTest(TmuxTestCase):
         start_directory: '~'
         focus: true
       - shell_command:
-        - cowsay "hey"
+        - echo "hey"
       - shell_command:
-        - cowsay "moo"
+        - echo "moo"
 
     '''
 
@@ -165,8 +166,6 @@ def test_split_windows(self):
 
 class WindowOptions(TmuxTestCase):
 
-    '''sample config with no session name'''
-
     yaml_config = '''
     session_name: test window options
     start_directory: '~'
@@ -179,9 +178,9 @@ class WindowOptions(TmuxTestCase):
         - vim
         start_directory: '~'
       - shell_command:
-        - cowsay "hey"
+        - echo "hey"
       - shell_command:
-        - cowsay "moo"
+        - echo "moo"
       window_name: editor
     '''
 
@@ -208,6 +207,69 @@ def test_window_options(self):
             w.select_layout(wconf['layout'])
 
 
+class WindowAutomaticRename(TmuxTestCase):
+
+    yaml_config = '''
+    session_name: test window options
+    start_directory: '~'
+    windows:
+    - layout: main-horizontal
+      options:
+        automatic-rename: on
+      panes:
+      - shell_command:
+        - nano
+        start_directory: '~'
+      - shell_command:
+        - echo "hey"
+      - shell_command:
+        - echo "moo"
+    '''
+
+    def test_automatic_rename_option(self):
+        """ with option automatic-rename: on. """
+        s = self.session
+        sconfig = kaptan.Kaptan(handler='yaml')
+        sconfig = sconfig.import_config(self.yaml_config).get()
+
+        builder = WorkspaceBuilder(sconf=sconfig)
+
+        window_count = len(self.session._windows)  # current window count
+        self.assertEqual(len(s._windows), window_count)
+        for w, wconf in builder.iter_create_windows(s):
+
+            window_pane_count = len(w._panes)
+            for p in builder.iter_create_panes(w, wconf):
+                p = p
+                self.assertEqual(len(s._windows), window_count)
+            self.assertIsInstance(w, Window)
+            self.assertEqual(w.show_window_option('automatic-rename'), 'on')
+
+            self.assertEqual(len(s._windows), window_count)
+
+            window_count += 1
+            w.select_layout(wconf['layout'])
+
+        w = s.attached_window()
+
+        for i in range(5):
+            w = s.attached_window()
+            if w['window_name'] == 'nano':
+                break
+            time.sleep(.1)
+
+        self.assertEqual(w.get('window_name'), 'nano')
+
+        w.select_pane('-D')
+        for i in range(5):
+            w = s.attached_window()
+            if w['window_name'] != 'nano':
+                break
+            time.sleep(.1)
+
+        self.assertNotEqual(w.get('window_name'), 'nano')
+
+
 class TestsToDo(object):
 
     def test_uses_first_window_if_exists(self):

tmuxp/window.py

@@ -235,16 +235,17 @@ def select_pane(self, target_pane):
         '''
             ``$ tmux select-pane``
 
-        :param target_pane: ``target_pane``, or ``-U``,``-D``, ``-L``, ``-R``.
+        :param target_pane: ``target_pane``, or ``-U``,``-D``, ``-L``, ``-R``
+            or ``-l``.
         :type target_pane: string
         :rtype: :class:`Pane`
 
-        Todo: make 'up', 'down', 'left', 'right' acceptable ``target_pane``.
         '''
-        # if isinstance(target_pane, basestring) and not ':' not in target_pane or isinstance(target_pane, int):
-        #    target_pane = "%s.%s" % (self.target, target_pane)
 
-        proc = self.tmux('select-pane', '-t%s' % target_pane)
+        if target_pane in ['-l', '-U', '-D', '-L', '-R']:
+            proc = self.tmux('select-pane', '-t%s' % self.get('window_id'), target_pane)
+        else:
+            proc = self.tmux('select-pane', '-t%s' % target_pane)
 
         if proc.stderr:
             raise Exception(proc.stderr)