]> source.dussan.org Git - vaadin-framework.git/commitdiff
Refactor AbstractComponentTest to extend AbstractTestUI
authorJohannes Dahlström <johannesd@vaadin.com>
Wed, 31 Aug 2016 14:33:21 +0000 (17:33 +0300)
committerJohannes Dahlström <johannesd@vaadin.com>
Mon, 5 Sep 2016 14:05:20 +0000 (17:05 +0300)
Change-Id: Ie614e7a7bf069ab4035cfcd35c3bc98007adfd4b

55 files changed:
uitest/src/main/java/com/vaadin/tests/Components.java [deleted file]
uitest/src/main/java/com/vaadin/tests/components/AbstractComponentTest.java
uitest/src/main/java/com/vaadin/tests/components/AbstractComponentTestCase.java
uitest/src/main/java/com/vaadin/tests/components/ComponentTestCase.java
uitest/src/main/java/com/vaadin/tests/components/TouchDevicesTooltip.java
uitest/src/main/java/com/vaadin/tests/components/button/ButtonHtml.java
uitest/src/main/java/com/vaadin/tests/components/button/Buttons.java
uitest/src/main/java/com/vaadin/tests/components/button/ButtonsWaiAria.java
uitest/src/main/java/com/vaadin/tests/components/checkbox/CheckBoxes.java
uitest/src/main/java/com/vaadin/tests/components/combobox/ComboBoxPageLength.java
uitest/src/main/java/com/vaadin/tests/components/combobox/ComboBoxScrollingToPageDisabled.java
uitest/src/main/java/com/vaadin/tests/components/combobox/Comboboxes.java
uitest/src/main/java/com/vaadin/tests/components/combobox/FilteringTurkishLocale.java
uitest/src/main/java/com/vaadin/tests/components/datefield/InlineDateFields.java
uitest/src/main/java/com/vaadin/tests/components/datefield/PopupDateFields.java
uitest/src/main/java/com/vaadin/tests/components/gridlayout/ComponentAlignments.java
uitest/src/main/java/com/vaadin/tests/components/label/LabelModes.java
uitest/src/main/java/com/vaadin/tests/components/label/Labels.java
uitest/src/main/java/com/vaadin/tests/components/menubar/MenuBarHtmlItems.java
uitest/src/main/java/com/vaadin/tests/components/menubar/MenuBarNavigation.java
uitest/src/main/java/com/vaadin/tests/components/menubar/MenuBarTest.java
uitest/src/main/java/com/vaadin/tests/components/menubar/MenuItemStyleRemoved.java
uitest/src/main/java/com/vaadin/tests/components/menubar/Menubars.java
uitest/src/main/java/com/vaadin/tests/components/notification/ChromeBottomNotification.java
uitest/src/main/java/com/vaadin/tests/components/optiongroup/DisabledOptionGroupItems.java
uitest/src/main/java/com/vaadin/tests/components/optiongroup/HtmlOptionGroupItems.java
uitest/src/main/java/com/vaadin/tests/components/select/NativeSelects.java
uitest/src/main/java/com/vaadin/tests/components/table/ContextMenuSize.java
uitest/src/main/java/com/vaadin/tests/components/table/Footer.java
uitest/src/main/java/com/vaadin/tests/components/table/FooterClick.java
uitest/src/main/java/com/vaadin/tests/components/table/HeaderClick.java
uitest/src/main/java/com/vaadin/tests/components/table/HeaderFooterClickLeftRightMiddle.java
uitest/src/main/java/com/vaadin/tests/components/table/HeaderPositionWhenSorting.java
uitest/src/main/java/com/vaadin/tests/components/table/HeaderSyncOnScroll.java
uitest/src/main/java/com/vaadin/tests/components/table/HeaderUpdateWhenNoRows.java
uitest/src/main/java/com/vaadin/tests/components/table/TableSetUndefinedSize.java
uitest/src/main/java/com/vaadin/tests/components/table/TableVisibleColumnsUpdate.java
uitest/src/main/java/com/vaadin/tests/components/tabsheet/FirstTabNotVisibleWhenTabsheetNotClipped.java
uitest/src/main/java/com/vaadin/tests/components/tabsheet/VetoTabChange.java
uitest/src/main/java/com/vaadin/tests/components/upload/DisabledUploadButton.java
uitest/src/main/java/com/vaadin/tests/components/upload/TestImmediateUploadInFormLayout.java
uitest/src/main/java/com/vaadin/tests/components/upload/TestUpload.java
uitest/src/main/java/com/vaadin/tests/components/upload/TestUploadAndDisableOnSuccess.java
uitest/src/main/java/com/vaadin/tests/components/window/WindowTest.java
uitest/src/main/java/com/vaadin/tests/debug/PushVersionInfo.java
uitest/src/main/java/com/vaadin/tests/extensions/IframeIsOpenedInNonIOS.java
uitest/src/main/java/com/vaadin/tests/fieldgroup/BasicPersonForm.java
uitest/src/main/java/com/vaadin/tests/fieldgroup/DateForm.java
uitest/src/main/java/com/vaadin/tests/themes/ThemeChangeFavicon.java
uitest/src/main/java/com/vaadin/v7/tests/components/grid/GridCheckBoxDisplay.java
uitest/src/main/java/com/vaadin/v7/tests/components/grid/GridDetailsWidth.java
uitest/src/main/java/com/vaadin/v7/tests/components/grid/GridEditorMultiselect.java
uitest/src/main/java/com/vaadin/v7/tests/components/grid/GridResizeHiddenColumn.java
uitest/src/main/webapp/WEB-INF/web.xml
uitest/src/test/java/com/vaadin/tests/applicationservlet/MultipleServletConfigurationTest.java

