test: add regression test for #32

Test that ValueChangeEvent fires when adding or removing an additional (either programatically or from the browser).

Author: javier-godoy

src/test/java/com/flowingcode/vaadin/addons/chipfield/integration/IntegrationView.java

@@ -21,6 +21,7 @@
 
 import java.lang.reflect.Method;
 import java.util.Arrays;
+import java.util.List;
 import java.util.Optional;
 
 import com.flowingcode.vaadin.addons.chipfield.ChipField;
@@ -35,6 +36,7 @@
 public class IntegrationView extends Div implements IntegrationViewCallables {
 
 	public ChipField<String> field;
+	private List<String> lastValueChange;
 
 	public IntegrationView() {
 		setId("view");
@@ -111,4 +113,18 @@ public boolean hasItemWithLabel(String label) {
 		}
 	}
 
-}
+	@Override
+	@ClientCallable
+	public void addValueChangeListener() {
+		field.addValueChangeListener(ev -> {
+			lastValueChange = ev.getValue();
+		});
+	}
+
+	@Override
+	@ClientCallable
+	public JsonArrayList<String> getLastValueChange() {
+		return JsonArrayList.fromStringArray(lastValueChange);
+	}
+
+}

src/test/java/com/flowingcode/vaadin/addons/chipfield/integration/IntegrationViewCallables.java

@@ -25,6 +25,10 @@ public interface IntegrationViewCallables {
 
 	void testCallable(boolean arg);
 
+	void addValueChangeListener();
+
+	JsonArrayList<String> getLastValueChange();
+
 	void allowAdditionalItems(boolean value);
 
 	void setFieldReadOnly(boolean value);

src/test/java/com/flowingcode/vaadin/addons/chipfield/integration/ViewIT.java

@@ -186,9 +186,44 @@ public void testRemoveSelectedItem() {
 	}
 
 	/**
-	 * Test that findItemByLabel returns the additional chip
+	 * Test that valueChangeListener detects when adding a or removing a chip programatically.
 	 *
-	 * @see https://github.com/FlowingCode/ChipFieldAddon/issues/35
+	 * @see https://github.com/FlowingCode/ChipFieldAddon/issues/32
+	 */
+	@Test
+	public void testValueChangeEventOnSetValue() {
+		$server.addValueChangeListener();
+		$server.allowAdditionalItems(true);
+		$server.useNewItemHandler(true);
+		$server.setValue(LOREM, ADDITIONAL);
+		assertThat($server.getLastValueChange(), isEqualTo(LOREM, ADDITIONAL));
+		$server.setValue(LOREM);
+		assertThat($server.getLastValueChange(), isEqualTo(LOREM));
+	}
+
+	/**
+	 * Test that valueChangeListener catch the event when adding or removing an additional chip from the UI.
+	 *
+	 * @throws InterruptedException
+	 *
+	 * @see https://github.com/FlowingCode/ChipFieldAddon/issues/32
+	 */
+	@Test
+	public void testValueChangeEventOnAdditionalItem() throws InterruptedException {
+		$server.allowAdditionalItems(true);
+		$server.useNewItemHandler(true);
+		$server.addValueChangeListener();
+		$server.setValue(LOREM);
+		chipfield.sendKeys(ADDITIONAL, Keys.ENTER);
+		assertThat($server.getLastValueChange(), isEqualTo(LOREM, ADDITIONAL));
+		chipfield.sendKeys(Keys.BACK_SPACE);
+		assertThat($server.getLastValueChange(), isEqualTo(LOREM));
+	}
+
+	/**
+	 * Test that findItemByLabel returns the additional chips https://github.com/FlowingCode/ChipFieldAddon/issues/35
+	 *
+	 * @see https://github.com/FlowingCode/ChipFieldAddon/issues/36
 	 */
 	@Test
 	public void testFindItemByLabel() {