Flake8 refactoring

Author: mdmintz

seleniumbase/common/encryption.py

@@ -45,8 +45,8 @@ def reverse_shuffle_string(string):
         return string
     new_string = ""
     odd = (len(string) % 2 == 1)
-    part1 = string[:int(len(string)/2):1]
-    part2 = string[int(len(string)/2)::1]
+    part1 = string[:int(len(string) / 2):1]
+    part2 = string[int(len(string) / 2)::1]
     for c in range(len(part1)):
         new_string += part2[c]
         new_string += part1[c]
@@ -118,18 +118,18 @@ def decrypt(string):
             rem4 = (len(string) + ord_string_sum(string)) % 2
         if len(string) % 2 != 0:
             if rem3 == 1:
-                string = (chr(ord(string[-1])-5-rem1) + string +
-                          chr(ord(string[-1])-13-rem1))
+                string = (chr(ord(string[-1]) - 5 - rem1) + string +
+                          chr(ord(string[-1]) - 13 - rem1))
             else:
-                string = (chr(ord(string[-1])-11-rem1) + string +
-                          chr(ord(string[-1])-23-rem1))
+                string = (chr(ord(string[-1]) - 11 - rem1) + string +
+                          chr(ord(string[-1]) - 23 - rem1))
         elif len(string) > 1:
             if rem4 == 1:
-                string = (chr(ord(string[0])-19+rem2) + string +
-                          chr(ord(string[0])-7-rem2))
+                string = (chr(ord(string[0]) - 19 + rem2) + string +
+                          chr(ord(string[0]) - 7 - rem2))
             else:
-                string = (chr(ord(string[0])-26+rem2) + string +
-                          chr(ord(string[0])-12-rem2))
+                string = (chr(ord(string[0]) - 26 + rem2) + string +
+                          chr(ord(string[0]) - 12 - rem2))
         rem5 = (len(string) + ord_string_sum(string)) % 23
         string = rotate(string, rem5)
         result = str_xor(shuffle_string(string)[::-1], xor_key)

seleniumbase/console_scripts/sb_install.py

@@ -62,7 +62,7 @@ def main():
             invalid_run_command()
     else:
         invalid_run_command()
-    name = sys.argv[num_args-1]
+    name = sys.argv[num_args - 1]
 
     file_name = None
     download_url = None

seleniumbase/console_scripts/sb_mkdir.py

@@ -40,7 +40,7 @@ def main():
             invalid_run_command()
     else:
         invalid_run_command()
-    dir_name = sys.argv[num_args-1]
+    dir_name = sys.argv[num_args - 1]
     if len(str(dir_name)) < 2:
         raise Exception('Directory name length must be at least 2 '
                         'characters long!')

seleniumbase/core/style_sheet.py

@@ -2,8 +2,8 @@
         <title>Test Report</title>
         <link rel="SHORTCUT ICON"
         href="%s" /> ''' % (
-            "https://raw.githubusercontent.com/seleniumbase/SeleniumBase"
-            "/master/seleniumbase/resources/favicon.ico")
+        "https://raw.githubusercontent.com/seleniumbase/SeleniumBase"
+        "/master/seleniumbase/resources/favicon.ico")
 
 style = title + '''<style type="text/css">
         html {

seleniumbase/core/testcase_manager.py

@@ -84,7 +84,7 @@ def get_params(self):
             "total_execution_time": self.total_execution_time,
             "username": self.username,
             "guid": self.guid
-            }
+        }
 
 
 class TestcaseDataPayload:

seleniumbase/fixtures/xpath_to_css.py

@@ -70,7 +70,7 @@ def _filter_xpath_grouping(xpath):
     index = xpath.rfind(')')
     if index == -1:
         raise XpathException("Invalid or unsupported Xpath: %s" % xpath)
-    xpath = xpath[:index] + xpath[index+1:]
+    xpath = xpath[:index] + xpath[index + 1:]
     return xpath
 
 

seleniumbase/plugins/pytest_plugin.py

@@ -23,12 +23,13 @@ def pytest_addoption(parser):
     parser.addoption('--env', action='store',
                      dest='environment',
                      choices=(
-                        constants.Environment.QA,
-                        constants.Environment.STAGING,
-                        constants.Environment.PRODUCTION,
-                        constants.Environment.MASTER,
-                        constants.Environment.LOCAL,
-                        constants.Environment.TEST),
+                         constants.Environment.QA,
+                         constants.Environment.STAGING,
+                         constants.Environment.PRODUCTION,
+                         constants.Environment.MASTER,
+                         constants.Environment.LOCAL,
+                         constants.Environment.TEST
+                     ),
                      default=constants.Environment.TEST,
                      help="The environment to run the tests in.")
     parser.addoption('--data', dest='data',
@@ -48,7 +49,8 @@ def pytest_addoption(parser):
     parser.addoption('--database_env', action='store',
                      dest='database_env',
                      choices=(
-                        'prod', 'qa', 'staging', 'test', 'local', 'master'),
+                         'prod', 'qa', 'staging', 'test', 'local', 'master'
+                     ),
                      default='test',
                      help=optparse.SUPPRESS_HELP)
     parser.addoption('--with-s3_logging', action="store_true",