Skip to content

Fix tests in org.eclipse.che.selenium.editor.autocomplete#10975

Merged
svor merged 1 commit into5730_java_ls_pocfrom
autocomplite
Aug 30, 2018
Merged

Fix tests in org.eclipse.che.selenium.editor.autocomplete#10975
svor merged 1 commit into5730_java_ls_pocfrom
autocomplite

Conversation

@svor
Copy link
Copy Markdown
Contributor

@svor svor commented Aug 28, 2018

Signed-off-by: Valeriy Svydenko vsvydenk@redhat.com

What does this PR do?

Fixes tests in autocomplete package

What issues does this PR fix or reference?

#10974

Release Notes

Docs PR

@svor svor added kind/bug Outline of a bug - must adhere to the bug report template. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. target/branch Indicates that a PR will be merged into a branch other than master. team/languages labels Aug 28, 2018
@svor svor self-assigned this Aug 28, 2018
@svor svor requested a review from tsmaeder August 28, 2018 19:46
popupElement.appendChild(headerElement);

popupBodyElement = Elements.createDivElement(popupResources.popupStyle().body());
popupBodyElement.setClassName("signaturesContent");
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Isn't gwt-debug-id the way to go here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

replaced to ID

* @return true - if {@code expectedText} is present in javadoc body, false - if not
*/
public boolean waitAndCheckTextPresenceInJavaDoc(String expectedText) {
seleniumWebDriverHelper.moveCursorTo(By.xpath(JAVA_DOC_POPUP));
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is Javadoc still a thing? Shouldn't this be some generic language server popup?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

renamed to TEXT_VIEW_TOOLTIP

Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
@svor svor merged commit 0b2ea49 into 5730_java_ls_poc Aug 30, 2018
@svor svor deleted the autocomplite branch August 30, 2018 08:45
@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 Aug 30, 2018
tolusha pushed a commit that referenced this pull request Sep 5, 2018
Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
tsmaeder pushed a commit that referenced this pull request Sep 13, 2018
Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
tsmaeder pushed a commit that referenced this pull request Sep 20, 2018
Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
tsmaeder pushed a commit that referenced this pull request Sep 26, 2018
Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
tsmaeder pushed a commit that referenced this pull request Oct 1, 2018
Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
tsmaeder pushed a commit that referenced this pull request Oct 5, 2018
Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
tsmaeder pushed a commit that referenced this pull request Oct 12, 2018
Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
tsmaeder pushed a commit that referenced this pull request Oct 16, 2018
Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
tsmaeder pushed a commit that referenced this pull request Oct 17, 2018
Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
tsmaeder pushed a commit that referenced this pull request Oct 17, 2018
Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
tsmaeder pushed a commit that referenced this pull request Oct 17, 2018
Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
tsmaeder pushed a commit that referenced this pull request Oct 17, 2018
Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
tsmaeder pushed a commit that referenced this pull request Oct 17, 2018
Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bug Outline of a bug - must adhere to the bug report template. target/branch Indicates that a PR will be merged into a branch other than master.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants