#5692 Generics and warnings cleanup: Ticket tests (with slight modifications from...

svn changeset:15385/svn branch:6.5

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket1362Login.java b/tests/src/com/vaadin/tests/tickets/Ticket1362Login.java
index 6d20b7c1cd606256457f2e1bfbedf5baa9b8124d..5541b7259ff95834e9d47f8c08c9f4162766a265 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket1362Login.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket1362Login.java
@@ -89,23 +89,22 @@ public class Ticket1362Login extends Application {
 
         private ParameterHandler paramHandler = new ParameterHandler() {
 
-            public void handleParameters(Map parameters) {
+            public void handleParameters(Map<String, String[]> parameters) {
                 if (parameters.containsKey("username")) {
                     getWindow().addURIHandler(uriHandler);
 
-                    HashMap params = new HashMap();
+                    HashMap<String, String> params = new HashMap<String, String>();
                     // expecting single params
-                    for (Iterator it = parameters.keySet().iterator(); it
+                    for (Iterator<String> it = parameters.keySet().iterator(); it
                             .hasNext();) {
-                        String key = (String) it.next();
-                        String value = ((String[]) parameters.get(key))[0];
+                        String key = it.next();
+                        String value = (parameters.get(key))[0];
                         params.put(key, value);
                     }
                     LoginEvent event = new LoginEvent(params);
-                    for (Iterator iterator = listeners.iterator(); iterator
-                            .hasNext();) {
-                        LoginListener listener = (LoginListener) iterator
-                                .next();
+                    for (Iterator<LoginListener> iterator = listeners
+                            .iterator(); iterator.hasNext();) {
+                        LoginListener listener = iterator.next();
                         listener.onLogin(event);
                     }
                 }
@@ -125,7 +124,7 @@ public class Ticket1362Login extends Application {
             }
         };
 
-        private LinkedList listeners = new LinkedList();
+        private LinkedList<LoginListener> listeners = new LinkedList<LoginListener>();
 
         private Window window;
 
@@ -159,9 +158,9 @@ public class Ticket1362Login extends Application {
          */
         public class LoginEvent {
 
-            private Map params;
+            private Map<String, String> params;
 
-            private LoginEvent(Map params) {
+            private LoginEvent(Map<String, String> params) {
                 this.params = params;
             }
 
@@ -173,7 +172,7 @@ public class Ticket1362Login extends Application {
              */
             public String getLoginParameter(String name) {
                 if (params.containsKey(name)) {
-                    return (String) params.get(name);
+                    return params.get(name);
                 } else {
                     return null;
                 }

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket1506_TestContainer.java b/tests/src/com/vaadin/tests/tickets/Ticket1506_TestContainer.java
index 071a5130b8020191051875a82fde6433eafba6e3..b8b6cfe5ddb4ed40814161336aa8a1df319f0b93 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket1506_TestContainer.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket1506_TestContainer.java
@@ -16,7 +16,7 @@ import com.vaadin.data.util.PropertysetItem;
  * @version $Revision$, $Date$\r
  */\r
 public class Ticket1506_TestContainer implements Container {\r
-    private Map items = new HashMap();\r
+    private Map<String, PropertysetItem> items = new HashMap<String, PropertysetItem>();\r
     public static final String ITEM_1_ID = "1";\r
     public static final String ITEM_2_ID = "2";\r
     public static final String PROPERTY_1_ID = "property 1";\r
@@ -42,24 +42,24 @@ public class Ticket1506_TestContainer implements Container {
         if (items.isEmpty()) {\r
             loadItems();\r
         }\r
-        return (Item) items.get(itemId);\r
+        return items.get(itemId);\r
     }\r
 \r
-    public Collection getContainerPropertyIds() {\r
+    public Collection<String> getContainerPropertyIds() {\r
         if (items.isEmpty()) {\r
             loadItems();\r
         }\r
-        ArrayList a = new ArrayList();\r
+        ArrayList<String> a = new ArrayList<String>();\r
         a.add(PROPERTY_1_ID);\r
         a.add(PROPERTY_2_ID);\r
         return a;\r
     }\r
 \r
-    public Collection getItemIds() {\r
+    public Collection<String> getItemIds() {\r
         if (items.isEmpty()) {\r
             loadItems();\r
         }\r
-        ArrayList a = new ArrayList();\r
+        ArrayList<String> a = new ArrayList<String>();\r
         a.add(ITEM_1_ID);\r
         a.add(ITEM_2_ID);\r
         return a;\r
@@ -69,14 +69,14 @@ public class Ticket1506_TestContainer implements Container {
         if (items.isEmpty()) {\r
             loadItems();\r
         }\r
-        Item item = (Item) items.get(itemId);\r
+        Item item = items.get(itemId);\r
         if (item != null) {\r
             return item.getItemProperty(propertyId);\r
         }\r
         return null;\r
     }\r
 \r
-    public Class getType(Object propertyId) {\r
+    public Class<String> getType(Object propertyId) {\r
         if (items.isEmpty()) {\r
             loadItems();\r
         }\r

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket1506_TestContainer2.java b/tests/src/com/vaadin/tests/tickets/Ticket1506_TestContainer2.java
index 4a4c846ed0ea5061de9fcd4b2b71b43b2efbede8..c39454027ac472d495b735c40c1b2d5ad5edc138 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket1506_TestContainer2.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket1506_TestContainer2.java
@@ -16,7 +16,7 @@ import com.vaadin.data.util.PropertysetItem;
  * @version $Revision$, $Date$\r
  */\r
 public class Ticket1506_TestContainer2 implements Container {\r
-    private Map items = new HashMap();\r
+    private Map<String, PropertysetItem> items = new HashMap<String, PropertysetItem>();\r
     public static final String ITEM_1_ID = "1";\r
     public static final String ITEM_2_ID = "2";\r
     public static final String PROPERTY_1_ID = "property 1";\r
@@ -37,20 +37,20 @@ public class Ticket1506_TestContainer2 implements Container {
         if (items.isEmpty()) {\r
             loadItems();\r
         }\r
-        return (Item) items.get(itemId);\r
+        return items.get(itemId);\r
     }\r
 \r
-    public Collection getContainerPropertyIds() {\r
+    public Collection<String> getContainerPropertyIds() {\r
         if (items.isEmpty()) {\r
             loadItems();\r
         }\r
-        ArrayList a = new ArrayList();\r
+        ArrayList<String> a = new ArrayList<String>();\r
         a.add(PROPERTY_1_ID);\r
         a.add(PROPERTY_2_ID);\r
         return a;\r
     }\r
 \r
-    public Collection getItemIds() {\r
+    public Collection<String> getItemIds() {\r
         if (items.isEmpty()) {\r
             loadItems();\r
         }\r
@@ -61,14 +61,14 @@ public class Ticket1506_TestContainer2 implements Container {
         if (items.isEmpty()) {\r
             loadItems();\r
         }\r
-        Item item = (Item) items.get(itemId);\r
+        Item item = items.get(itemId);\r
         if (item != null) {\r
             return item.getItemProperty(propertyId);\r
         }\r
         return null;\r
     }\r
 \r
-    public Class getType(Object propertyId) {\r
+    public Class<String> getType(Object propertyId) {\r
         if (items.isEmpty()) {\r
             loadItems();\r
         }\r

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket1710.java b/tests/src/com/vaadin/tests/tickets/Ticket1710.java
index 76e065a395d24c8cd7d9a5d554e59af5118fd3c4..f0d0476d0815fc7651124e15cb4b81b2f4740177 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket1710.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket1710.java
@@ -1,7 +1,6 @@
 package com.vaadin.tests.tickets;\r
 \r
 import java.util.Iterator;\r
-import java.util.LinkedList;\r
 \r
 import com.vaadin.data.Property;\r
 import com.vaadin.data.Property.ValueChangeEvent;\r
@@ -28,8 +27,7 @@ import com.vaadin.ui.Window;
 \r
 public class Ticket1710 extends com.vaadin.Application {\r
 \r
-    LinkedList listOfAllFields = new LinkedList();\r
-\r
+    @Override\r
     public void init() {\r
 \r
         setTheme("tests-tickets");\r
@@ -92,7 +90,7 @@ public class Ticket1710 extends com.vaadin.Application {
         elp.setVisible(false);\r
         el.setHeight(700);\r
         addFields(el);\r
-        Component firstComponent = (Component) el.getComponentIterator().next();\r
+        Component firstComponent = el.getComponentIterator().next();\r
         firstComponent.setSizeFull();\r
         el.expand(firstComponent);\r
         lo.addComponent(elp);\r
@@ -106,8 +104,7 @@ public class Ticket1710 extends com.vaadin.Application {
         elh.setWidth(2000);\r
         elh.setHeight(100);\r
         addFields(elh);\r
-        Component firstComponentElh = (Component) elh.getComponentIterator()\r
-                .next();\r
+        Component firstComponentElh = elh.getComponentIterator().next();\r
         firstComponentElh.setSizeFull();\r
         elh.expand(firstComponentElh);\r
         lo.addComponent(elhp);\r
@@ -129,7 +126,8 @@ public class Ticket1710 extends com.vaadin.Application {
         formPanel.addComponent(getFormPanelExample());\r
         lo.addComponent(formPanel);\r
 \r
-        for (Iterator i = hidingControls.getComponentIterator(); i.hasNext();) {\r
+        for (Iterator<Component> i = hidingControls.getComponentIterator(); i\r
+                .hasNext();) {\r
             ((AbstractComponent) i.next()).setImmediate(true);\r
         }\r
 \r
@@ -304,7 +302,8 @@ public class Ticket1710 extends com.vaadin.Application {
             testedLayout.setStyleName("tested-layout");\r
             setStyleName("layout-testing-panel");\r
 \r
-            for (Iterator i = controls.getComponentIterator(); i.hasNext();) {\r
+            for (Iterator<Component> i = controls.getComponentIterator(); i\r
+                    .hasNext();) {\r
                 ((AbstractComponent) i.next()).setImmediate(true);\r
             }\r
         }\r
@@ -387,9 +386,10 @@ public class Ticket1710 extends com.vaadin.Application {
         }\r
 \r
         private void updateAlignments(int h, int v) {\r
-            for (Iterator i = testedLayout.getComponentIterator(); i.hasNext();) {\r
+            for (Iterator<Component> i = testedLayout.getComponentIterator(); i\r
+                    .hasNext();) {\r
                 ((Layout.AlignmentHandler) testedLayout).setComponentAlignment(\r
-                        (Component) i.next(), h, v);\r
+                        i.next(), h, v);\r
             }\r
         }\r
 \r

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket1804.java b/tests/src/com/vaadin/tests/tickets/Ticket1804.java
index 92aa8587f44292aa4d2d1af52489cab74c3504fa..8d8740895de4542b956852f5e7570eb5c5f2e0bf 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket1804.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket1804.java
@@ -16,7 +16,7 @@ import com.vaadin.ui.Window;
 \r
 public class Ticket1804 extends com.vaadin.Application {\r
 \r
-    LinkedList listOfAllFields = new LinkedList();\r
+    LinkedList<Select> listOfAllFields = new LinkedList<Select>();\r
 \r
     @Override\r
     public void init() {\r
@@ -73,8 +73,8 @@ public class Ticket1804 extends com.vaadin.Application {
         s.setComponentError(new SystemError("Test error message"));\r
         listOfAllFields.add(s);\r
 \r
-        for (Iterator i = listOfAllFields.iterator(); i.hasNext();) {\r
-            s = (Select) i.next();\r
+        for (Iterator<Select> i = listOfAllFields.iterator(); i.hasNext();) {\r
+            s = i.next();\r
             main.addComponent(s);\r
             s.addItem("foo");\r
             s.addItem("");\r
@@ -92,9 +92,9 @@ public class Ticket1804 extends com.vaadin.Application {
 \r
             public void buttonClick(ClickEvent event) {\r
                 StringBuffer msg = new StringBuffer();\r
-                for (java.util.Iterator i = listOfAllFields.iterator(); i\r
+                for (Iterator<Select> i = listOfAllFields.iterator(); i\r
                         .hasNext();) {\r
-                    AbstractField af = (AbstractField) i.next();\r
+                    AbstractField af = i.next();\r
                     msg.append("<h1>" + af.getCaption() + "</h1>\n");\r
                     msg.append("Value=" + af.getValue() + "<br/>\n");\r
                     if (af.isValid()) {\r

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket1811.java b/tests/src/com/vaadin/tests/tickets/Ticket1811.java
index 35c3992bf0b1ce76977c61bb9c1716b87764d05b..307db4f47a12751e8d5e6a9b50d11eea3b2f95b2 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket1811.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket1811.java
@@ -13,7 +13,7 @@ import com.vaadin.ui.Window;
 \r
 public class Ticket1811 extends com.vaadin.Application {\r
 \r
-    LinkedList listOfAllFields = new LinkedList();\r
+    LinkedList<TextField> listOfAllFields = new LinkedList<TextField>();\r
 \r
     @Override\r
     public void init() {\r
@@ -44,8 +44,8 @@ public class Ticket1811 extends com.vaadin.Application {
         tf4.addValidator(strLenValidator);\r
         listOfAllFields.add(tf4);\r
 \r
-        for (Iterator i = listOfAllFields.iterator(); i.hasNext();) {\r
-            TextField tf = (TextField) i.next();\r
+        for (Iterator<TextField> i = listOfAllFields.iterator(); i.hasNext();) {\r
+            TextField tf = i.next();\r
             main.addComponent(tf);\r
             tf.setImmediate(true);\r
         }\r
@@ -56,9 +56,9 @@ public class Ticket1811 extends com.vaadin.Application {
 \r
             public void buttonClick(ClickEvent event) {\r
                 StringBuffer msg = new StringBuffer();\r
-                for (java.util.Iterator i = listOfAllFields.iterator(); i\r
-                        .hasNext();) {\r
-                    TextField tf = (TextField) i.next();\r
+                for (Iterator<TextField> i = listOfAllFields\r
+                        .iterator(); i.hasNext();) {\r
+                    TextField tf = i.next();\r
                     msg.append("<h1>" + tf.getCaption() + "</h1>\n");\r
                     if (tf.isValid()) {\r
                         msg.append("VALID\n<hr/>");\r

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket1819.java b/tests/src/com/vaadin/tests/tickets/Ticket1819.java
index 836d6b4ae9ab796c7907c016837a697be30dc34b..6f06055fc4dd134451cb196bb9f657737e733a78 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket1819.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket1819.java
@@ -12,7 +12,7 @@ import com.vaadin.ui.Window;
 \r
 public class Ticket1819 extends com.vaadin.Application {\r
 \r
-    LinkedList listOfAllFields = new LinkedList();\r
+    LinkedList<Select> listOfAllFields = new LinkedList<Select>();\r
 \r
     @Override\r
     public void init() {\r
@@ -30,8 +30,8 @@ public class Ticket1819 extends com.vaadin.Application {
         s.setNullSelectionAllowed(false);\r
         listOfAllFields.add(s);\r
 \r
-        for (Iterator i = listOfAllFields.iterator(); i.hasNext();) {\r
-            s = (Select) i.next();\r
+        for (Iterator<Select> i = listOfAllFields.iterator(); i.hasNext();) {\r
+            s = i.next();\r
             main.addComponent(s);\r
             s.addItem("-null-");\r
             s.addItem("");\r
@@ -47,9 +47,9 @@ public class Ticket1819 extends com.vaadin.Application {
 \r
             public void buttonClick(ClickEvent event) {\r
                 StringBuffer msg = new StringBuffer();\r
-                for (java.util.Iterator i = listOfAllFields.iterator(); i\r
+                for (Iterator<Select> i = listOfAllFields.iterator(); i\r
                         .hasNext();) {\r
-                    AbstractField af = (AbstractField) i.next();\r
+                    AbstractField af = i.next();\r
                     msg.append("<h1>" + af.getCaption() + "</h1>\n");\r
                     msg.append("Value=" + af.getValue() + "<br/>\n");\r
                 }\r

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket1869.java b/tests/src/com/vaadin/tests/tickets/Ticket1869.java
index 6c101441a6847a7b965cc55a5104a1cb33b193b4..452006b201183c0b4bee75e60aa0e66692c446c8 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket1869.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket1869.java
@@ -10,8 +10,6 @@ import com.vaadin.ui.Window;
 \r
 public class Ticket1869 extends com.vaadin.Application {\r
 \r
-    LinkedList listOfAllFields = new LinkedList();\r
-\r
     @Override\r
     public void init() {\r
 \r

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket1878.java b/tests/src/com/vaadin/tests/tickets/Ticket1878.java
index 711bb533f8b91343c0497718e78b4915befccb28..8de24cbdcf06e18201b1f0f533a89d89c06b7d8e 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket1878.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket1878.java
@@ -13,6 +13,7 @@ import com.vaadin.terminal.UserError;
 import com.vaadin.ui.Button;\r
 import com.vaadin.ui.Button.ClickEvent;\r
 import com.vaadin.ui.Button.ClickListener;\r
+import com.vaadin.ui.Component;\r
 import com.vaadin.ui.ExpandLayout;\r
 import com.vaadin.ui.Field;\r
 import com.vaadin.ui.Form;\r
@@ -184,9 +185,9 @@ public class Ticket1878 extends Application {
         Random r = new Random();\r
         GridLayout l1 = new GridLayout(1, 3);\r
         form = createForm(l1, "200px", "500px");\r
-        BeanItem item = new BeanItem(new FormObject());\r
+        BeanItem<FormObject> item = new BeanItem<FormObject>(new FormObject());\r
         form.setItemDataSource(item);\r
-        for (Iterator i = item.getItemPropertyIds().iterator(); i.hasNext();) {\r
+        for (Iterator<?> i = item.getItemPropertyIds().iterator(); i.hasNext();) {\r
             Object property = i.next();\r
             Field f = form.getField(property);\r
 \r
@@ -247,7 +248,7 @@ public class Ticket1878 extends Application {
         switchToFormsButton.setEnabled(true);\r
 \r
         b.setEnabled(false);\r
-        java.util.Iterator i = mainLayout.getComponentIterator();\r
+        Iterator<Component> i = mainLayout.getComponentIterator();\r
         i.next();\r
         Layout l = (Layout) i.next();\r
 \r

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket1921.java b/tests/src/com/vaadin/tests/tickets/Ticket1921.java
index 0bb79f224a9905ddce618682dec994901fb52b4c..f9477f9df00b56a034c987b4434a51afbb4077ec 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket1921.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket1921.java
@@ -102,8 +102,8 @@ public class Ticket1921 extends Application implements ParameterHandler {
         }
     }
 
-    public void handleParameters(Map parameters) {
-        String[] s = (String[]) parameters.get("state");
+    public void handleParameters(Map<String, String[]> parameters) {
+        String[] s = parameters.get("state");
         if (s == null || s.length != 1) {
             return;
         }

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket1966.java b/tests/src/com/vaadin/tests/tickets/Ticket1966.java
index 2d0ec549923a69ca0a993f919185915457533e16..b89610c1f69ff90557fbc176331466e605eefd1c 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket1966.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket1966.java
@@ -21,7 +21,7 @@ public class Ticket1966 extends Application {
     private static final int VCENTER = OrderedLayout.ALIGNMENT_VERTICAL_CENTER;\r
     private static final int BOTTOM = OrderedLayout.ALIGNMENT_BOTTOM;\r
 \r
-    private static Map names = new HashMap();\r
+    private static Map<Integer, String> names = new HashMap<Integer, String>();\r
     static {\r
         names.put(new Integer(LEFT), "Left");\r
         names.put(new Integer(CENTER), "Center");\r

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket1966_2.java b/tests/src/com/vaadin/tests/tickets/Ticket1966_2.java
index e325922abca5b9ead6b6e58a03b81fa30bc758dc..444eb5fea7e0cf07a4f33fb078b1cd039a82c9f7 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket1966_2.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket1966_2.java
@@ -22,7 +22,7 @@ public class Ticket1966_2 extends Application {
     private static final int VCENTER = OrderedLayout.ALIGNMENT_VERTICAL_CENTER;\r
     private static final int BOTTOM = OrderedLayout.ALIGNMENT_BOTTOM;\r
 \r
-    private static Map names = new HashMap();\r
+    private static Map<Integer, String> names = new HashMap<Integer, String>();\r
     static {\r
         names.put(new Integer(LEFT), "Left");\r
         names.put(new Integer(CENTER), "Center");\r

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket1970.java b/tests/src/com/vaadin/tests/tickets/Ticket1970.java
index c39f28bb86e99b8995b02153c10083abeb08a3c5..1442c9ea5b6beb6cb9d612cd66ed7efa568eb56f 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket1970.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket1970.java
@@ -40,8 +40,9 @@ public class Ticket1970 extends Application {
 
                     public void buttonClick(ClickEvent event) {
                         String openWindows = "";
-                        for (Iterator i = getWindows().iterator(); i.hasNext();) {
-                            Window t = (Window) i.next();
+                        for (Iterator<Window> i = getWindows().iterator(); i
+                                .hasNext();) {
+                            Window t = i.next();
                             openWindows += (openWindows.length() > 0 ? "," : "")
                                     + t.getName();
                         }

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket1982.java b/tests/src/com/vaadin/tests/tickets/Ticket1982.java
index 8d0f22f6a3a39605c952737275970a935718e9cd..fe5f759428601a91860e9b2753272ffbec3b7a2b 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket1982.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket1982.java
@@ -14,7 +14,7 @@ import com.vaadin.ui.Window;
 
 public class Ticket1982 extends Application {
 
-    private List components = new ArrayList();
+    private List<TitleBar> components = new ArrayList<TitleBar>();
 
     @Override
     public void init() {
@@ -41,10 +41,10 @@ public class Ticket1982 extends Application {
 
     private void restoreComponents(GridLayout gl) {
         gl.removeAllComponents();
-        gl.addComponent((TitleBar) components.get(0));
-        gl.addComponent((TitleBar) components.get(1));
-        gl.addComponent((TitleBar) components.get(2));
-        gl.addComponent((TitleBar) components.get(3));
+        gl.addComponent(components.get(0));
+        gl.addComponent(components.get(1));
+        gl.addComponent(components.get(2));
+        gl.addComponent(components.get(3));
     }
 
     private class TitleBar extends ExpandLayout {

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket2181.java b/tests/src/com/vaadin/tests/tickets/Ticket2181.java
index 98bfdce04c159f7681b70563d2d252b81f60e31c..2754ad2541bfd4e297fd3d8a294537c199c21a71 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket2181.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket2181.java
@@ -76,6 +76,7 @@ public class Ticket2181 extends Application implements Button.ClickListener {
     }
 
     private void set() {
+        @SuppressWarnings("unchecked")
         Set<String> values = (Set<String>) options.getValue();
         TextField[] tfs = new TextField[] { tf1, tf2 };
         for (TextField tf : tfs) {

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket2204.java b/tests/src/com/vaadin/tests/tickets/Ticket2204.java
index 6401867289de7ee0c5586a4320c65db471d15278..0ba1b5e34633b89dfb83336bbb7fb032ffacccd0 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket2204.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket2204.java
@@ -58,7 +58,7 @@ public class Ticket2204 extends Application {
         ts = new TabSheet();
         layout.addComponent(ts);
 
-        for (Class c : classes) {
+        for (Class<? extends Component> c : classes) {
             ts.addTab(createComponent(c), c.getSimpleName(), null);
         }
         rta = new RichTextArea();
@@ -95,7 +95,7 @@ public class Ticket2204 extends Application {
         c.setVisible(!c.isVisible());
     }
 
-    private Component createComponent(Class c) {
+    private Component createComponent(Class<? extends Component> c) {
         RichTextArea textArea = new RichTextArea();
         textArea.setVisible(false);
         textArea.setCaption("This is the textArea");
@@ -105,7 +105,7 @@ public class Ticket2204 extends Application {
         Component cc = null;
 
         try {
-            cc = (Component) c.newInstance();
+            cc = c.newInstance();
         } catch (Exception e) {
             // TODO Auto-generated catch block
             e.printStackTrace();
@@ -124,7 +124,7 @@ public class Ticket2204 extends Application {
             Form f = (Form) cc;
             f.setFieldFactory(new FieldFactory() {
 
-                public Field createField(Class type, Component uiContext) {
+                public Field createField(Class<?> type, Component uiContext) {
                     return createField();
                 }
 
@@ -149,7 +149,7 @@ public class Ticket2204 extends Application {
                 }
 
             });
-            f.setItemDataSource(new BeanItem(new Object() {
+            f.setItemDataSource(new BeanItem<Object>(new Object() {
                 private int a;
 
                 public int getA() {

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket2244.java b/tests/src/com/vaadin/tests/tickets/Ticket2244.java
index 1330116d80b3dfb76586b4c10e97ee0994ac309b..e9ae3ac72020110737d193c7d958afd8513671c2 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket2244.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket2244.java
@@ -25,7 +25,7 @@ public class Ticket2244 extends Application {
         gl.newLine();
 
         form = new Form(gl);
-        form.setItemDataSource(new BeanItem(new MyBean()));
+        form.setItemDataSource(new BeanItem<MyBean>(new MyBean()));
 
         gl.addComponent(new Label("After form"));
 
@@ -34,7 +34,7 @@ public class Ticket2244 extends Application {
         w.addComponent(new Button("new item", new Button.ClickListener() {
 
             public void buttonClick(ClickEvent event) {
-                form.setItemDataSource(new BeanItem(new MyBean()));
+                form.setItemDataSource(new BeanItem<MyBean>(new MyBean()));
 
             }
 
@@ -43,7 +43,8 @@ public class Ticket2244 extends Application {
                 new Button.ClickListener() {
 
                     public void buttonClick(ClickEvent event) {
-                        form.setItemDataSource(new BeanItem(new MyBiggerBean()));
+                        form.setItemDataSource(new BeanItem<MyBean>(
+                                new MyBiggerBean()));
 
                     }
 

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket3146.java b/tests/src/com/vaadin/tests/tickets/Ticket3146.java
index 623b88f616ffbf8532b92e1b1a1562c83fbb62fc..6afa25efdabb7f703fe033232d178de5f044b6a1 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket3146.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket3146.java
@@ -76,7 +76,7 @@ public class Ticket3146 extends Application {
     }\r
 \r
     void clearSelection2() {\r
-        table.setValue(new HashSet());\r
+        table.setValue(new HashSet<Object>());\r
     }\r
 \r
     void clearSelection3() {\r
@@ -87,7 +87,7 @@ public class Ticket3146 extends Application {
 \r
     void printSelection() {\r
         String selection = "";\r
-        for (Object item : (Collection) table.getValue()) {\r
+        for (Object item : (Collection<?>) table.getValue()) {\r
             selection = selection + item + ' ';\r
         }\r
         result.setValue(selection);\r

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket677.java b/tests/src/com/vaadin/tests/tickets/Ticket677.java
index ca9c9227b27f446e2470354c3056eddbdda61b94..d551278659773f423252869a64bafa255b206421 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket677.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket677.java
@@ -120,7 +120,7 @@ public class Ticket677 extends Application {
             }\r
 \r
         });\r
-        form.setItemDataSource(new BeanItem(new MyBean()));\r
+        form.setItemDataSource(new BeanItem<MyBean>(new MyBean()));\r
         root.addComponent(form);\r
 \r
         table = new Table("Enabled");\r

diff --git a/tests/src/com/vaadin/tests/tickets/Ticket736.java b/tests/src/com/vaadin/tests/tickets/Ticket736.java
index 0530f4eb6656141bb0230330e802145598abb180..49cfb5d70814f5a1b603567ada2e665c854fa7cd 100644 (file)
--- a/tests/src/com/vaadin/tests/tickets/Ticket736.java
+++ b/tests/src/com/vaadin/tests/tickets/Ticket736.java
@@ -28,8 +28,8 @@ public class Ticket736 extends Application {
 
         // Create form for editing address
         final Form f = new Form();
-        f.setItemDataSource(new BeanItem(address, new String[] { "name",
-                "street", "zip", "city", "state", "country" }));
+        f.setItemDataSource(new BeanItem<Address>(address, new String[] {
+                "name", "street", "zip", "city", "state", "country" }));
         f.setCaption("Office address");
         f.setIcon(new ThemeResource("../runo/icons/16/document.png"));
         f.setDescription("Jep jpe, this is form description.");