-
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
CHE-6024: Add selenium test for git revert action #6550
Conversation
new WebDriverWait(seleniumWebDriver, REDRAW_UI_ELEMENTS_TIMEOUT_SEC) | ||
.until( | ||
ExpectedConditions.visibilityOfElementLocated( | ||
By.xpath("//*[contains(text(),'" + revision + "')]"))) |
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 looks strange to search on revision number in the whole page. We can potentially find the revision number in the wrong place. IMHO, it's better to restrict search by Git revert commit panel.
@@ -98,16 +97,10 @@ public void clickOnGitStatusBarTab() { | |||
} | |||
|
|||
/** wait expected message into the IDE git status bar */ |
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.
Let's update method docs as well.
@@ -287,6 +287,7 @@ | |||
<exclude name="pullConflictsTest"/> | |||
</methods> | |||
</class> | |||
<class name="org.eclipse.che.selenium.git.RevertCommitTest"/> |
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.
Please, don't forget to add test to the CodenvyOnpremSuite.xml and CodenvySaasSuite.xml as well.
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/3959/ |
What does this PR do?
Adds selenium test for git revert action
What issues does this PR fix or reference?
#6024
not required
Release Notes
not required
Docs PR
not required