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

Rework tests for checking TestRunner plugin #6559

Merged
merged 9 commits into from
Oct 5, 2017
Merged

Conversation

musienko-maxim
Copy link
Contributor

@musienko-maxim musienko-maxim commented Oct 3, 2017

What does this PR do?

We've have unexpected unstable behavior tests related on TestRunner plugin. This PR add special ids for more clear using of web elements for result nodes and avoiding unstable steps in the selenium tests. Also it will allow more clarify to understand of potential problems during interaction of the tests and IDE.

What issues does this PR fix or reference?

#6497

@svor, @vparfonov, @tolusha

@musienko-maxim musienko-maxim requested review from tolusha and svor and removed request for evidolob October 3, 2017 14:57
@benoitf benoitf added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. kind/task Internal things, technical debt, and to-do tasks to be performed. labels Oct 3, 2017
@codenvy-ci
Copy link

Build # 3967 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/3967/ to view the results.

@garagatyi
Copy link

Please squash this PR before merge

@dmytro-ndp
Copy link
Contributor

dmytro-ndp commented Oct 4, 2017

@garagatyi: we can use "Squash and merge" button instead, can't we?

private static final String METHODS_MARKED_AS_IGNORED =
"//div[contains(@id,'gwt-uid')]//div[@style='text-decoration: line-through; color: yellow;']";

private static final String METHODS_MARKED_AS_PASSED = "gwt-debug-test_state_passed";
Copy link
Contributor

Choose a reason for hiding this comment

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

Use - instead of _

@musienko-maxim
Copy link
Contributor Author

ci-build

@garagatyi
Copy link

we can use "Squash and merge" button instead, can't we?

yes

@codenvy-ci
Copy link

@musienko-maxim musienko-maxim merged commit 47da5d3 into master Oct 5, 2017
@musienko-maxim musienko-maxim deleted the CHE#6497 branch October 5, 2017 11:14
@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 5, 2017
@benoitf benoitf added this to the 5.19.0 milestone Oct 5, 2017
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.

8 participants