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: change timeouts in unstable selenium tests #8314

Merged
merged 4 commits into from
Jan 16, 2018

Conversation

SkorikSergey
Copy link
Contributor

What does this PR do?

This PR:

  1. increases timeout for waiting that the Terminal tab is visible
    (WorkspaceDetailsComposeTest failed test - https://ci.codenvycorp.com/view/qa/job/che-integration-tests-multiuser-master-docker/87/Selenium_tests_report/);

  2. increases timeout for waiting a message in the Toast Loader that a test workspace is started or stopped
    (ProjectStateAfterWorkspaceRestartTest failed test - https://ci.codenvycorp.com/job/che-pullrequests-test/340/Selenium_tests_report/);

  3. increases timeout for waiting that a workspace name in workspace detail toolbar changed after workspace renaming();

  4. adds timeout before clicking on the Save button in the Edit Machine form
    (WorkspaceDetailsComposeTest failed test - https://ci.codenvycorp.com/job/che-pullrequests-test/340/Selenium_tests_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. team/production labels Jan 16, 2018
@SkorikSergey SkorikSergey merged commit 548563f into master Jan 16, 2018
@SkorikSergey SkorikSergey deleted the changeTimeoutsInUnstableSeleniumTests branch January 16, 2018 14:34
@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 Jan 16, 2018
@benoitf benoitf added this to the 6.0.0-M5 milestone Jan 16, 2018
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants