PEP8 Styling Fixes

autostack/features/query_features.py

@@ -11,6 +11,7 @@
     print_accepted_post
 )
 
+
 def custom_query():
     '''
     Searches Stack Overflow using a custom search query from the user.
@@ -28,8 +29,10 @@ def custom_query():
         # Display Stack Overflow posts for the error.
         print_accepted_post(post)
 
-        # If the user's question has been answered,
-        # don't keep looping over posts.
+        '''
+        If the user's question has been answered,
+        don't keep looping over posts.
+        '''
         while True:
             print('Did this answer your question? (Y/n): ', end='')
             question_answered = input()

autostack/main.py

@@ -87,16 +87,19 @@ def main():
 
         # Variable to count number of "no"s.
         no_counter = 0
-
-        # If the current line of output is a python error,
-        # query Stack Overflow.
+        '''
+        If the current line of output is a python error,
+        query Stack Overflow.
+        '''
         if output.split()[0][:-1] in EXCEPTIONS:
             for post in accepted_posts(output):
                 # Display Stack Overflow posts for the error.
                 print_accepted_post(post)
 
-                # If the user's question has been answered,
-                # don't keep looping over posts.
+                '''
+                If the user's question has been answered,
+                don't keep looping over posts.
+                '''
                 while True:
                     print('Did this answer your question? (Y/n): ', end='')
                     question_answered = input()
@@ -107,10 +110,16 @@ def main():
                     break
                 elif question_answered == 'n':
                     no_counter += 1
-                    # If three "no"s occur in a row let the user enter a custom query.
+                    '''
+                    If three "no"s occur in a row
+                    let the user enter a custom query.
+                    '''
                     if(no_counter == 3):
                         no_counter = 0
                         custom_query()
-                        # After user finds an answer from their query, break loop to listen for more errors.
+                        '''
+                        After user finds an answer from their query,
+                        break loop to listen for more errors.
+                        '''
                         break
                     continue

autostack/tests/test_main.py

@@ -25,14 +25,14 @@
 
 def test_main_pipe_dir_doesnt_exist():
     '''
-    Test to ensure that the pipe is created even if the 
+    Test to ensure that the pipe is created even if the
     pipe dir doesn't exist.
     '''
 
     # 1. Given.
     try:
         os.system('rm -rf /tmp/')
-    except: 
+    except:
         assert True
 
     # 2. When.
@@ -62,7 +62,7 @@ def test_main_pipe_doesnt_exist():
 
 def test_main_input_y():
     '''
-    Test to ensure that, when an error is detected, inputting 
+    Test to ensure that, when an error is detected, inputting
     'Y' exits the post query loop.
     '''
 
@@ -77,7 +77,7 @@ def test_main_input_y():
 
 def test_main_input_n():
     '''
-    Test to ensure that, when an error is detected, inputting 
+    Test to ensure that, when an error is detected, inputting
     'n' continues the post query loop.
     '''
 
@@ -87,4 +87,4 @@ def test_main_input_n():
 
     # 3. Then.
 
-    pass
+    pass

autostack/web_scraper/stack_overflow_scraper.py

@@ -42,7 +42,10 @@ def accepted_posts(query):
         # The 'soup' of the query page.
         request = requests.get(query_url)
 
-        # Erros ordered from specific to general so the specific ones don't get masked by the general ones.
+        '''
+        Errors ordered from specific to general so the
+        specific ones don't get masked by the general ones.
+        '''
         try:
             # Raise exception or error if it exists.
             request.raise_for_status()
@@ -58,11 +61,6 @@ def accepted_posts(query):
         except requests.exceptions.SSLError as errs:
             print ('SSL Error:',errs)
             return None
-        # except requests.exceptions.Timeout as errt:
-        #     Catching this error catches both Connection and Read Timeout
-        #     # Maybe set up for a retry, or continue in a retry loop
-        #     print ('Timeout Error:',errt)
-        #     return None
         except requests.exceptions.ConnectTimeout as errct:
             print ('Connection Timeout:',errct)
             return None