fix: incorrect logic for counting invalid checks

This commit is contained in:
Adam Miller 2025-01-23 11:04:20 -08:00
parent 6ef4f0a59c
commit 61d64a1166

View File

@ -793,7 +793,13 @@ class Browser:
while True:
elapsed = time.time() - start
if elapsed > timeout:
logging.info("behavior reached hard timeout after %.1fs", elapsed)
logging.info(
"behavior reached hard timeout after %.1fs and %s valid checks, "
"and %s invalid checks",
elapsed,
valid_behavior_checks,
invalid_behavior_checks,
)
return
brozzler.sleep(check_interval)
@ -818,6 +824,7 @@ class Browser:
# valid behavior response while still running
# {'id': 8, 'result': {'result': {'type': 'boolean', 'value': False}}}
valid_behavior_checks += 1
continue
if (
msg
@ -844,7 +851,7 @@ class Browser:
invalid_behavior_checks += 1
except BrowsingTimeout:
pass
invalid_behavior_checks += 1
def try_login(self, username, password, timeout=300):
try_login_js = (