Skip to content

Commit 495da7e

Browse files
committed
Refactor timing
1 parent 4bfd8c9 commit 495da7e

File tree

2 files changed

+18
-12
lines changed

2 files changed

+18
-12
lines changed

seleniumbase/fixtures/base_case.py

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -395,7 +395,7 @@ def click(
395395
self.__shadow_click(selector, timeout)
396396
return
397397
if self.__needs_minimum_wait() or self.browser == "safari":
398-
time.sleep(0.03)
398+
time.sleep(0.04)
399399
element = page_actions.wait_for_element_visible(
400400
self.driver,
401401
selector,
@@ -632,13 +632,13 @@ def click(
632632
except Exception:
633633
pass
634634
if self.__needs_minimum_wait() or self.browser == "safari":
635-
time.sleep(0.03)
635+
time.sleep(0.04)
636636
try:
637637
if self.driver.current_url != pre_action_url:
638638
self.__ad_block_as_needed()
639639
self.__disable_beforeunload_as_needed()
640640
if self.__needs_minimum_wait():
641-
time.sleep(0.03)
641+
time.sleep(0.04)
642642
except Exception:
643643
try:
644644
self.wait_for_ready_state_complete()
@@ -951,6 +951,10 @@ def update_text(
951951
raise
952952
if settings.WAIT_FOR_RSC_ON_PAGE_LOADS:
953953
self.wait_for_ready_state_complete()
954+
if self.__needs_minimum_wait():
955+
time.sleep(0.03)
956+
if self.undetectable:
957+
time.sleep(0.025)
954958
except Exception:
955959
self.wait_for_ready_state_complete()
956960
time.sleep(0.14)
@@ -975,9 +979,9 @@ def update_text(
975979
if settings.WAIT_FOR_RSC_ON_PAGE_LOADS:
976980
self.wait_for_ready_state_complete()
977981
if self.__needs_minimum_wait():
978-
time.sleep(0.01)
982+
time.sleep(0.03)
979983
if self.undetectable:
980-
time.sleep(0.015)
984+
time.sleep(0.025)
981985
if (
982986
retry
983987
and element.get_attribute("value") != text
@@ -6735,11 +6739,13 @@ def choose_file(
67356739
timeout = self.__get_new_timeout(timeout)
67366740
selector, by = self.__recalculate_selector(selector, by)
67376741
abs_path = os.path.abspath(file_path)
6742+
if self.__needs_minimum_wait():
6743+
time.sleep(0.02)
67386744
element = self.wait_for_element_present(
67396745
selector, by=by, timeout=timeout
67406746
)
67416747
if self.__needs_minimum_wait():
6742-
time.sleep(0.08) # Force a minimum wait, even if skipping waits.
6748+
time.sleep(0.08)
67436749
if self.is_element_visible(selector, by=by):
67446750
self.__demo_mode_highlight_if_active(selector, by)
67456751
if not self.demo_mode and not self.slow_mode:
@@ -12951,7 +12957,7 @@ def __click_with_offset(
1295112957

1295212958
self.wait_for_ready_state_complete()
1295312959
if self.__needs_minimum_wait():
12954-
time.sleep(0.14) # Force a minimum wait, even if skipping waits.
12960+
time.sleep(0.14)
1295512961
if not timeout:
1295612962
timeout = settings.SMALL_TIMEOUT
1295712963
if self.timeout_multiplier and timeout == settings.SMALL_TIMEOUT:

seleniumbase/fixtures/js_utils.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ def is_jquery_activated(driver):
111111

112112
def wait_for_jquery_active(driver, timeout=None):
113113
if not timeout:
114-
timeout = 22
114+
timeout = 2
115115
else:
116116
timeout = int(timeout * 10.0)
117117
for x in range(timeout):
@@ -778,7 +778,7 @@ def activate_messenger(driver):
778778

779779
if not is_jquery_activated(driver):
780780
add_js_link(driver, jquery_js)
781-
wait_for_jquery_active(driver, timeout=1)
781+
wait_for_jquery_active(driver, timeout=1.1)
782782
add_css_link(driver, messenger_css)
783783
add_css_link(driver, msgr_theme_flat_css)
784784
add_css_link(driver, msgr_theme_future_css)
@@ -894,11 +894,11 @@ def post_message(driver, message, msg_dur=None, style="info"):
894894
try:
895895
driver.execute_script(messenger_script)
896896
except Exception:
897-
time.sleep(0.2)
897+
time.sleep(0.17)
898898
activate_messenger(driver)
899-
time.sleep(0.2)
899+
time.sleep(0.17)
900900
set_messenger_theme(driver)
901-
time.sleep(0.3)
901+
time.sleep(0.27)
902902
driver.execute_script(messenger_script)
903903

904904

0 commit comments

Comments
 (0)