Flake8 fixes

Author: aaltat

src/SeleniumLibrary/base/context.py

@@ -95,8 +95,7 @@ def is_text_present(self, text):
 
     def is_element_enabled(self, locator, tag=None):
         element = self.find_element(locator, tag)
-        return (element.is_enabled() and
-                element.get_attribute('readonly') is None)
+        return element.is_enabled() and element.get_attribute('readonly') is None
 
     def is_visible(self, locator):
         element = self.find_element(locator, required=False)

src/SeleniumLibrary/base/robotlibcore.py

@@ -28,6 +28,7 @@
     def keyword(name=None, tags=()):
         if callable(name):
             return keyword()(name)
+
         def decorator(func):
             func.robot_name = name
             func.robot_tags = tags

src/SeleniumLibrary/keywords/alert.py

@@ -19,7 +19,7 @@
 from selenium.webdriver.support.ui import WebDriverWait
 
 from SeleniumLibrary.base import keyword, LibraryComponent
-from SeleniumLibrary.utils import is_truthy, secs_to_timestr
+from SeleniumLibrary.utils import secs_to_timestr
 
 
 class AlertKeywords(LibraryComponent):
@@ -134,8 +134,6 @@ def _wait_alert(self, timeout=None):
         try:
             return wait.until(EC.alert_is_present())
         except TimeoutException:
-            raise AssertionError('Alert not found in %s.'
-                             % secs_to_timestr(timeout))
+            raise AssertionError('Alert not found in %s.' % secs_to_timestr(timeout))
         except WebDriverException as err:
-            raise AssertionError('An exception occurred waiting for alert: %s'
-                                 % err)
+            raise AssertionError('An exception occurred waiting for alert: %s' % err)

src/SeleniumLibrary/keywords/browsermanagement.py

@@ -89,7 +89,7 @@ def open_browser(self, url=None, browser='firefox', alias=None,
         Headless Chrome are new additions in SeleniumLibrary 3.1.0
         and require Selenium 3.8.0 or newer.
 
-        After opening the browser, it is possible to use optional 
+        After opening the browser, it is possible to use optional
         ``url`` to navigate the browser to the desired address.
 
         Optional ``alias`` is an alias given for this browser instance and
@@ -205,7 +205,7 @@ def open_browser(self, url=None, browser='firefox', alias=None,
         ``options``.
 
         Example the ``options`` argument can be used to launch Chomium-based
-        applications which utilize the 
+        applications which utilize the
         [https://bitbucket.org/chromiumembedded/cef/wiki/UsingChromeDriver|Chromium Embedded Framework]
         . To lauch Chomium-based application, use ``options`` to define
         `binary_location` attribute and use `add_argument` method to define
@@ -270,7 +270,7 @@ def open_browser(self, url=None, browser='firefox', alias=None,
         accepting an instance of the `selenium.webdriver.FirefoxProfile`
         and support defining FirefoxProfile with methods and
         attributes are new in SeleniumLibrary 4.0.
-        
+
         Making ``url`` optional is new in SeleniumLibrary 4.1.
 
         The ``executable_path`` argument is new in SeleniumLibrary 4.2.
@@ -304,8 +304,7 @@ def _make_new_browser(self, url=None, browser='firefox', alias=None,
             try:
                 driver.get(url)
             except Exception:
-                self.debug("Opened browser with session id %s but failed "
-                        "to open url '%s'." % (driver.session_id, url))
+                self.debug("Opened browser with session id %s but failed to open url '%s'." % (driver.session_id, url))
                 raise
         self.debug('Opened browser with session id %s.' % driver.session_id)
         return index

src/SeleniumLibrary/keywords/cookie.py

@@ -43,11 +43,11 @@ def delete_cookie(self, name):
     def get_cookies(self, as_dict=False):
         """Returns all cookies of the current page.
 
-        If ``as_dict`` argument evaluates as false, see `Boolean arguments` 
-        for more details, then cookie information is returned as 
+        If ``as_dict`` argument evaluates as false, see `Boolean arguments`
+        for more details, then cookie information is returned as
         a single string in format ``name1=value1; name2=value2; name3=value3``.
         When ``as_dict`` argument evaluates as true, cookie information
-        is returned as Robot Framework dictionary format. The string format 
+        is returned as Robot Framework dictionary format. The string format
         can be used, for example, for logging purposes or in headers when
         sending HTTP requests. The dictionary format is helpful when
         the result can be passed to requests library's Create Session

src/SeleniumLibrary/keywords/formelement.py

@@ -203,7 +203,7 @@ def choose_file(self, locator, file_path):
         points at a file and when using Selenium Grid, Selenium will
         [https://seleniumhq.github.io/selenium/docs/api/py/webdriver_remote/selenium.webdriver.remote.command.html?highlight=upload#selenium.webdriver.remote.command.Command.UPLOAD_FILE|magically],
         transfer the file from the machine where the tests are executed
-        to the Selenium Grid node where the browser is running. 
+        to the Selenium Grid node where the browser is running.
         Then Selenium will send the file path, from the nodes file
         system, to the browser.
 

src/SeleniumLibrary/keywords/selectelement.py

@@ -14,7 +14,6 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-from selenium.common.exceptions import NoSuchElementException
 from selenium.webdriver.support.ui import Select
 
 from SeleniumLibrary.base import LibraryComponent, keyword

src/SeleniumLibrary/keywords/waiting.py

@@ -94,7 +94,6 @@ def wait_until_location_contains(self, expected, timeout=None, message=None):
                          "Location did not contain '%s' in <TIMEOUT>." % expected,
                          timeout, message)
 
-
     @keyword
     def wait_until_page_contains(self, text, timeout=None, error=None):
         """Waits until ``text`` appears on the current page.

src/SeleniumLibrary/keywords/webdrivertools/webdrivertools.py

@@ -71,8 +71,7 @@ def create_driver(self, browser, desired_capabilities, remote_url, profile_dir=N
         if service_log_path:
             logger.info('Browser driver log file created to: %s' % service_log_path)
             self._create_directory(service_log_path)
-        if (creation_method == self.create_firefox
-                or creation_method == self.create_headless_firefox):
+        if creation_method == self.create_firefox or creation_method == self.create_headless_firefox:
             return creation_method(desired_capabilities, remote_url, profile_dir, options=options,
                                    service_log_path=service_log_path, executable_path=executable_path)
         return creation_method(desired_capabilities, remote_url, options=options,

src/SeleniumLibrary/locators/elementfinder.py

@@ -240,7 +240,7 @@ def _parse_locator(self, locator):
         if index != -1:
             prefix = locator[:index].strip()
             if prefix in self._strategies:
-                return prefix, locator[index+1:].lstrip()
+                return prefix, locator[index + 1:].lstrip()
         return 'default', locator
 
     def _get_locator_separator_index(self, locator):