Skip to content

Commit

Permalink
Reduced the timeout on the expected timeout within expected conditions
Browse files Browse the repository at this point in the history
The delay for the title change was 600 miliseconds while the timeout was
0.5 seconds. I suspect this gives little time for error. The 600
milisecond timeout was specifically choosen along with the other
timeouts in the expected conditions tests such that any combination of
delays would be unique. This gives us a framework to test various
combinations. So I didn't want to change that. Noting I could simply
swap the 600 millisecond test for the 900 millisecond tes .. which might
be the best bet. But for now trying to reduce the timeout to 0.3
seconds.
  • Loading branch information
emanlove committed Apr 3, 2024
1 parent fd751bb commit 204ceda
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions atest/acceptance/keywords/expected_conditions.robot
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,10 @@ Wait For Expected Conditions One Argument
Title Should Be Delayed
Wait For Expected Condition Times out within set timeout
[Documentation] FAIL REGEXP: TimeoutException: Message: Expected Condition not met within set timeout of 0.5*
[Documentation] FAIL REGEXP: TimeoutException: Message: Expected Condition not met within set timeout of 0.3*
Title Should Be Original
Click Element link=delayed change title
Wait For Expected Condition title_is Delayed timeout=0.5
Wait For Expected Condition title_is Delayed timeout=0.3
Wait For Expected Conditions using WebElement as locator
Click Button Change the button state
Expand Down

0 comments on commit 204ceda

Please sign in to comment.