You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
def _handle_alert has hard coded 5 seconds of wait, but must use selenium timeout value #917
There is hard corded 5 seconds waiting time in keywords for browser alerts.
For example keyword Confirm action, if there is no alert will take 5 second until will be failed. The same for Alert should be present, etc. The main problem: it slows down tests.
What I suggest:
this keywords must use Selenium Timeout which is defined within importing library or by Set Selenium Timeout keyword.
Activity
aaltat commentedon Sep 25, 2017
This is true and good suggestion. Would you like to make PR to fix it?
pekkaklarck commentedon Oct 3, 2017
Closing this in favor of #942.