Refactored failing test for OptionGroup.

Change-Id: Ia7cc646649dd3e98822ace82e1eefac3f2ba2a5e

server/tests/src/com/vaadin/tests/server/component/abstractselect/OptionGroupTests.java

@@ -0,0 +1,32 @@
+package com.vaadin.tests.server.component.abstractselect;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertTrue;
+
+import java.util.Collection;
+
+import org.junit.Before;
+import org.junit.Test;
+
+import com.vaadin.ui.OptionGroup;
+
+public class OptionGroupTests {
+
+    private OptionGroup optionGroup;
+
+    @Before
+    public void setup() {
+        optionGroup = new OptionGroup();
+    }
+
+    @Test
+    public void itemsAreAdded() {
+        optionGroup.addItems("foo", "bar");
+
+        Collection<?> itemIds = optionGroup.getItemIds();
+
+        assertEquals(2, itemIds.size());
+        assertTrue(itemIds.contains("foo"));
+        assertTrue(itemIds.contains("bar"));
+    }
+}

server/tests/src/com/vaadin/tests/server/component/abstractselect/TestVarargsItemAddition.java

@@ -1,26 +0,0 @@
-package com.vaadin.tests.server.component.abstractselect;
-
-import java.util.Collection;
-
-import junit.framework.TestCase;
-
-import org.junit.Assert;
-
-import com.vaadin.ui.OptionGroup;
-
-public class TestVarargsItemAddition extends TestCase {
-
-    public void itemAddition() throws Exception {
-
-        OptionGroup optionGroup = new OptionGroup();
-
-        optionGroup.addItems("foo", "bar", "car");
-
-        Collection<?> itemIds = optionGroup.getItemIds();
-        Assert.assertEquals(3, itemIds.size());
-        Assert.assertTrue(itemIds.contains("foo"));
-        Assert.assertTrue(itemIds.contains("bar"));
-        Assert.assertTrue(itemIds.contains("car"));
-
-    }
-}