test: implement client callable interface

Author: javier-godoy

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

@@ -7,42 +7,68 @@
 import com.vaadin.flow.component.html.Div;
 import com.vaadin.flow.router.Route;
 
+import elemental.json.Json;
+
 @Route("it")
-public class IntegrationView extends Div {
+public class IntegrationView extends Div implements IntegrationViewCallables {
 
-	private ChipField<String> field;
+	public ChipField<String> field;
 
 	public IntegrationView() {
 		setId("view");
 		add(field = new ChipField<>("Field"));
 		field.setItems("Lorem", "Ipsum");
 	}
 
+	@Override
 	@ClientCallable
-	private void testCallable(boolean arg) {
+	public void testCallable(boolean arg) {
 		if (!arg) {
 			throw new IllegalArgumentException();
 		}
 	}
 
+	@Override
 	@ClientCallable
-	private void allowAdditionalItems(boolean value) {
+	public void allowAdditionalItems(boolean value) {
 		field.setAllowAdditionalItems(value);
 	}
 
+	@Override
 	@ClientCallable
-	private void setFieldReadOnly(boolean value) {
+	public void setFieldReadOnly(boolean value) {
 		field.setReadOnly(value);
 	}
 
+	@Override
 	@ClientCallable
-	private void setFieldEnabled(boolean value) {
+	public void setFieldEnabled(boolean value) {
 		field.setEnabled(value);
 	}
 
+	@Override
 	@ClientCallable
-	private void setValue(String... items) {
+	public void setValue(String... items) {
 		field.setValue(Arrays.asList(items));
 	}
 
+	@Override
+	@ClientCallable
+	public void useNewItemHandler(boolean useHandler) {
+		field.setNewItemHandler(useHandler ? Object::toString : null);
+	}
+
+	@ClientCallable
+	public void assertValue(String... items) {
+		if (!field.getValue().equals(Arrays.asList(items))) {
+			throw new AssertionError();
+		}
+	}
+
+	@Override
+	@ClientCallable
+	public JsonArrayList<String> getValue() {
+		return JsonArrayList.createArray(field.getValue(), Json::create);
+	}
+
 }

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

@@ -0,0 +1,19 @@
+package com.flowingcode.vaadin.addons.chipfield.integration;
+
+public interface IntegrationViewCallables {
+
+	void testCallable(boolean arg);
+
+	void allowAdditionalItems(boolean value);
+
+	void setFieldReadOnly(boolean value);
+
+	void setFieldEnabled(boolean value);
+
+	void setValue(String... items);
+
+	void useNewItemHandler(boolean useHandler);
+
+	JsonArrayList<String> getValue();
+
+}

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

@@ -17,6 +17,8 @@ public class ViewIT extends AbstractChipfieldTest {
 	private static final String IPSUM = "Ipsum";
 	private static final String ADDITIONAL = "Additional";
 
+	IntegrationViewCallables $server = createCallableProxy(IntegrationViewCallables.class);
+
     @Test
 	public void testUpgradedToCustomElement() {
 		ChipFieldElement chipfield = $(ChipFieldElement.class).first();
@@ -25,72 +27,76 @@ public void testUpgradedToCustomElement() {
 
 	@Test
 	public void testCallableSuccess() {
-		// test that the callable mechanism worls
-		call("testCallable", true);
+		// test that the callable mechanism works
+		$server.testCallable(true);
 	}
 
 	@Test
 	public void testCallableFailure() {
 		// test that the callable mechanism detect failures
-		assertThrows(RuntimeException.class, () -> call("testCallable", false));
+		assertThrows(RuntimeException.class, () -> $server.testCallable(false));
 	}
 
 	private Matcher<Collection<String>> isEqualTo(String... values) {
 		return Matchers.equalTo(Arrays.asList(values));
 	}
 
-	@Test
-	public void testCallableFailure2() {
-		// test that the callable mechanism detect failures
-		assertThrows(RuntimeException.class, () -> call("testCallable"));
-	}
-
 	@Test
 	public void testSelectByText() {
 		chipfield.selectByText(LOREM);
 		assertThat(chipfield.getValue(), isEqualTo(LOREM));
+		assertThat($server.getValue(), isEqualTo(LOREM));
 
 		chipfield.selectByText(IPSUM);
 		assertThat(chipfield.getValue(), isEqualTo(LOREM, IPSUM));
+		assertThat($server.getValue(), isEqualTo(LOREM, IPSUM));
 
 		chipfield.sendKeys(Keys.BACK_SPACE);
 		assertThat(chipfield.getValue(), isEqualTo(LOREM));
+		assertThat($server.getValue(), isEqualTo(LOREM));
 
 		chipfield.sendKeys(Keys.BACK_SPACE);
 		assertThat(chipfield.getValue(), Matchers.empty());
+		assertThat($server.getValue(), Matchers.empty());
     }
 
 
 	@Test
 	public void testAdditionalItemEnabled() {
-		call("allowAdditionalItems", true);
+		$server.allowAdditionalItems(true);
+		$server.useNewItemHandler(true);
 
 		chipfield.sendKeys(ADDITIONAL, Keys.ENTER);
 		assertThat(chipfield.getValue(), isEqualTo(ADDITIONAL));
+		assertThat($server.getValue(), isEqualTo(ADDITIONAL));
 
 		chipfield.sendKeys(LOREM, Keys.ENTER);
 		assertThat(chipfield.getValue(), isEqualTo(ADDITIONAL, LOREM));
+		assertThat($server.getValue(), isEqualTo(ADDITIONAL, LOREM));
 	}
 
 	@Test
 	public void testAdditionalItemDisabled() {
 		chipfield.sendKeys("Additional", Keys.ENTER);
 		assertThat(chipfield.getValue(), Matchers.empty());
+		assertThat($server.getValue(), Matchers.empty());
 	}
 
 	@Test
 	public void testReadOnly() {
-
 		chipfield.selectByText(LOREM);
 		assertThat(chipfield.getValue(), isEqualTo(LOREM));
+		assertThat($server.getValue(), isEqualTo(LOREM));
 
-
+		$server.setFieldReadOnly(true);
 		chipfield.sendKeys(Keys.BACK_SPACE);
 		assertThat(chipfield.getValue(), isEqualTo(LOREM));
+		assertThat($server.getValue(), isEqualTo(LOREM));
 
-
+		$server.allowAdditionalItems(true);
 		chipfield.sendKeys(ADDITIONAL, Keys.ENTER);
 		assertThat(chipfield.getValue(), isEqualTo(LOREM));
+		assertThat($server.getValue(), isEqualTo(LOREM));
 	}