PEP257

Author: tony

tmuxp/testsuite/test_config_tmuxinator.py

@@ -94,13 +94,14 @@ def test_config_to_dict(self):
 
 class TmuxinatorDeprecationsTest(TestCase):
 
-    """ tmuxinator uses `tabs` instead of `windows` in older versions
+    """Tmuxinator uses `tabs` instead of `windows` in older versions.
 
     https://github.com/aziz/tmuxinator/blob/master/lib/tmuxinator/project.rb#L18
 
     https://github.com/aziz/tmuxinator/blob/master/spec/fixtures/sample.deprecations.yml
 
     LICENSE: https://github.com/aziz/tmuxinator/blob/master/LICENSE
+
     """
 
     tmuxinator_yaml = """\
@@ -149,7 +150,10 @@ 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',
@@ -279,9 +283,12 @@ def test_config_to_dict(self):
 
 class TmuxinatoriSampleTest(TestCase):
 
-    """https://github.com/aziz/tmuxinator/blob/master/spec/fixtures/sample.yml
+    """Tests against tmuxinator's <spec/fixtures/sample.yml>.
+
+    https://github.com/aziz/tmuxinator/blob/master/spec/fixtures/sample.yml
 
     LICENSE: https://github.com/aziz/tmuxinator/blob/master/LICENSE
+
     """
 
     tmuxinator_yaml = """\
@@ -335,7 +342,10 @@ 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_server.py

@@ -68,6 +68,3 @@ def test_88_colors(self):
 
         self.assertIn('-8', proc.cmd)
         self.assertNotIn('-2', proc.cmd)
-
-if __name__ == '__main__':
-    unittest.main()

tmuxp/testsuite/test_session.py

@@ -26,16 +26,17 @@
 class SessionTest(TmuxTestCase):
 
     def test_has_session(self):
-        """Server.has_session returns True if has session_name exists"""
+        """Server.has_session returns True if has session_name exists."""
         self.assertTrue(t.has_session(self.TEST_SESSION_NAME))
         self.assertFalse(t.has_session('asdf2314324321'))
 
     def test_select_window(self):
-        """Session.select_window moves window"""
+        """Session.select_window moves window."""
         # get the current window_base_index, since different user tmux config
         # may start at 0 or 1, or whatever they want.
         window_base_index = int(
-            self.session.attached_window().get('window_index'))
+            self.session.attached_window().get('window_index')
+        )
 
         window = self.session.new_window(window_name='test_window')
         window_count = len(self.session._windows)
@@ -67,7 +68,7 @@ def test_select_window(self):
             selected_window1.__dict__, selected_window2.__dict__)
 
     def test_select_window_returns_Window(self):
-        """Session.select_window returns Window object"""
+        """Session.select_window returns Window object."""
 
         window_count = len(self.session._windows)
         self.assertEqual(len(self.session._windows), window_count)
@@ -78,15 +79,15 @@ def test_select_window_returns_Window(self):
             window_base_index), Window)
 
     def test_attached_window(self):
-        """Session.attached_window() returns Window"""
+        """Session.attached_window() returns Window."""
         self.assertIsInstance(self.session.attached_window(), Window)
 
     def test_attached_pane(self):
-        """Session.attached_pane() returns Pane"""
+        """Session.attached_pane() returns Pane."""
         self.assertIsInstance(self.session.attached_pane(), Pane)
 
     def test_session_rename(self):
-        """Session.rename_session renames session"""
+        """Session.rename_session renames session."""
         test_name = 'testingdis_sessname'
         self.session.rename_session(test_name)
         self.assertEqual(self.session.get('session_name'), test_name)
@@ -100,7 +101,7 @@ def test_session_rename(self):
 class SessionNewTest(TmuxTestCase):
 
     def test_new_session(self):
-        """Server.new_session creates new session"""
+        """Server.new_session creates new session."""
         new_session_name = TEST_SESSION_PREFIX + str(randint(0, 1337))
         new_session = t.new_session(session_name=new_session_name, detach=True)
 
@@ -117,8 +118,7 @@ def test_show_options(self):
         self.assertIsInstance(options, dict)
 
     def test_set_show_options_single(self):
-        """Set option then Session.show_options(key)
-        """
+        """Set option then Session.show_options(key)."""
 
         self.session.set_option('history-limit', 20)
         self.assertEqual(20, self.session.show_options('history-limit'))
@@ -129,8 +129,7 @@ def test_set_show_options_single(self):
         self.assertEqual(40, self.session.show_options()['history-limit'])
 
     def test_set_show_option(self):
-        """Set option then Session.show_option(key)
-        """
+        """Set option then Session.show_option(key)."""
         self.session.set_option('history-limit', 20)
         self.assertEqual(20, self.session.show_option('history-limit'))
 
@@ -139,6 +138,6 @@ def test_set_show_option(self):
         self.assertEqual(40, self.session.show_option('history-limit'))
 
     def test_set_option_bad(self):
-        """Session.set_option raises ValueError for bad option key"""
+        """Session.set_option raises ValueError for bad option key."""
         with self.assertRaises(ValueError):
             self.session.set_option('afewewfew', 43)