-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Fix bug in AskForValueDialog in typeAndWaitText method #7545
Conversation
(cherry picked from commit f2ca2fa)
@@ -137,7 +138,10 @@ public void clickCancelButtonJava() { | |||
} | |||
|
|||
public void typeAndWaitText(String text) { | |||
input.sendKeys(text); | |||
waitFormToOpen(); |
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.
It should be checked before the call of the method.
@@ -137,7 +138,10 @@ public void clickCancelButtonJava() { | |||
} | |||
|
|||
public void typeAndWaitText(String text) { | |||
input.sendKeys(text); | |||
waitFormToOpen(); | |||
new WebDriverWait(seleniumWebDriver, TestTimeoutsConstants.LOAD_PAGE_TIMEOUT_SEC) |
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.
Good catch!
Lets extract new WebDriverWait(seleniumWebDriver, TestTimeoutsConstants.LOAD_PAGE_TIMEOUT_SEC)
into the separate constant and add similar verification in other methods.
loadPageWait.until(visibilityOf(inputNameJavaClass)).sendKeys(text); | ||
} | ||
|
||
public void waitTextInFieldName(String expectedText) { |
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.
IMHO it's better to enclose call of this method into the typeTextInFieldName() in similar way, as it was done for waitInputNameContains() method.
@@ -31,11 +31,11 @@ che.selenium.infrastructure=docker | |||
# Define configuration for che default test user. | |||
# For che single user assembly, the default values will be enough. | |||
# For che multiuser assembly, user must be already created on identity provider service. | |||
che.test_user.email=che@eclipse.org | |||
che.test_user.password=secret | |||
che.test_user.email=admin@admin.com |
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.
Is it really what you wanted to change? :-)
What does this PR do?
Fix bug in AskForValueDialog in typeAndWaitText method
What issues does this PR fix or reference?
Issue #7543
Release Notes
Docs PR