From 87e0e6e24c24ccdfabcc67e7ac21de38a2a44dbf Mon Sep 17 00:00:00 2001
From: labrys <labrys@useres.noreply.github.com>
Date: Fri, 27 Nov 2015 19:15:26 -0500
Subject: [PATCH] Lint 10/10

---
 tests/issue_submitter_tests.py | 34 ++++++++++++++++++++++++----------
 1 file changed, 24 insertions(+), 10 deletions(-)

diff --git a/tests/issue_submitter_tests.py b/tests/issue_submitter_tests.py
index ea8a7a495..79829f5b4 100644
--- a/tests/issue_submitter_tests.py
+++ b/tests/issue_submitter_tests.py
@@ -17,29 +17,42 @@
 # You should have received a copy of the GNU General Public License
 # along with SickRage.  If not, see <http://www.gnu.org/licenses/>.
 
+"""
+Test exception logging
+"""
 
-import sys, os.path
+import sys
+import os.path
+import unittest
 
 sys.path.insert(1, os.path.abspath(os.path.join(os.path.dirname(__file__), '../lib')))
 sys.path.insert(1, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
 
-import unittest
-
 from sickbeard import logger
 from sickrage.helper.exceptions import ex
 
-def error():
+
+def exception_generator():
+    """
+    Dummy function to raise a fake exception and log it
+    """
     try:
         raise Exception('FAKE EXCEPTION')
-    except Exception as e:
-        logger.log(u"FAKE ERROR: " + ex(e), logger.ERROR)
-        logger.submit_errors()
+    except Exception as error:
+        logger.log(u"FAKE ERROR: " + ex(error), logger.ERROR)  # pylint: disable=no-member
+        logger.submit_errors()  # pylint: disable=no-member
         raise
 
 
 class IssueSubmitterBasicTests(unittest.TestCase):
+    """
+    Tests logging of exceptions
+    """
     def test_submitter(self):
-        self.assertRaises(Exception, error)
+        """
+        Test that an exception is raised
+        """
+        self.assertRaises(Exception, exception_generator)
 
 
 if __name__ == "__main__":
@@ -47,5 +60,6 @@ if __name__ == "__main__":
     print "STARTING - ISSUE SUBMITTER TESTS"
     print "=================="
     print "######################################################################"
-    suite = unittest.TestLoader().loadTestsFromTestCase(IssueSubmitterBasicTests)
-    unittest.TextTestRunner(verbosity=2).run(suite)
+
+    SUITE = unittest.TestLoader().loadTestsFromTestCase(IssueSubmitterBasicTests)
+    unittest.TextTestRunner(verbosity=2).run(SUITE)
-- 
GitLab