Change showTooltip to wait for tooltip to appear

[rogozinds]

This change is 

[Artur-]

Review status: 0 of 3 files reviewed at latest revision, 3 unresolved discussions.

---

testbench-integration-tests/src/test/java/com/vaadin/tests/elements/TestbenchElementTooltipIT.java, line 18 at r1 (raw file):

    @Test
    public void elementCompareScreen() throws Exception {

Rename

---

testbench-integration-tests/src/test/java/com/vaadin/tests/elements/TestbenchElementTooltipIT.java, line 28 at r1 (raw file):

        Assert.assertEquals("Tooltip",tooltipText);
    }

There should be another test when you

1. Show tooltip for component 1
2. Show tooltip for component 2
3. Verify that tooltip for component 2 and not component 1 is shown

---

vaadin-testbench-core/src/main/java/com/vaadin/testbench/TestBenchElement.java, line 173 at r1 (raw file):

            .moveToElement(actualElement).perform();

        new WebDriverWait(getDriver(), 3)

Is 3s always enough?

---

Comments from Reviewable

[rogozinds]

Review status: 0 of 3 files reviewed at latest revision, 3 unresolved discussions.

---

testbench-integration-tests/src/test/java/com/vaadin/tests/elements/TestbenchElementTooltipIT.java, line 18 at r1 (raw file):

Previously, Artur- (Artur) wrote…

Rename

Done.

---

testbench-integration-tests/src/test/java/com/vaadin/tests/elements/TestbenchElementTooltipIT.java, line 28 at r1 (raw file):

Previously, Artur- (Artur) wrote…

There should be another test when you

1. Show tooltip for component 1
2. Show tooltip for component 2
3. Verify that tooltip for component 2 and not component 1 is shown

Done.

---

vaadin-testbench-core/src/main/java/com/vaadin/testbench/TestBenchElement.java, line 173 at r1 (raw file):

Previously, Artur- (Artur) wrote…

Is 3s always enough?

it was 0.75 before

---

Comments from Reviewable

[rogozinds]

Don't merge, does not work in IE11

---

Review status: 0 of 3 files reviewed at latest revision, 1 unresolved discussion.

---

Comments from Reviewable

[rogozinds]

Turned out that the new tests won't pass with previous code neither.

---

Review status: 0 of 3 files reviewed at latest revision, 1 unresolved discussion.

---

Comments from Reviewable

[alvarezguille]

what was the decision here? does it require a special fix for IE?

[Artur-]

No reaction in half a year, please reopen if this is still relevant