Skip to content

Commit abbffb6

Browse files
committed
Add comments to base_case methods
1 parent 4f47c4c commit abbffb6

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

seleniumbase/fixtures/base_case.py

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -794,39 +794,45 @@ def jquery_click(self, selector, by=By.CSS_SELECTOR):
794794
self._demo_mode_pause_if_active()
795795

796796
def hide_element(self, selector, by=By.CSS_SELECTOR):
797+
""" Hide the first element on the page that matches the selector. """
797798
selector, by = self._recalculate_selector(selector, by)
798799
selector = self.convert_to_css_selector(selector, by=by)
799800
selector = self._make_css_match_first_element_only(selector)
800801
hide_script = """jQuery('%s').hide()""" % selector
801802
self.safe_execute_script(hide_script)
802803

803804
def hide_elements(self, selector, by=By.CSS_SELECTOR):
805+
""" Hide all elements on the page that match the selector. """
804806
selector, by = self._recalculate_selector(selector, by)
805807
selector = self.convert_to_css_selector(selector, by=by)
806808
hide_script = """jQuery('%s').hide()""" % selector
807809
self.safe_execute_script(hide_script)
808810

809811
def show_element(self, selector, by=By.CSS_SELECTOR):
812+
""" Show the first element on the page that matches the selector. """
810813
selector, by = self._recalculate_selector(selector, by)
811814
selector = self.convert_to_css_selector(selector, by=by)
812815
selector = self._make_css_match_first_element_only(selector)
813816
show_script = """jQuery('%s').show(0)""" % selector
814817
self.safe_execute_script(show_script)
815818

816819
def show_elements(self, selector, by=By.CSS_SELECTOR):
820+
""" Show all elements on the page that match the selector. """
817821
selector, by = self._recalculate_selector(selector, by)
818822
selector = self.convert_to_css_selector(selector, by=by)
819823
show_script = """jQuery('%s').show(0)""" % selector
820824
self.safe_execute_script(show_script)
821825

822826
def remove_element(self, selector, by=By.CSS_SELECTOR):
827+
""" Remove the first element on the page that matches the selector. """
823828
selector, by = self._recalculate_selector(selector, by)
824829
selector = self.convert_to_css_selector(selector, by=by)
825830
selector = self._make_css_match_first_element_only(selector)
826831
remove_script = """jQuery('%s').remove()""" % selector
827832
self.safe_execute_script(remove_script)
828833

829834
def remove_elements(self, selector, by=By.CSS_SELECTOR):
835+
""" Remove all elements on the page that match the selector. """
830836
selector, by = self._recalculate_selector(selector, by)
831837
selector = self.convert_to_css_selector(selector, by=by)
832838
remove_script = """jQuery('%s').remove()""" % selector
@@ -839,6 +845,9 @@ def get_domain_url(self, url):
839845
return page_utils.get_domain_url(url)
840846

841847
def safe_execute_script(self, script):
848+
""" When executing a script that contains a jQuery command,
849+
it's important that the jQuery library has been loaded first.
850+
This method will load jQuery if it wasn't already loaded. """
842851
try:
843852
self.execute_script(script)
844853
except Exception:

0 commit comments

Comments
 (0)