Fix string concatenation in pep8 in test_config_tmuxinator. PEP257 and PEP8

Author: tony

tmuxp/testsuite/test_config_tmuxinator.py

@@ -150,10 +150,8 @@ class TmuxinatorDeprecationsTest(TestCase):
             {
                 'editor': {
                     'pre': [
-                        (
-                            'echo "I get run in each pane,',
-                            'before each pane command!"'
-                        ),
+                        'echo "I get run in each pane, '
+                        'before each pane command!"',
                         None
                     ],
                     'layout': 'main-vertical',
@@ -283,7 +281,7 @@ def test_config_to_dict(self):
 
 class TmuxinatoriSampleTest(TestCase):
 
-    """Tests against tmuxinator's <spec/fixtures/sample.yml>.
+    """Test importing <spec/fixtures/sample.yml>.
 
     https://github.com/aziz/tmuxinator/blob/master/spec/fixtures/sample.yml
 
@@ -342,10 +340,8 @@ class TmuxinatoriSampleTest(TestCase):
             {
                 'editor': {
                     'pre': [
-                        (
-                            'echo "I get run in each pane, ',
-                            'before each pane command!"'
-                        ),
+                        'echo "I get run in each pane, '
+                        'before each pane command!"',
                         None
                     ],
                     'layout': 'main-vertical',

tmuxp/testsuite/test_tmuxobject.py

@@ -25,15 +25,18 @@ class TmuxObjectTest(TmuxTestCase):
     """Test the :class:`TmuxRelationalObject` base class object."""
 
     def test_findWhere(self):
-        """findWhere."""
+        """Test that findWhere() retrieves single matching object."""
         self.maxDiff = None
         # server.findWhere
         for session in t.sessions:
             session_id = session.get('session_id')
 
             self.assertEqual(t.findWhere({'session_id': session_id}), session)
-            self.assertIsInstance(t.findWhere({
-                                  'session_id': session_id}), Session)
+            self.assertIsInstance(
+                t.findWhere({
+                    'session_id': session_id
+                }), Session
+            )
 
             # session.findWhere
             for window in session.windows:
@@ -96,7 +99,7 @@ def test_findWhere_multiple_attrs(self):
                     self.assertIsInstance(find_where, Pane)
 
     def test_where(self):
-        """.where"""
+        """Test self.where() returns matching objects."""
 
         window = self.session.attached_window()
         window.split_window()  # create second pane
@@ -145,7 +148,7 @@ def test_where(self):
                     self.assertIsInstance(where[0], Pane)
 
     def test_getById(self):
-        """.getById"""
+        """Test self.getById() retrieves child object."""
 
         window = self.session.attached_window()
 

tmuxp/testsuite/test_window.py

@@ -93,9 +93,6 @@ def test_zfresh_window_data(self):
 class NewTest2(TmuxTestCase):
 
     def test_newest_pane_data(self):
-        # self.session.select_window(1)
-        #
-        #
         window = self.session.new_window(window_name='test', attach=True)
         self.assertIsInstance(window, Window)
         self.assertEqual(1, len(window.panes))
@@ -111,7 +108,7 @@ def test_newest_pane_data(self):
 class NewTest3(TmuxTestCase):
 
     def test_attached_pane(self):
-        """Window.attached_window() returns active Pane"""
+        """Window.attached_window() returns active Pane."""
 
         window = self.session.attached_window()  # current window
         self.assertIsInstance(window.attached_pane(), Pane)
@@ -134,7 +131,7 @@ class RenameTest(TmuxTestCase):
     window_name_after = 'ha ha ha fjewlkjflwef'
 
     def test_window_rename(self):
-        """Window.rename_window.rename_window()"""
+        """Window.rename_window()."""
         self.session.set_option('automatic-rename', 'off')
         window = self.session.new_window(
             window_name=self.window_name_before, attach=True)
@@ -183,8 +180,7 @@ def test_show_window_options(self):
         self.assertIsInstance(options, dict)
 
     def test_set_show_window_options(self):
-        """Set option then Window.show_window_options(key)
-        """
+        """Set option then Window.show_window_options(key)."""
         window = self.session.new_window(window_name='test_window')
 
         window.set_window_option('main-pane-height', 20)
@@ -196,8 +192,7 @@ def test_set_show_window_options(self):
         self.assertEqual(40, window.show_window_options()['main-pane-height'])
 
     def test_show_window_option(self):
-        """Set option then Window.show_window_option(key)
-        """
+        """Set option then Window.show_window_option(key)."""
         window = self.session.new_window(window_name='test_window')
 
         window.set_window_option('main-pane-height', 20)
@@ -209,7 +204,7 @@ def test_show_window_option(self):
         self.assertEqual(40, window.show_window_option('main-pane-height'))
 
     def test_set_window_option_bad(self):
-        """Window.set_window_option raises ValueError for bad option key"""
+        """Window.set_window_option raises ValueError for bad option key."""
 
         window = self.session.new_window(window_name='test_window')