Skip to content

Selenium: update "JsonFileEditingTest" selenium test according to language servers test plan#11307

Merged
SkorikSergey merged 5 commits intomasterfrom
updateJsonFileEditingTest
Sep 21, 2018
Merged

Selenium: update "JsonFileEditingTest" selenium test according to language servers test plan#11307
SkorikSergey merged 5 commits intomasterfrom
updateJsonFileEditingTest

Conversation

@SkorikSergey
Copy link
Copy Markdown
Contributor

What does this PR do?

Update "JsonFileEditingTest" selenium test according to language servers test plan.

What issues does this PR fix or reference?

#11219

@SkorikSergey SkorikSergey 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. team/che-qe labels Sep 21, 2018
"Item with expected visible text: \"%s\" has not been detected", visibleText);
throw new RuntimeException(exceptionMessage);
}
String exceptionMessage =
Copy link
Copy Markdown
Contributor

@dmytro-ndp dmytro-ndp Sep 21, 2018

Choose a reason for hiding this comment

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

Empty row before would improve readability.

@SkorikSergey SkorikSergey merged commit 6bb6e8b into master Sep 21, 2018
@SkorikSergey SkorikSergey deleted the updateJsonFileEditingTest branch September 21, 2018 12:46
@SkorikSergey SkorikSergey removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Sep 21, 2018
@benoitf benoitf added this to the 6.12.0 milestone Sep 21, 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.

4 participants