Skip to content
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

Selenium: remove info about known 10177 and 10699 issues #11734

Merged
merged 9 commits into from
Oct 26, 2018

Conversation

SkorikSergey
Copy link
Contributor

What does this PR do?

This PR:

What issues does this PR fix or reference?

#11732

@SkorikSergey SkorikSergey added status/in-progress This issue has been taken by an engineer and is under active development. kind/task Internal things, technical debt, and to-do tasks to be performed. team/che-qe labels Oct 25, 2018
@SkorikSergey
Copy link
Contributor Author

ci-test

@riuvshin
Copy link
Contributor

Results of automated E2E tests of Eclipse Che Multiuser on OCP:
Build details
Test report
docker image: eclipseche/che-server:11734
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

@SkorikSergey SkorikSergey added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. and removed status/in-progress This issue has been taken by an engineer and is under active development. labels Oct 26, 2018
(ExpectedCondition<Boolean>)
webDriver -> getAllVisibleTextFromSignaturesContainer().contains(expectedProposal));
}
///
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks redundant here

* @param expectedProposal text which should be present in the container
*/
public void waitProposalIntoSignaturesContainer(final String expectedProposal) {
webDriverWaitFactory
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's use seleniumWebDriverHelper here

@SkorikSergey
Copy link
Contributor Author

ci-test

@riuvshin
Copy link
Contributor

Results of automated E2E tests of Eclipse Che Multiuser on OCP:
Build details
Test report
docker image: eclipseche/che-server:11734
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

@SkorikSergey SkorikSergey merged commit 61e1491 into master Oct 26, 2018
@SkorikSergey SkorikSergey deleted the updateSeleniumTestsFromLanguageserversPackage branch October 26, 2018 11:27
@benoitf benoitf added this to the 6.14.0 milestone Oct 26, 2018
@benoitf benoitf removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Oct 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/task Internal things, technical debt, and to-do tasks to be performed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants