Skip to content

Commit dd08898

Browse files
committed
Improve reliability for the html_inspector
1 parent d85e288 commit dd08898

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

seleniumbase/fixtures/base_case.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2301,6 +2301,8 @@ def assert_no_js_errors(self):
23012301
self.__highlight_with_assert_success(messenger_post, "html")
23022302

23032303
def __activate_html_inspector(self):
2304+
self.wait_for_ready_state_complete()
2305+
time.sleep(0.05)
23042306
js_utils.activate_html_inspector(self.driver)
23052307

23062308
def inspect_html(self):

seleniumbase/fixtures/js_utils.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -385,11 +385,11 @@ def activate_jquery_confirm(driver):
385385

386386
if not is_jquery_activated(driver):
387387
add_js_link(driver, jquery_js)
388-
wait_for_jquery_active(driver, timeout=0.6)
388+
wait_for_jquery_active(driver, timeout=0.9)
389389
add_css_link(driver, jq_confirm_css)
390390
add_js_link(driver, jq_confirm_js)
391391

392-
for x in range(7):
392+
for x in range(15):
393393
# jQuery-Confirm needs a small amount of time to load & activate.
394394
try:
395395
driver.execute_script("jconfirm")
@@ -408,10 +408,10 @@ def activate_html_inspector(driver):
408408
return
409409
if not is_jquery_activated(driver):
410410
add_js_link(driver, jquery_js)
411-
wait_for_jquery_active(driver, timeout=0.6)
411+
wait_for_jquery_active(driver, timeout=0.9)
412412
add_js_link(driver, html_inspector_js)
413413

414-
for x in range(7):
414+
for x in range(15):
415415
# HTML-Inspector needs a small amount of time to load & activate.
416416
try:
417417
driver.execute_script("HTMLInspector")

0 commit comments

Comments
 (0)