diff --git a/uitest/src/main/java/com/vaadin/tests/Components.java b/uitest/src/main/java/com/vaadin/tests/Components.java
deleted file mode 100644 (file)
index eea70f6..0000000
+++ /dev/null
@@ -1,270 +0,0 @@
-package com.vaadin.tests;
-
-import java.lang.reflect.Modifier;
-import java.util.ArrayList;
-import java.util.Collection;
-import java.util.HashMap;
-import java.util.HashSet;
-import java.util.List;
-import java.util.Map;
-import java.util.Set;
-
-import com.vaadin.server.ExternalResource;
-import com.vaadin.server.LegacyApplication;
-import com.vaadin.server.Sizeable;
-import com.vaadin.shared.MouseEventDetails.MouseButton;
-import com.vaadin.shared.ui.label.ContentMode;
-import com.vaadin.tests.components.AbstractComponentTest;
-import com.vaadin.ui.AbstractComponent;
-import com.vaadin.ui.Component;
-import com.vaadin.ui.Embedded;
-import com.vaadin.ui.HorizontalSplitPanel;
-import com.vaadin.ui.Label;
-import com.vaadin.ui.LegacyWindow;
-import com.vaadin.ui.VerticalLayout;
-import com.vaadin.v7.data.Item;
-import com.vaadin.v7.data.util.DefaultItemSorter;
-import com.vaadin.v7.data.util.HierarchicalContainer;
-import com.vaadin.v7.event.ItemClickEvent;
-import com.vaadin.v7.event.ItemClickEvent.ItemClickListener;
-import com.vaadin.v7.ui.Tree;
-import com.vaadin.v7.ui.Tree.ItemStyleGenerator;
-
-public class Components extends LegacyApplication {
-
-    private static final Object CAPTION = "c";
-    private Map<Class<? extends AbstractComponentTest>, String> tests = new HashMap<>();
-    private Tree naviTree;
-    private HorizontalSplitPanel sp;
-    private LegacyWindow mainWindow;
-    private final Embedded applicationEmbedder = new Embedded();
-    private String baseUrl;
-    private List<Class<? extends Component>> componentsWithoutTests = new ArrayList<>();
-
-    {
-        for (Class<?> c : VaadinClasses.getBasicComponentTests()) {
-            String testClass = c.getSimpleName();
-            tests.put((Class<? extends AbstractComponentTest>) c, testClass);
-        }
-
-        List<Class<? extends Component>> componentsWithoutTest = VaadinClasses
-                .getComponents();
-        Set<String> availableTests = new HashSet<>();
-        for (String testName : tests.values()) {
-            availableTests.add(testName);
-        }
-
-        for (Class<? extends Component> component : componentsWithoutTest) {
-            String baseName = component.getSimpleName();
-            if (availableTests.contains(baseName + "es")) {
-                continue;
-            }
-            if (availableTests.contains(baseName + "es2")) {
-                continue;
-            }
-            if (availableTests.contains(baseName + "s2")) {
-                continue;
-            }
-            if (availableTests.contains(baseName + "s")) {
-                continue;
-            }
-            if (availableTests.contains(baseName + "Test")) {
-                continue;
-            }
-
-            componentsWithoutTests.add(component);
-        }
-
-    }
-
-    class MissingTest extends AbstractComponentTest<AbstractComponent> {
-        @Override
-        protected Class<AbstractComponent> getTestClass() {
-            return null;
-        }
-    }
-
-    @Override
-    public void init() {
-        mainWindow = new LegacyWindow();
-        setTheme("tests-components");
-        mainWindow.getContent().setSizeFull();
-        setMainWindow(mainWindow);
-        sp = new HorizontalSplitPanel();
-        sp.setSizeFull();
-        VerticalLayout naviLayout = new VerticalLayout();
-        naviLayout.addComponent(new Label(
-                "Click to open a test case.<br/>Right click to open test in a new window<br/><br/>",
-                ContentMode.HTML));
-        naviLayout.addComponent(createMenu());
-        naviLayout.addComponent(createMissingTestsList());
-
-        sp.setFirstComponent(naviLayout);
-        sp.setSplitPosition(250, Sizeable.UNITS_PIXELS);
-        VerticalLayout embeddingLayout = new VerticalLayout();
-        embeddingLayout.setSizeFull();
-        embeddingLayout.addComponent(new Label(
-                "<b>Do not use the embedded version for creating automated tests. Open the test in a new window before recording.</b><br/>",
-                ContentMode.HTML));
-        applicationEmbedder.setSizeFull();
-        embeddingLayout.addComponent(applicationEmbedder);
-        embeddingLayout.setExpandRatio(applicationEmbedder, 1);
-        sp.setSecondComponent(embeddingLayout);
-        mainWindow.addComponent(sp);
-
-        applicationEmbedder.setType(Embedded.TYPE_BROWSER);
-        baseUrl = getURL().toString().replace(getClass().getName(), "")
-                .replaceAll("//$", "/");
-    }
-
-    private Component createMissingTestsList() {
-        String missingTests = "";
-        for (Class<? extends Component> component : componentsWithoutTests) {
-            String cls = "missing";
-            if (component.getAnnotation(Deprecated.class) != null) {
-                cls = "missing-deprecated";
-            }
-            missingTests += "<font class=\"" + cls + "\">"
-                    + component.getSimpleName() + "</font><br/>";
-        }
-        return new Label(
-                "<b>Components without a test:</B><br/>" + missingTests,
-                ContentMode.HTML);
-    }
-
-    private Component createMenu() {
-        naviTree = new Tree();
-        naviTree.setItemStyleGenerator(new ItemStyleGenerator() {
-
-            @Override
-            public String getStyle(Tree source, Object itemId) {
-                Class<?> cls = (Class<?>) itemId;
-                if (!isAbstract(cls)) {
-                    return "blue";
-                }
-                return null;
-            }
-        });
-        HierarchicalContainer hc = new HierarchicalContainer();
-        naviTree.setContainerDataSource(hc);
-        DefaultItemSorter sorter = new DefaultItemSorter() {
-            @SuppressWarnings("rawtypes")
-            @Override
-            public int compare(Object o1, Object o2) {
-                if (o1 instanceof Class && o2 instanceof Class && o1 != null
-                        && o2 != null) {
-                    Class<?> c1 = (Class) o1;
-                    Class<?> c2 = (Class) o2;
-                    boolean a1 = isAbstract(c1);
-                    boolean a2 = isAbstract(c2);
-
-                    if (a1 && !a2) {
-                        return 1;
-                    } else if (!a1 && a2) {
-                        return -1;
-                    }
-
-                }
-                return super.compare(o1, o2);
-            }
-        };
-        hc.setItemSorter(sorter);
-        naviTree.addContainerProperty(CAPTION, String.class, "");
-        naviTree.setItemCaptionPropertyId(CAPTION);
-        for (Class<? extends AbstractComponentTest> cls : tests.keySet()) {
-            addTreeItem(cls);
-        }
-        hc.sort(new Object[] { CAPTION }, new boolean[] { true });
-        naviTree.setSelectable(false);
-        for (Object o : naviTree.rootItemIds()) {
-            expandAndSetChildrenAllowed(o);
-        }
-
-        naviTree.addListener(new ItemClickListener() {
-
-            @Override
-            public void itemClick(ItemClickEvent event) {
-                Class<?> cls = (Class<?>) event.getItemId();
-                if (!isAbstract(cls)) {
-                    String url = baseUrl + cls.getName()
-                            + "?restartApplication";
-                    if (event.getButton() == MouseButton.LEFT) {
-                        openEmbedded(url);
-                        naviTree.setValue(event.getItemId());
-                    } else if (event.getButton() == MouseButton.RIGHT) {
-                        openInNewTab(url);
-                    }
-                }
-            }
-
-        });
-        return naviTree;
-    }
-
-    protected void openInNewTab(String url) {
-        getMainWindow().open(new ExternalResource(url), "_blank");
-    }
-
-    protected void openEmbedded(String url) {
-        applicationEmbedder.setSource(new ExternalResource(url));
-    }
-
-    private void expandAndSetChildrenAllowed(Object o) {
-        Collection<?> children = naviTree.getChildren(o);
-        if (children == null || children.size() == 0) {
-            naviTree.setChildrenAllowed(o, false);
-        } else {
-            naviTree.expandItem(o);
-            for (Object c : children) {
-                expandAndSetChildrenAllowed(c);
-            }
-        }
-
-    }
-
-    protected boolean isAbstract(Class<?> cls) {
-        return Modifier.isAbstract(cls.getModifiers());
-    }
-
-    @SuppressWarnings("unchecked")
-    private void addTreeItem(Class<? extends AbstractComponentTest> cls) {
-        String name = tests.get(cls);
-        if (name == null) {
-            name = cls.getSimpleName();
-        }
-
-        Class<? extends AbstractComponentTest> superClass = (Class<? extends AbstractComponentTest>) cls
-                .getSuperclass();
-
-        // This cast is needed only to make compilation through Ant work ..
-        if (((Class<?>) cls) != AbstractComponentTest.class) {
-            addTreeItem(superClass);
-        }
-        if (naviTree.containsId(cls)) {
-            return;
-        }
-
-        Item i = naviTree.addItem(cls);
-        i.getItemProperty(CAPTION).setValue(name);
-        naviTree.setParent(cls, superClass);
-    }
-
-    protected Component createTestComponent(
-            Class<? extends AbstractComponentTest> cls) {
-        try {
-            AbstractComponentTest t = cls.newInstance();
-            t.init();
-            Component c = t.getMainWindow().getContent();
-            t.getMainWindow().setContent(null);
-            return c;
-        } catch (InstantiationException e) {
-            // TODO Auto-generated catch block
-            e.printStackTrace();
-        } catch (IllegalAccessException e) {
-            // TODO Auto-generated catch block
-            e.printStackTrace();
-        }
-        return null;
-    }
-
-}
index 2c51bd132d7173ee6742b54c00468f4dd5504369..1e0f54bebc37552fc3e85ea2d976b5cd64ca9cb5 100644 (file)
@@ -9,23 +9,24 @@ import java.util.Locale;
 import java.util.Map;
 import java.util.Set;
 
