Refactoring

Author: mdmintz

seleniumbase/fixtures/base_case.py

@@ -1228,26 +1228,22 @@ def click_link_text(self, link_text, timeout=None):
             if text_id:
                 link_css = '[id="%s"]' % link_text
                 found_css = True
-
             if not found_css:
                 href = self.__get_href_from_link_text(link_text, False)
                 if href:
                     if href.startswith("/") or page_utils.is_valid_url(href):
                         link_css = '[href="%s"]' % href
                         found_css = True
-
             if not found_css:
                 ngclick = self.get_link_attribute(link_text, "ng-click", False)
                 if ngclick:
                     link_css = '[ng-click="%s"]' % ngclick
                     found_css = True
-
             if not found_css:
                 onclick = self.get_link_attribute(link_text, "onclick", False)
                 if onclick:
                     link_css = '[onclick="%s"]' % onclick
                     found_css = True
-
             success = False
             if found_css:
                 if self.is_element_visible(link_css):
@@ -1258,13 +1254,11 @@ def click_link_text(self, link_text, timeout=None):
                     success = self.__click_dropdown_link_text(
                         link_text, link_css
                     )
-
             if not success:
                 element = self.wait_for_link_text_visible(
                     link_text, timeout=settings.MINI_TIMEOUT
                 )
                 element.click()
-
         latest_window_count = len(self.driver.window_handles)
         if (
             latest_window_count > pre_window_count
@@ -1369,7 +1363,6 @@ def click_partial_link_text(self, partial_link_text, timeout=None):
             if text_id:
                 link_css = '[id="%s"]' % partial_link_text
                 found_css = True
-
             if not found_css:
                 href = self.__get_href_from_partial_link_text(
                     partial_link_text, False
@@ -1378,23 +1371,20 @@ def click_partial_link_text(self, partial_link_text, timeout=None):
                     if href.startswith("/") or page_utils.is_valid_url(href):
                         link_css = '[href="%s"]' % href
                         found_css = True
-
             if not found_css:
                 ngclick = self.get_partial_link_text_attribute(
                     partial_link_text, "ng-click", False
                 )
                 if ngclick:
                     link_css = '[ng-click="%s"]' % ngclick
                     found_css = True
-
             if not found_css:
                 onclick = self.get_partial_link_text_attribute(
                     partial_link_text, "onclick", False
                 )
                 if onclick:
                     link_css = '[onclick="%s"]' % onclick
                     found_css = True
-
             success = False
             if found_css:
                 if self.is_element_visible(link_css):
@@ -1405,13 +1395,11 @@ def click_partial_link_text(self, partial_link_text, timeout=None):
                     success = self.__click_dropdown_partial_link_text(
                         partial_link_text, link_css
                     )
-
             if not success:
                 element = self.wait_for_partial_link_text(
                     partial_link_text, timeout=settings.MINI_TIMEOUT
                 )
                 element.click()
-
         latest_window_count = len(self.driver.window_handles)
         if (
             latest_window_count > pre_window_count

seleniumbase/fixtures/words.py

@@ -1,5 +1,7 @@
 # -*- coding: utf-8 -*-
-""" Small Dictionary """
+"""
+Small Dictionary for Demo Mode translations
+"""
 
 
 class SD:

seleniumbase/plugins/base_plugin.py

@@ -1,5 +1,7 @@
 # -*- coding: utf-8 -*-
-""" This is the Nosetest plugin for setting base configuration and logging. """
+"""
+This is the Nosetest plugin for setting base configuration and logging.
+"""
 
 import ast
 import sys

seleniumbase/plugins/pytest_plugin.py

@@ -1,5 +1,7 @@
 # -*- coding: utf-8 -*-
-""" This is the pytest configuration file for setting test options. """
+"""
+This is the pytest configuration file for setting test options.
+"""
 
 import colorama
 import os

seleniumbase/plugins/selenium_plugin.py

@@ -1,5 +1,7 @@
 # -*- coding: utf-8 -*-
-""" This is the Nosetest plugin for setting Selenium test configuration. """
+"""
+This is the Nosetest plugin for setting Selenium test configuration.
+"""
 
 import sys
 from nose.plugins import Plugin