Split init / exec of tmux_cmd for debuggability

tmux_cmd initialization composes the command

the command will be available in the instance attribute
.cmd

.execute() instance method will run tmux_cmd

see Server.cmd for example of new usage

Related #77

Author: tony

libtmux/common.py

@@ -160,7 +160,12 @@ class tmux_cmd(object):
 
     .. code-block:: python
 
-        proc = tmux_cmd('new-session', '-s%' % 'my session')
+        c = tmux_cmd('new-session', '-s%' % 'my session')
+
+        # You can actually see the command in the .cmd attribute
+        print(c.cmd)
+
+        proc = c.execute()
 
         if proc.stderr:
             raise exc.LibTmuxException(
@@ -200,6 +205,8 @@ def __init__(self, *args, **kwargs):
 
         self.cmd = cmd
 
+    def execute(self):
+        cmd = self.cmd
         try:
             self.process = subprocess.Popen(
                 cmd, stdout=subprocess.PIPE, stderr=subprocess.PIPE
@@ -224,6 +231,7 @@ def __init__(self, *args, **kwargs):
                 self.stdout = self.stderr[0]
 
         logger.debug("self.stdout for %s: \n%s" % (" ".join(cmd), self.stdout))
+        return self
 
 
 class TmuxMappingObject(MutableMapping):
@@ -450,7 +458,7 @@ def get_version():
     :class:`distutils.version.LooseVersion`
         tmux version according to :func:`libtmux.common.which`'s tmux
     """
-    proc = tmux_cmd("-V")
+    proc = tmux_cmd("-V").execute()
     if proc.stderr:
         if proc.stderr[0] == "tmux: unknown option -- V":
             if sys.platform.startswith("openbsd"):  # openbsd has no tmux -V

libtmux/server.py

@@ -123,7 +123,7 @@ def cmd(self, *args, **kwargs):
             else:
                 raise ValueError("Server.colors must equal 88 or 256")
 
-        return tmux_cmd(*args, **kwargs)
+        return tmux_cmd(*args, **kwargs).execute()
 
     def _list_sessions(self):
         """

tests/test_common.py

@@ -33,6 +33,9 @@ class Hi(object):
             stdout = ["tmux master"]
             stderr = None
 
+            def execute(self):
+                return self
+
         return Hi()
 
     monkeypatch.setattr(libtmux.common, "tmux_cmd", mock_tmux_cmd)
@@ -51,6 +54,9 @@ class Hi(object):
             stdout = ["tmux next-2.9"]
             stderr = None
 
+            def execute(self):
+                return self
+
         return Hi()
 
     monkeypatch.setattr(libtmux.common, "tmux_cmd", mock_tmux_cmd)
@@ -66,6 +72,9 @@ def mock_tmux_cmd(param):
         class Hi(object):
             stderr = ["tmux: unknown option -- V"]
 
+            def execute(self):
+                return self
+
         return Hi()
 
     monkeypatch.setattr(libtmux.common, "tmux_cmd", mock_tmux_cmd)
@@ -83,6 +92,9 @@ def mock_tmux_cmd(param):
         class Hi(object):
             stderr = ["tmux: unknown option -- V"]
 
+            def execute(self):
+                return self
+
         return Hi()
 
     monkeypatch.setattr(libtmux.common, "tmux_cmd", mock_tmux_cmd)
@@ -183,6 +195,12 @@ def test_tmux_cmd_unicode(session):
     session.cmd("new-window", "-t", 3, "-n", "юникод", "-F", "Ελληνικά")
 
 
+def test_tmux_cmd_makes_cmd_available():
+    """tmux_cmd objects should make .cmd attribute available."""
+    command = tmux_cmd("-V")
+    assert hasattr(command, "cmd")
+
+
 @pytest.mark.parametrize(
     "session_name,raises,exc_msg_regex",
     [