#41 imporved docstring

Author: chavarera

s_tool/driver.py

@@ -42,11 +42,23 @@ def close(self):
             self.driver.quit()
 
     def __exit__(self, type, value, traceback):
-        """release the resources occupied with the current session"""
+        """release the resources occupied with the current session
+
+        Args:
+            type ([type]): type
+            value ([type]): value
+            traceback ([type]): and traceback
+        """
         self.close()
 
     def __enter__(self):
-        """llows you to implement objects which can be used easily with the with statement"""
+        """Returns an selenium webdriver
+        Allows you to implement objects
+        which can be used easily with the with statement
+
+        Returns:
+            [webDriver]: selenium webdriver
+        """
         return self._load_driver()
 
     def _load_driver(self):

s_tool/exceptions.py

@@ -4,8 +4,17 @@ class SToolException(Exception):
     """
 
     def __init__(self, message):
+        """[summary]
+
+        Args:
+            message ([str]): Exception message
+        """
         self.message = message
 
     def __str__(self):
-        """Returns an Exception"""
+        """Return Exception message
+
+        Returns:
+            [str]: Exception
+        """
         return str(self.message)

s_tool/parser.py

@@ -2,17 +2,19 @@
 from s_tool.utils import get_element
 
 
-def select_options(element, swap=None, text_exclude=[]):
+def select_options(element, swap=None, text_exclude=set()):
     """Return dropdown option in key value pair
 
     Args:
-        element      : An select element
-        text_exclude : list of values to exclude from result
+        element ([selenium.element]): An select element
+        swap ([bool], optional): Value as key if True. Defaults to None.
+        text_exclude (set, optional): to exclude from result. Defaults to set().
 
     Returns:
-        return dict of values and text of select element,
-        return empty dict() if element is not valid or not exists
+        [dict]: return dict of values and text of select element,
+                return empty dict() if element is not valid or not exists
     """
+
     option_dict = dict()
     if element and hasattr(element, "tag_name") and element.tag_name == "select":
         options = get_element(element, "option", "tag_name", many=True)
@@ -36,7 +38,6 @@ def get_table(table):
     Returns:
         return list of row list,
         return [] if table not valid
-
     """
     results = []
     if table and hasattr(table, "tag_name") and table.tag_name == "table":