-
Notifications
You must be signed in to change notification settings - Fork 575
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for custom duration in switchTo().frame()
#1722
Add support for custom duration in switchTo().frame()
#1722
Conversation
8c5b6f7
to
46768d4
Compare
* @param duration the timeout duration. It overrides default Config.timeout() | ||
*/ | ||
@Nonnull | ||
public WebDriver frame(int index, Duration duration) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rename parameter duration
to timeout
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
renamed, thx
* @param duration the timeout duration. It overrides default Config.timeout() | ||
*/ | ||
@Nonnull | ||
public WebDriver frame(String nameOrId, Duration duration) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rename duration
-> timeout
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
renamed, thx
|
||
@AfterEach | ||
void tearDown() { | ||
driver().close(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we don't need to close the browser after every test: it makes tests much slower.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed unnecessary closing
46768d4
to
8c22f03
Compare
Proposed changes
Add support for custom duration in
switchTo().frame()
Checklist
gradlew check chrome_headless firefox_headless
command