From 0c6829673d1b51cce136c47beb4a080a4d82f7be Mon Sep 17 00:00:00 2001
From: Sauli Tähkäpää <sauli@vaadin.com>
Date: Thu, 18 Sep 2014 08:10:20 +0300
Subject: Make CalendarNotificationsTestIE work on IE8.

Change-Id: I395a03fe2c1815004a72e2bb3ecde937d504e969
---
 .../calendar/CalendarNotificationsTestIE.java      | 22 ++++++++--------------
 .../src/com/vaadin/tests/tb3/MultiBrowserTest.java | 10 ++++++++++
 2 files changed, 18 insertions(+), 14 deletions(-)

(limited to 'uitest/src')

diff --git a/uitest/src/com/vaadin/tests/components/calendar/CalendarNotificationsTestIE.java b/uitest/src/com/vaadin/tests/components/calendar/CalendarNotificationsTestIE.java
index 7ab6f01113..933bca8c7d 100644
--- a/uitest/src/com/vaadin/tests/components/calendar/CalendarNotificationsTestIE.java
+++ b/uitest/src/com/vaadin/tests/components/calendar/CalendarNotificationsTestIE.java
@@ -41,24 +41,18 @@ public class CalendarNotificationsTestIE extends MultiBrowserTest {
     }
 
     @Override
-    protected DesiredCapabilities getDesiredCapabilities() {
-        DesiredCapabilities desiredCapabilities = new DesiredCapabilities(
-                super.getDesiredCapabilities());
-        desiredCapabilities.setCapability("enablePersistentHover", false);
-        desiredCapabilities.setCapability("requireWindowFocus", true);
+    protected boolean requireWindowFocusForIE() {
+        return true;
+    }
 
-        return desiredCapabilities;
+    @Override
+    protected boolean usePersistentHoverForIE() {
+        return false;
     }
 
     @Override
     public List<DesiredCapabilities> getBrowsersToTest() {
-        List<DesiredCapabilities> browsers = super.getBrowsersToTest();
-        browsers.remove(Browser.CHROME.getDesiredCapabilities());
-        browsers.remove(Browser.FIREFOX.getDesiredCapabilities());
-        browsers.remove(Browser.OPERA.getDesiredCapabilities());
-        browsers.remove(Browser.PHANTOMJS.getDesiredCapabilities());
-        browsers.remove(Browser.SAFARI.getDesiredCapabilities());
-        return browsers;
+        return getIEBrowsersOnly();
     }
 
     @Test
@@ -68,7 +62,7 @@ public class CalendarNotificationsTestIE extends MultiBrowserTest {
         WebElement day = findElements(By.className("v-calendar-day-number"))
                 .get(2);
         // IE8 requires you to click on the text part to fire the event
-        new Actions(getDriver()).moveToElement(day, 83, 11).click().perform();
+        new Actions(getDriver()).moveToElement(day, day.getSize().getWidth() - 3, day.getSize().getHeight() / 2).click().perform();
 
         // check that a notification was opened, this is done with a log instead
         // of a screenshot or element presence check due to problems with IE
diff --git a/uitest/src/com/vaadin/tests/tb3/MultiBrowserTest.java b/uitest/src/com/vaadin/tests/tb3/MultiBrowserTest.java
index 414aebbd33..72fb2c18d1 100644
--- a/uitest/src/com/vaadin/tests/tb3/MultiBrowserTest.java
+++ b/uitest/src/com/vaadin/tests/tb3/MultiBrowserTest.java
@@ -58,6 +58,16 @@ public abstract class MultiBrowserTest extends PrivateTB3Configuration {
         browsers.remove(Browser.FIREFOX.getDesiredCapabilities());
         browsers.remove(Browser.PHANTOMJS.getDesiredCapabilities());
 
+		return browsers;
+	}
+
+    protected List<DesiredCapabilities> getIEBrowsersOnly() {
+        List<DesiredCapabilities> browsers = new ArrayList<DesiredCapabilities>();
+        browsers.add(Browser.IE8.getDesiredCapabilities());
+        browsers.add(Browser.IE9.getDesiredCapabilities());
+        browsers.add(Browser.IE10.getDesiredCapabilities());
+        browsers.add(Browser.IE11.getDesiredCapabilities());
+
         return browsers;
     }
 
-- 
cgit v1.2.3