+import com.vaadin.annotations.Theme;
 import com.vaadin.event.FieldEvents.BlurEvent;
 import com.vaadin.event.FieldEvents.BlurListener;
 import com.vaadin.event.FieldEvents.BlurNotifier;
 import com.vaadin.event.FieldEvents.FocusEvent;
 import com.vaadin.event.FieldEvents.FocusListener;
 import com.vaadin.event.FieldEvents.FocusNotifier;
-import com.vaadin.server.DefaultErrorHandler;
 import com.vaadin.server.Resource;
 import com.vaadin.server.ThemeResource;
+import com.vaadin.server.VaadinRequest;
 import com.vaadin.tests.util.Log;
 import com.vaadin.tests.util.LoremIpsum;
 import com.vaadin.ui.AbstractComponent;
-import com.vaadin.ui.Component.Focusable;
 import com.vaadin.ui.MenuBar;
 import com.vaadin.ui.MenuBar.MenuItem;
 import com.vaadin.ui.themes.BaseTheme;
 
+@Theme("tests-components")
 public abstract class AbstractComponentTest<T extends AbstractComponent> extends
         AbstractComponentTestCase<T> implements FocusListener, BlurListener {
 
@@ -81,15 +82,14 @@ public abstract class AbstractComponentTest<T extends AbstractComponent> extends
     protected static final String CATEGORY_DECORATIONS = "Decorations";
 
     @Override
-    protected final void setup() {
-        setTheme("tests-components");
+    protected final void setup(VaadinRequest request) {
 
         // Create menu here so it appears before the components
         addComponent(createMainMenu());
 
         getLayout().setSizeFull();
         createLog();
-        super.setup();
+        super.setup(request);
 
         // Create menu actions and trigger default actions
         createActions();
@@ -747,16 +747,6 @@ public abstract class AbstractComponentTest<T extends AbstractComponent> extends
         super.doCommand(commandName, command, value, data);
     }
 
-    @Override
-    public void error(com.vaadin.server.ErrorEvent event) {
-        final Throwable throwable = DefaultErrorHandler
-                .findRelevantThrowable(event.getThrowable());
-
-        log.log("Exception occured, " + throwable.getClass().getName() + ": "
-                + throwable.getMessage());
-        throwable.printStackTrace();
-    }
-
     @Override
     public void focus(FocusEvent event) {
         log(event.getClass().getSimpleName());
index bdb1ff3f836486fa9b00a4c8f0864e5c17b98bed..c52e332b94096d9f1b934a4dda826c16f0f5b03a 100644 (file)
@@ -9,12 +9,13 @@ import com.vaadin.data.HasRequired;
 import com.vaadin.server.Resource;
 import com.vaadin.server.ThemeResource;
 import com.vaadin.server.UserError;
+import com.vaadin.server.VaadinRequest;
 import com.vaadin.ui.AbstractComponent;
 import com.vaadin.ui.Layout.SpacingHandler;
 import com.vaadin.v7.ui.Field;
 
 public abstract class AbstractComponentTestCase<T extends AbstractComponent>
-        extends TestBase {
+        extends AbstractTestUI {
 
     protected static final ThemeResource ICON_16_HELP_PNG_CACHEABLE = cacheableThemeResource(
             "../runo/icons/16/help.png");
@@ -52,7 +53,7 @@ public abstract class AbstractComponentTestCase<T extends AbstractComponent>
     abstract protected void initializeComponents();
 
     @Override
-    protected void setup() {
+    protected void setup(VaadinRequest request) {
         ((SpacingHandler) getLayout()).setSpacing(true);
 
         // Create Components
@@ -77,136 +78,66 @@ public abstract class AbstractComponentTestCase<T extends AbstractComponent>
         public void execute(T c, VALUETYPE value, Object data);
     }
 
-    /* COMMANDS */
-
-    protected Command<T, String> widthCommand = new Command<T, String>() {
-
-        @Override
-        public void execute(T t, String value, Object data) {
-            t.setWidth(value);
-        }
-    };
-    protected Command<T, String> heightCommand = new Command<T, String>() {
-
-        @Override
-        public void execute(T t, String value, Object data) {
-            t.setHeight(value);
-        }
-    };
-
-    protected Command<T, Boolean> enabledCommand = new Command<T, Boolean>() {
-
-        @Override
-        public void execute(T c, Boolean enabled, Object data) {
-            c.setEnabled(enabled);
-        }
-    };
-
-    protected Command<T, Boolean> immediateCommand = new Command<T, Boolean>() {
-
-        @Override
-        public void execute(T c, Boolean immediate, Object data) {
-            c.setImmediate(immediate);
-        }
-    };
-
-    protected Command<T, Boolean> errorIndicatorCommand = new Command<T, Boolean>() {
-
-        @Override
-        public void execute(T c, Boolean enabled, Object data) {
-            if (enabled) {
-                c.setComponentError(new UserError(errorMessage));
-            } else {
-                c.setComponentError(null);
-
-            }
-        }
-    };
     private String errorMessage = null;
 
-    protected Command<T, String> errorMessageCommand = new Command<T, String>() {
-
-        @Override
-        public void execute(T c, String value, Object data) {
-            errorMessage = value;
-            if (c.getComponentError() != null) {
-                errorIndicatorCommand.execute(c, true, null);
-            }
-
-        }
-
-    };
+    /* COMMANDS */
 
-    // TODO Move to AbstractFieldTestCase
-    protected Command<T, Boolean> requiredCommand = new Command<T, Boolean>() {
-
-        @Override
-        public void execute(T c, Boolean enabled, Object data) {
-            if (c instanceof HasRequired) {
-                ((HasRequired) c).setRequired(enabled);
-            } else {
-                throw new IllegalArgumentException(c.getClass().getName()
-                        + " is not a field and cannot be set to required");
-            }
-        }
-    };
-    protected Command<T, String> requiredErrorMessageCommand = new Command<T, String>() {
+    protected Command<T, String> widthCommand = (t, value, data) -> t.setWidth(
+            value);
+    protected Command<T, String> heightCommand = (t, value, data) -> t
+            .setHeight(value);
 
-        @Override
-        public void execute(T c, String value, Object data) {
-            ((Field<?>) c).setRequiredError(value);
-        }
+    protected Command<T, Boolean> enabledCommand = (c, enabled, data) -> c
+            .setEnabled(enabled);
 
-    };
+    protected Command<T, Boolean> immediateCommand = (c, immediate, data) -> c
+            .setImmediate(immediate);
 
-    protected Command<T, String> descriptionCommand = new Command<T, String>() {
-        @Override
-        public void execute(T c, String value, Object data) {
-            c.setDescription(value);
+    protected Command<T, Boolean> errorIndicatorCommand = (c, enabled,
+            data) -> {
+        if (enabled) {
+            c.setComponentError(new UserError(errorMessage));
+        } else {
+            c.setComponentError(null);
         }
     };
 
-    protected Command<T, Boolean> readonlyCommand = new Command<T, Boolean>() {
-
-        @Override
-        public void execute(T c, Boolean enabled, Object data) {
-            c.setReadOnly(enabled);
+    protected Command<T, String> errorMessageCommand = (c, value, data) -> {
+        errorMessage = value;
+        if (c.getComponentError() != null) {
+            errorIndicatorCommand.execute(c, true, null);
         }
-    };
-
-    protected Command<T, Boolean> visibleCommand = new Command<T, Boolean>() {
 
-        @Override
-        public void execute(T c, Boolean enabled, Object data) {
-            c.setVisible(enabled);
-        }
     };
 
-    protected Command<T, Resource> iconCommand = new Command<T, Resource>() {
-
-        @Override
-        public void execute(T c, Resource value, Object data) {
-            c.setIcon(value);
+    // TODO Move to AbstractFieldTestCase
+    protected Command<T, Boolean> requiredCommand = (c, enabled, data) -> {
+        if (c instanceof HasRequired) {
+            ((HasRequired) c).setRequired(enabled);
+        } else {
+            throw new IllegalArgumentException(c.getClass().getName()
+                    + " is not a field and cannot be set to required");
         }
-
     };
-    protected Command<T, String> captionCommand = new Command<T, String>() {
+    protected Command<T, String> requiredErrorMessageCommand = (c, value,
+            data) -> ((Field<?>) c).setRequiredError(value);
 
-        @Override
-        public void execute(T c, String value, Object data) {
-            c.setCaption(value);
-        }
+    protected Command<T, String> descriptionCommand = (c, value, data) -> c
+            .setDescription(value);
 
-    };
+    protected Command<T, Boolean> readonlyCommand = (c, enabled, data) -> c
+            .setReadOnly(enabled);
 
-    protected Command<T, Locale> localeCommand = new Command<T, Locale>() {
+    protected Command<T, Boolean> visibleCommand = (c, enabled, data) -> c
+            .setVisible(enabled);
 
-        @Override
-        public void execute(T c, Locale value, Object data) {
-            c.setLocale(value);
-        }
+    protected Command<T, Resource> iconCommand = (c, value, data) -> c.setIcon(
+            value);
+    protected Command<T, String> captionCommand = (c, value, data) -> c
+            .setCaption(value);
 
-    };
+    protected Command<T, Locale> localeCommand = (c, value, data) -> c
+            .setLocale(value);
 
     protected <VALUET> void doCommand(Command<T, VALUET> command,
             VALUET value) {
@@ -230,23 +161,14 @@ public abstract class AbstractComponentTestCase<T extends AbstractComponent>
         doCommand(command, value, data);
     }
 
-    protected Command<T, String> styleNameCommand = new Command<T, String>() {
-        @Override
-        public void execute(T c, String value, Object data) {
-            c.setStyleName(value);
-        }
-    };
+    protected Command<T, String> styleNameCommand = (c, value, data) -> c
+            .setStyleName(value);
 
-    protected Command<T, String> primaryStyleNameCommand = new Command<T, String>() {
-        @Override
-        public void execute(T c, String value, Object data) {
-            c.setPrimaryStyleName(value);
-        }
-    };
+    protected Command<T, String> primaryStyleNameCommand = (c, value, data) -> c
+            .setPrimaryStyleName(value);
 
     @Override
-    protected String getDescription() {
+    protected String getTestDescription() {
         return "Generic test case for " + getTestClass().getSimpleName();
     }
-
 }
index 2786eb85589643b56e70cef0a84bcacd942b57e1..86a743001b1b150eb8a77604bfe0b5a0ad4c0b3d 100644 (file)
@@ -4,16 +4,14 @@ import java.util.ArrayList;
 import java.util.LinkedHashMap;
 import java.util.List;
 
+import com.vaadin.server.VaadinRequest;
 import com.vaadin.ui.AbstractComponent;
 import com.vaadin.ui.Alignment;
 import com.vaadin.ui.Button;
-import com.vaadin.ui.Button.ClickEvent;
 import com.vaadin.ui.CheckBox;
 import com.vaadin.ui.Component;
 import com.vaadin.ui.HorizontalLayout;
 import com.vaadin.v7.data.Item;
-import com.vaadin.v7.data.Property;
-import com.vaadin.v7.data.Property.ValueChangeEvent;
 import com.vaadin.v7.ui.Field;
 import com.vaadin.v7.ui.NativeSelect;
 
@@ -25,12 +23,12 @@ public abstract class ComponentTestCase<T extends AbstractComponent>
     private HorizontalLayout actionLayout;
 
     @Override
-    protected final void setup() {
+    protected final void setup(VaadinRequest request) {
         // Create action layout so it appears before the components
         actionLayout = createActionLayout();
         addComponent(actionLayout);
 
-        super.setup();
+        super.setup(request);
 
         // Create actions and add to layout
         populateActionLayout();
@@ -129,14 +127,11 @@ public abstract class ComponentTestCase<T extends AbstractComponent>
 
         Button button = new Button(caption);
         button.setData(Boolean.FALSE);
-        button.addListener(new Button.ClickListener() {
-            @Override
-            public void buttonClick(ClickEvent event) {
-                Button b = event.getButton();
-                boolean state = (Boolean) b.getData();
-                b.setData(!state);
-                doCommand(command, state);
-            }
+        button.addClickListener(event -> {
+            Button b = event.getButton();
+            boolean state = (Boolean) b.getData();
+            b.setData(!state);
+            doCommand(command, state);
         });
 
         button.setId("buttonaction-" + caption);
@@ -156,17 +151,12 @@ public abstract class ComponentTestCase<T extends AbstractComponent>
         select.addContainerProperty(VALUE, Object.class, "");
         select.setItemCaptionPropertyId(CAPTION);
         select.setNullSelectionAllowed(false);
-        select.addListener(new Property.ValueChangeListener() {
-
-            @Override
-            public void valueChange(ValueChangeEvent event) {
-                Object itemId = event.getProperty().getValue();
-                Item item = select.getItem(itemId);
-                @SuppressWarnings("unchecked")
-                TYPE value = (TYPE) item.getItemProperty(VALUE).getValue();
-                doCommand(command, value);
-
-            }
+        select.addValueChangeListener(event -> {
+            Object itemId = event.getProperty().getValue();
+            Item item = select.getItem(itemId);
+            @SuppressWarnings("unchecked")
+            TYPE value = (TYPE) item.getItemProperty(VALUE).getValue();
+            doCommand(command, value);
         });
 
         for (String itemCaption : options.keySet()) {
@@ -177,7 +167,6 @@ public abstract class ComponentTestCase<T extends AbstractComponent>
             if (itemCaption.equals(initialValue)) {
                 select.setValue(itemId);
             }
-
         }
 
         select.setId("selectaction-" + caption);
index 8db90bf332b4afb5a7d05ff9b204362fae73c2ff..24ba6c3a202a36bbe36cc007b4167f41e6fdd65f 100644 (file)
@@ -62,7 +62,7 @@ public class TouchDevicesTooltip extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "Unable to dismiss a tooltip on touch devices";
     }
-}
\ No newline at end of file
+}
index 1c3070f38b973130304ee5e67a8a26e5cd91f485..0b1ab42667f158ced30dfe87bb092f499e6e9927 100644 (file)
@@ -4,6 +4,12 @@ import com.vaadin.tests.components.TestBase;
 import com.vaadin.ui.Button;
 import com.vaadin.ui.Button.ClickEvent;
 
+/*
+ * NOTE This class is arbitrarily picked to represent a legacy application in
+ * MultipleServletConfigurationTest and the corresponding "Embed App 1" servlet
+ * configuration. The test will break if this class is refactored to extend UI
+ * instead of LegacyApplication. Just a friendly warning.
+ */
 public class ButtonHtml extends TestBase {
 
     @Override
index a6c4e326feb901c2be162fa294c1a928ed62089b..2913c64a9826d34c8c1bd29439dbb94578883ec7 100644 (file)
@@ -64,7 +64,7 @@ public class Buttons extends ComponentTestCase<Button> {
     }
 
     @Override
-    protected String getDescription() {
+    protected String getTestDescription() {
         return "A generic test for Buttons in different configurations";
     }
 
index 24dc9421714185cd69d378c9eb726172140decce..de803256f5bec32e80a0dbfcf37e47232b8cee81 100644 (file)
@@ -50,7 +50,7 @@ public class ButtonsWaiAria extends ComponentTestCase<Button> {
     }
 
     @Override
-    protected String getDescription() {
+    protected String getTestDescription() {
         return "A generic test for Buttons in different configurations";
     }
 }
index 24f53889b319ac7ccba774538094a6e40aa6c128..87e7dae16f0819ea85769b5418816d9a0a8fec7e 100644 (file)
@@ -64,7 +64,7 @@ public class CheckBoxes extends ComponentTestCase<CheckBox> {
     }
 
     @Override
-    protected String getDescription() {
+    protected String getTestDescription() {
         return "A generic test for CheckBoxes in different configurations";
     }
 
index dc8b152185262aefe24ca71eddba4b2c77faa88e..bdf7131318894894d227dced2c290b0d2afd8c67 100644 (file)
@@ -33,9 +33,8 @@ public class ComboBoxPageLength extends ComboBoxes2<ComboBox> {
     }
 
     @Override
-    protected String getDescription() {
-        return super.getDescription()
+    protected String getTestDescription() {
+        return super.getTestDescription()
                 + ", changing ComboBox value will change the ComboBox pageLength to the # of the selected item, or to 0 in null selection.";
     }
-
 }
index e1c5c49a57990e695342e43115a9eb4c4a4b1b97..3bf724bad4e22e04fdcddc4ca803e37d22994fbb 100644 (file)
@@ -60,7 +60,7 @@ public class ComboBoxScrollingToPageDisabled
     }
 
     @Override
-    protected String getDescription() {
+    protected String getTestDescription() {
         return "Test that selected value appears on the client "
                 + "side even though setScrollToSelectedItem(false) "
                 + "has been called. Textbox should contain 'Item 50'.";
index e5c4a339922bf64f63776ca38a325cb753b419df..9ebc37a4bf8d0a13fd65c9172135f743de3c5980 100644 (file)
@@ -115,7 +115,7 @@ public class Comboboxes extends ComponentTestCase<ComboBox> {
     }
 
     @Override
-    protected String getDescription() {
+    protected String getTestDescription() {
         return "A generic test for ComboBoxes in different configurations";
     }
 
index 5d2ef019f166f4de17921955aa5e0cebe22e6e12..4624d213e9a7a88014354ec98bee3f075d5befc0 100644 (file)
@@ -48,10 +48,9 @@ public class FilteringTurkishLocale extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "When the Turkish locale is used,"
                 + " filtering for 'i' should show the option with a dot"
                 + " while filtering for 'ı' should show the option witout a dot";
     }
-
 }
index 81360970400db36055860d877a6d32b900b232ed..2c5f387292c191ba761768a935157f5928976583 100644 (file)
@@ -57,7 +57,7 @@ public class InlineDateFields extends ComponentTestCase<InlineDateField> {
     }
 
     @Override
-    protected String getDescription() {
+    protected String getTestDescription() {
         return "A generic test for InlineDateFields in different configurations";
     }
 
index 60e90a8f1b7a26339073d3781ae4ff4792b06e23..4847bb0c7060934fee85ae522b6f0652dd3aa82a 100644 (file)
@@ -53,7 +53,7 @@ public class PopupDateFields extends ComponentTestCase<PopupDateField> {
     }
 
     @Override
-    protected String getDescription() {
+    protected String getTestDescription() {
         return "A generic test for PopupDateFields in different configurations";
     }
 
index e50bdd8dcd7ab60cd4bc821320af9ec292cc792a..b84735c7403e68b6a12ee906587bcd3006cce43e 100644 (file)
@@ -55,8 +55,7 @@ public class ComponentAlignments extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "TOP_CENTER and TOP_RIGHT alignments should work in VerticalLayout";
     }
-
 }
index 6c61e16402ab51011ccdf50cac12c8dcf8afd3f6..dd6bb14aa2ab46a025964d99b43e589ee7da44c0 100644 (file)
@@ -51,7 +51,7 @@ public class LabelModes extends ComponentTestCase<Label> {
     }
 
     @Override
-    protected String getDescription() {
+    protected String getTestDescription() {
         return "A generic test for Labels in different configurations";
     }
 
index 0d296b1b92e40bc63dd11870239a748155b17545..416de00bdbbe13487abd91a4de37257c3acef97a 100644 (file)
@@ -67,7 +67,7 @@ public class Labels extends ComponentTestCase<Label> {
     }
 
     @Override
-    protected String getDescription() {
+    protected String getTestDescription() {
         return "A generic test for Labels in different configurations";
     }
 
index d6e79224a37687572b6dd0ece807c5a161ffdf61..068043c31cb6112b15137bacbc8b55cf74de51bd 100644 (file)
@@ -20,9 +20,9 @@ public class MenuBarHtmlItems extends ComponentTestCase<MenuBar> {
     @Override
     protected void initializeComponents() {
         MenuBar m = new MenuBar();
-        MenuItem submenu = m.addItem("Item <u>1</u>", getIcon(), null);
+        MenuItem submenu = m.addItem("Item <u>1</u>", getMenuIcon(), null);
         MenuItem subsubmenu = submenu.addItem("<b>Bold</b> item", null);
-        subsubmenu.addItem("<i><u>I</u>talic</i> item", getIcon(), null);
+        subsubmenu.addItem("<i><u>I</u>talic</i> item", getMenuIcon(), null);
         submenu.addItem(
                 "<span style='font-size: 30px'>Big</span> <span style='font-size: 8px'>disabled</span> item",
                 null).setEnabled(false);
@@ -32,7 +32,7 @@ public class MenuBarHtmlItems extends ComponentTestCase<MenuBar> {
         addTestComponent(m);
     }
 
-    private Resource getIcon() {
+    private Resource getMenuIcon() {
         return new ThemeResource("../runo/icons/16/user.png");
     }
 
@@ -57,7 +57,7 @@ public class MenuBarHtmlItems extends ComponentTestCase<MenuBar> {
     }
 
     @Override
-    protected String getDescription() {
+    protected String getTestDescription() {
         return "A menu containing items with embedded html. Items should chould either render the html or show it as plain text depending on the setting.";
     }
 
index c8814320f00c236a694ebaaffdb3338435c73dec..b7f2206979c550eb4eda9b95e1337ecc0bc27313 100644 (file)
@@ -14,12 +14,12 @@ public class MenuBarNavigation extends AbstractTestUIWithLog
     private MenuItem export;
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "Test case for mouse and keyboard navigation in MenuBar";
     }
 
     @Override
-    public Integer getTicketNumber() {
+    protected Integer getTicketNumber() {
         return 5174;
     }
 
index 99c84793fc4937c339daa134aa05858e2079c21b..8b47bc20dea029330ad7c483b43d85b21ce53bfa 100644 (file)
@@ -301,7 +301,7 @@ public class MenuBarTest extends AbstractComponentTest<MenuBar> {
 
     private void updateIcons(MenuItem item, int idx) {
         if (iconInterval > 0 && idx % iconInterval == 0) {
-            item.setIcon(getIcon());
+            item.setIcon(getMenuIcon());
         } else {
             item.setIcon(null);
         }
@@ -315,7 +315,7 @@ public class MenuBarTest extends AbstractComponentTest<MenuBar> {
 
     private long iconCacheIndex = new Date().getTime();
 
-    private Resource getIcon() {
+    private Resource getMenuIcon() {
         String resourceID = null;
         if (iconSize == 16) {
             resourceID = "../runo/icons/16/user.png";
index 8efafa3afab2ba5f338592a3510585fcb60077d4..716fef65b332d42e6320d099a9f6e1776f1d65d1 100644 (file)
@@ -48,7 +48,7 @@ public class MenuItemStyleRemoved extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "MenuItem's custom class names removed when hovering";
     }
 }
index 90dba78ec04b83e2183cfe1eeb0710d1f1e55bde..dd3c5cecb8caf0834f76b802feef49098cca8335 100644 (file)
@@ -72,7 +72,7 @@ public class Menubars extends ComponentTestCase<MenuBar> {
     }
 
     @Override
-    protected String getDescription() {
+    protected String getTestDescription() {
         return "A generic test for MenuBars in different configurations";
     }
 
index df1e0048be949f184793fe58c62e63e0199a9912..aa682887533aec6a4d874e2537d452832abb64e5 100644 (file)
@@ -26,7 +26,7 @@ public class ChromeBottomNotification extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "Bottom notification on Chrome goes up to top";
     }
 }
index 2cb400670e05ab1b54dfd736c39ae09b9ed809ae..c82310f2fa2340e456594be74b2ec495707ee2d1 100644 (file)
@@ -87,7 +87,7 @@ public class DisabledOptionGroupItems extends ComponentTestCase<OptionGroup> {
     }
 
     @Override
-    protected String getDescription() {
+    protected String getTestDescription() {
         return "Test case for disabled items in an OptionGroup";
     }
 }
index c6c0ac3d90cfc6e146d2bba499bab4c9012671f2..f4b6b584aca62ff33d50805cddcbf02c39ddf074 100644 (file)
@@ -101,7 +101,7 @@ public class HtmlOptionGroupItems extends ComponentTestCase<OptionGroup> {
     }
 
     @Override
-    protected String getDescription() {
+    protected String getTestDescription() {
         return "Test case for html items in an OptionGroup";
     }
 }
index 8c77cfd46556ab8117ac0909cc2da9005dbd7508..a207d31983845d5aae01fad943a4aa61132ca62f 100644 (file)
@@ -99,7 +99,7 @@ public class NativeSelects extends ComponentTestCase<NativeSelect> {
     }
 
     @Override
-    protected String getDescription() {
+    protected String getTestDescription() {
         return "A generic test for Labels in different configurations";
     }
 
index 34cf62c055b58c8d94a3b98f22498e5db2258498..9ab631aa225f827332552062c761dda2934b563c 100644 (file)
@@ -55,7 +55,7 @@ public class ContextMenuSize extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "If context menu original position doesn't allow to show it then "
                 + "its bottom should be aligned with the window bottom and height "
                 + "should be reset after repositioning.";
index e4022b08978d2c0c6482fa150cd506fc6caa4f5e..0686615664b0f2b2b3664ffe1801da5cb4067995 100644 (file)
@@ -100,7 +100,7 @@ public class Footer extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "Table with footer";
     }
 
index fbd632dc8527e71184455dcd36d635c43a00807b..aa80d15af88187e6b5eef2ba27a1e97de10de4fe 100644 (file)
@@ -62,7 +62,7 @@ public class FooterClick extends AbstractTestUIWithLog {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "Tests the footer click handler";
     }
 
index db79287433ca7a451d28a3206fc11544cce3f8e3..3e0a695b31ee74b704d97ae4fa296ef26d738d4c 100644 (file)
@@ -60,7 +60,7 @@ public class HeaderClick extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "Tests the header click listener";
     }
 
index 84ff51ae99dfab6c161a45b84662e83ee30d08e0..8263b481eb786ccb3c15aa21f0c503fc5a4f35ed 100644 (file)
@@ -118,7 +118,7 @@ public class HeaderFooterClickLeftRightMiddle extends AbstractTestUIWithLog {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "Tests the header click listener";
     }
 
index 6faa259d5a3be5c564c0775f00bb3a987ccbbedd..9f060ddf85320135ee7f98cf253cad816a118778 100644 (file)
@@ -111,8 +111,10 @@ public class HeaderPositionWhenSorting extends AbstractTestUI
     }
 
     @Override
-    public String getDescription() {
-        return "Table should only prevent the browser context menu when the right click is used for some Table specific operation. In practice these are either action handlers/context menu or item click listeners (right click). Note that item click listeners affects rows only, not the body.";
+    protected String getTestDescription() {
+        return "Table should only prevent the browser context menu when the right click is used for some Table specific operation. "
+                + "In practice these are either action handlers/context menu or item click listeners (right click). "
+                + "Note that item click listeners affects rows only, not the body.";
     }
 
     @Override
index 3d14cdcdae0517695f1a9533de1f96ee61bae0bc..1d4a620ff39dc7f49a628e2c7bf0bc485e7516df 100644 (file)
@@ -13,8 +13,8 @@ public class HeaderSyncOnScroll extends AbstractTestUI {
     private Table table;
 
     @Override
-    public String getDescription() {
-        return "Header's should be in sync when scrolling";
+    protected String getTestDescription() {
+        return "Headers should be in sync when scrolling";
     }
 
     @Override
index a612c678271a60a5f00877b15be1d739717ec364..ba88cc93a933b6445ea5eb3dd81eab2475095dcd 100644 (file)
@@ -35,7 +35,7 @@ public class HeaderUpdateWhenNoRows extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "The header should be updated when toggling column header mode";
     }
 
index 3d9e9a25d6e31f9be7a6f6ec32cb244f25dbf654..af850774ce4d780a6ee1ab1e79c4acc9db64a29d 100644 (file)
@@ -111,7 +111,7 @@ public class TableSetUndefinedSize extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "Table width and height changing to undefined doesn't update table size";
     }
 
index 53c7684ea11300b916e6e92d0f59535623bcc6b3..ad25d2b78dc5544f05ca1831c0e76eca2a690847 100644 (file)
@@ -14,7 +14,7 @@ public class TableVisibleColumnsUpdate extends AbstractTestUI {
     private Table table;
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "Columns should change between p1,p2,p3 and p1,p4,p3";
     }
 
index c58edc38a84c64ac9eee365bb2eca535d72de12e..c3817083a20c1ae5524f6b6361caa221bf182c43 100644 (file)
@@ -129,7 +129,7 @@ public class FirstTabNotVisibleWhenTabsheetNotClipped extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "TabSheet should display re-shown tab if there's room for it";
     }
 }
index 9025d87a2fe70d5a59312a4c7dd95e22cbdd757b..850bad42493c0a88592b9da4f4ee80ddfbf25ef9 100644 (file)
@@ -43,7 +43,7 @@ public class VetoTabChange extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "Tests the behavior when there's a listener that always changes back to the first tab.";
     }
 
index dcf2ac278472f62d39ddb8a857a51039ad38276b..13be9ad53d5484001559aceb5f793d57d90c235e 100644 (file)
@@ -29,7 +29,7 @@ public class DisabledUploadButton extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "Upload button should be disabled when upload "
                 + "is set to readonly and/or disabled";
     }
index 5ac15a7814b396cfd1e132882b2f254e555067ed..4cd28936b5c89c30b8264a2403fcdbc0528505d3 100644 (file)
@@ -8,6 +8,7 @@ import java.util.List;
 import com.vaadin.tests.components.ComponentTestCase;
 import com.vaadin.ui.Component;
 import com.vaadin.ui.FormLayout;
+import com.vaadin.ui.Notification;
 import com.vaadin.ui.Upload;
 import com.vaadin.ui.Upload.Receiver;
 
@@ -15,7 +16,7 @@ public class TestImmediateUploadInFormLayout
         extends ComponentTestCase<FormLayout> implements Receiver {
 
     @Override
-    protected String getDescription() {
+    protected String getTestDescription() {
         return "On Firefox 3.5 and Opera 10.10, clicking on an immediate upload in a wide FormLayout has no effect";
     }
 
@@ -48,7 +49,7 @@ public class TestImmediateUploadInFormLayout
 
     @Override
     public OutputStream receiveUpload(String filename, String MIMEType) {
-        getMainWindow().showNotification("Receiving upload");
+        Notification.show("Receiving upload");
         return new ByteArrayOutputStream();
     }
 }
index cac26c229748afc7af3496e95eb12873a93e1ddd..519296d0068a77fb01126d716599b21e080a56fa 100644 (file)
@@ -7,6 +7,7 @@ import java.util.List;
 
 import com.vaadin.tests.components.ComponentTestCase;
 import com.vaadin.ui.Component;
+import com.vaadin.ui.Notification;
 import com.vaadin.ui.Upload;
 import com.vaadin.ui.Upload.FinishedEvent;
 import com.vaadin.ui.Upload.Receiver;
@@ -31,10 +32,10 @@ public class TestUpload extends ComponentTestCase<Upload> implements Receiver {
         u.setSizeUndefined();
         addTestComponent(u);
 
-        u.addListener(new Upload.FinishedListener() {
+        u.addFinishedListener(new Upload.FinishedListener() {
             @Override
             public void uploadFinished(FinishedEvent event) {
-                getMainWindow().showNotification("Done");
+                Notification.show("Done");
             }
         });
 
@@ -54,7 +55,7 @@ public class TestUpload extends ComponentTestCase<Upload> implements Receiver {
 
     @Override
     public OutputStream receiveUpload(String filename, String MIMEType) {
-        getMainWindow().showNotification("Receiving upload");
+        Notification.show("Receiving upload");
         return new ByteArrayOutputStream();
     }
 
index c117f179f785064651b65cc7a154db0eebe61452..299195d266eaf5dc52ef49f81cb3e34171b3cdaa 100644 (file)
@@ -8,6 +8,7 @@ import java.util.List;
 import com.vaadin.tests.components.ComponentTestCase;
 import com.vaadin.ui.Component;
 import com.vaadin.ui.Label;
+import com.vaadin.ui.Notification;
 import com.vaadin.ui.Upload;
 import com.vaadin.ui.Upload.FinishedEvent;
 import com.vaadin.ui.Upload.Receiver;
@@ -17,7 +18,7 @@ public class TestUploadAndDisableOnSuccess extends ComponentTestCase<Upload>
         implements Receiver {
 
     @Override
-    protected String getDescription() {
+    protected String getTestDescription() {
         return "If upload is detached and attached during upload, the client side componenent never receives information that the upload has finished. Second update will not be successful.";
     }
 
@@ -49,7 +50,7 @@ public class TestUploadAndDisableOnSuccess extends ComponentTestCase<Upload>
         l = new Label(getUploadcount());
         addComponent(l);
 
-        u.addListener(new Upload.StartedListener() {
+        u.addStartedListener(new Upload.StartedListener() {
 
             @Override
             public void uploadStarted(StartedEvent event) {
@@ -63,10 +64,10 @@ public class TestUploadAndDisableOnSuccess extends ComponentTestCase<Upload>
             }
         });
 
-        u.addListener(new Upload.FinishedListener() {
+        u.addFinishedListener(new Upload.FinishedListener() {
             @Override
             public void uploadFinished(FinishedEvent event) {
-                getMainWindow().showNotification("Done");
+                Notification.show("Done");
                 l.setValue(getUploadcount());
             }
         });
index 2639e257f562de611fd3d78441746c4a0331e41e..ac8ca19e25f1ad5a61f4e92c44c2ab55bf2575ca 100644 (file)
@@ -12,12 +12,12 @@ public class WindowTest extends PanelTest<Window> {
     }
 
     @Override
-    protected void addTestComponent(Window c) {
+    protected void addTestComponent(Window w) {
         VerticalLayout layout = new VerticalLayout();
         layout.setMargin(true);
-        c.setContent(layout);
-        getMainWindow().addWindow(c);
-        getTestComponents().add(c);
+        w.setContent(layout);
+        addWindow(w);
+        getTestComponents().add(w);
     }
 
 }
index 0688eb431eb4f31a8d305b9deebb71c4acda955d..99f12013e7db9ca9f33d696be77f25e040e565cb 100644 (file)
@@ -40,7 +40,7 @@ public class PushVersionInfo extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "Debug window shows Push version in info Tab.";
     }
 
index 1311e02b424fad443d7921ad9896d5ee174aec75..745904ebe2a9afa149e34fd8b6beb7d9934a034e 100644 (file)
@@ -41,7 +41,7 @@ public class IframeIsOpenedInNonIOS extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "IFrame with a file is not shown in iOS";
     }
 }
index 1c1e2efcef50526604899bca3ac377a6900502e6..503e5bd4eb8d60b47138aadb92df860fabc239ca 100644 (file)
@@ -190,7 +190,7 @@ public class BasicPersonForm extends AbstractTestUIWithLog {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "Basic Person Form";
     }
 
index 39a8d8915305920568b91441e537618f310570a0..964a87a61431b20ff5ffa4c39d2ba0aa0299dd6b 100644 (file)
@@ -139,7 +139,7 @@ public class DateForm extends AbstractTestUIWithLog {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "Ensure FieldGroupFieldFactory supports Dates";
     }
 
index 152dc8f8575771fd8da7775b0d722ff599a5cea4..f670b2a6019bfcf0f53ce7f0ca837db8a240f660 100644 (file)
@@ -37,8 +37,7 @@ public class ThemeChangeFavicon extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "UI for testing that the favicon changes when changing themes";
     }
-
 }
index dd144bdc58e45545a0ab922d6b461d506bcbc87b..3cd4ab460190fde8099c06613ba6560448b3bff7 100644 (file)
@@ -56,7 +56,7 @@ public class GridCheckBoxDisplay extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "Verify that checkbox state is correct for all items in editor";
     }
 
index c0fbc82af0d19a3ed0be0ce721691b1c2fefb60a..ee345773cc246e0cecfcd80cd0fd39d72ebcfa66 100644 (file)
@@ -84,7 +84,7 @@ public class GridDetailsWidth extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "Tests that Escalator will not set explicit widths to the TD element in a details row.";
     }
 
index 047f1d7d84584d3edd770ba777e691416e826311..5dad03339f62a2c21184a0d7204af8ad5136abde 100644 (file)
@@ -29,7 +29,7 @@ public class GridEditorMultiselect extends AbstractTestUI {
     }
 
     @Override
-    public String getDescription() {
+    protected String getTestDescription() {
         return "Grid Multiselect: Edit mode allows invalid selection";
     }
 }
index ca94035edcf670aa5d113c4a31bc40652123770e..20fad06c1f6b9a852e8e3164bad7a01af1b08ace 100644 (file)
@@ -56,9 +56,4 @@ public class GridResizeHiddenColumn extends AbstractTestUIWithLog {
     protected Integer getTicketNumber() {
         return 19826;
     }
-
-    @Override
-    public String getDescription() {
-        return "Tests resize when columns with undefined width (-1) are hidden";
-    }
 }
index 99d2ced7a7114c450632016c2bfb4aa71298947b..b96257b1e0abd42bc77ec07b3e880a550301cd3c 100644 (file)
@@ -26,7 +26,7 @@
         <servlet-class>com.vaadin.server.LegacyVaadinServlet</servlet-class>
         <init-param>
             <param-name>application</param-name>
-            <param-value>com.vaadin.tests.components.button.Buttons</param-value>
+            <param-value>com.vaadin.tests.components.button.ButtonHtml</param-value>
         </init-param>
         <async-supported>true</async-supported>
     </servlet>
index 9ee118bae2b07b8edac0fbec320786d62924abf2..ac113a80483167e8a05ce318d8d9dbd16434ea03 100644 (file)
@@ -16,13 +16,13 @@ public class MultipleServletConfigurationTest extends MultiBrowserTest {
     public void testMultipleServletConfiguration() throws Exception {
         getDriver().get(getBaseURL() + "/embed1");
         assertLabelText(
-                "A generic test for Buttons in different configurations");
+                "Verify that Button HTML rendering works");
         getDriver().get(getBaseURL() + "/embed2");
         assertLabelText(
                 "Margins inside labels should not be allowed to collapse out of the label as it causes problems with layotus measuring the label.");
         getDriver().get(getBaseURL() + "/embed1");
         assertLabelText(
-                "A generic test for Buttons in different configurations");
+                "Verify that Button HTML rendering works");
     }
 
     private void assertLabelText(String expected) {