Fixed #722, #723, #724

Author: kevzlou7979

gwt-material/src/main/java/gwt/material/design/client/ui/MaterialListValueBox.java

@@ -151,8 +151,10 @@ public void unload() {
 
     @Override
     public void reload() {
-        unload();
-        load();
+        if (isAttached()) {
+            unload();
+            load();
+        }
     }
 
     public void add(T value) {

gwt-material/src/test/java/gwt/material/design/client/ui/MaterialListBoxTest.java

@@ -41,7 +41,7 @@ protected MaterialListBox createWidget() {
     @Override
     public void testValues() {
         // given
-        MaterialListBox listBox = (MaterialListBox)getWidget();
+        MaterialListBox listBox = (MaterialListBox) getWidget();
 
         List<String> users = new ArrayList<>();
         for (int i = 1; i <= 5; i++) {
@@ -53,12 +53,12 @@ public void testValues() {
     }
 
     public void testChildren() {
+        // UiBinder
         // given
         MaterialListBox listBox = (MaterialListBox)getWidget();
 
         // when / then
         assertEquals(3, listBox.getChildren().size());
-
         assertTrue(listBox.getWidget(0) instanceof ListBox);
         assertTrue(listBox.getWidget(1) instanceof Label);
         assertTrue(listBox.getWidget(2) instanceof MaterialLabel);

gwt-material/src/test/java/gwt/material/design/client/ui/MaterialListValueBoxTest.java

@@ -100,26 +100,54 @@ public void testValues() {
     }
 
     public void testProperties() {
+        // UiBinder
         // given
-        MaterialListValueBox<T> listValueBox = getWidget();
+        MaterialListValueBox<T> listValueBox = getWidget(false);
 
+        // when / then
+        checkProperties(listValueBox);
+
+        attachWidget();
+
+        // Standard
+        // given
+        checkProperties(listValueBox);
+    }
+
+    protected void checkProperties(MaterialListValueBox listValueBox) {
         // when / then
         listValueBox.setMultipleSelect(true);
         assertTrue(listValueBox.isMultipleSelect());
     }
 
     public void testBrowserDefault() {
+        // UiBinder
         // given
-        MaterialListValueBox<T> listValueBox = getWidget();
+        MaterialListValueBox<T> listValueBox = getWidget(false);
+
+        // when / then
+        checkBrowserDefault(listValueBox, false);
+
+        // Standard
+        // given
+        attachWidget();
 
         // when / then
+        checkBrowserDefault(listValueBox, true);
+    }
+
+    protected void checkBrowserDefault(MaterialListValueBox listValueBox, boolean checkElement) {
         listValueBox.setOld(true);
         assertTrue(listValueBox.isOld());
-        assertTrue(listValueBox.getListBox().getElement().hasClassName("browser-default"));
+        if (checkElement) {
+            assertTrue(listValueBox.getListBox().getElement().hasClassName("browser-default"));
+        }
 
         listValueBox.setOld(false);
         assertFalse(listValueBox.isOld());
-        assertFalse(listValueBox.getListBox().getElement().hasClassName("browser-default"));
+        if (checkElement) {
+            assertFalse(listValueBox.getListBox().getElement().hasClassName("browser-default"));
+        }
     }
 
     public void testEnabled() {
@@ -154,8 +182,16 @@ public void testErrorSuccess() {
     }
 
     public void testPlaceholder() {
+        // UiBinder
         // given
-        MaterialListValueBox<T> valueBox = getWidget();
+        MaterialListValueBox<T> valueBox = getWidget(false);
+
+        // when / then
+        checkPlaceholder(valueBox);
+
+        // Standard
+        // given
+        attachWidget();
 
         // when / then
         checkPlaceholder(valueBox);