Fixed and provided JUnit Test - MaterialComboBox function "public Option addItem(String text, T value)" return null always #228

Author: kevzlou7979

src/main/java/gwt/material/design/addins/client/combobox/MaterialComboBox.java

@@ -471,6 +471,7 @@ public Option addItem(String text, T value) {
             Option option = buildOption(text, value);
             values.add(value);
             listbox.add(option);
+            return option;
         }
         return null;
     }

src/test/java/gwt/material/design/addins/client/MaterialComboBoxTest.java

@@ -77,7 +77,8 @@ protected void checkEmptyOrNullValue(MaterialComboBox comboBox) {
     protected <T extends MaterialWidget & HasColors> void checkColor(T widget) {
         MaterialComboBox comboBox = new MaterialComboBox();
         for (int i = 1; i <= 5; i++) {
-            comboBox.addItem(String.valueOf(i), String.valueOf(i));
+            Option option = comboBox.addItem(String.valueOf(i), String.valueOf(i));
+            assertNotNull(option);
         }
         comboBox.setTextColor(Color.RED);
         RootPanel.get().add(comboBox);