diff options
author | Johannes Dahlström <johannesd@vaadin.com> | 2015-06-08 17:21:58 +0300 |
---|---|---|
committer | Vaadin Code Review <review@vaadin.com> | 2015-06-09 10:56:26 +0000 |
commit | 5ebfdecb1b4eb016b674f7a01257959a50047eb3 (patch) | |
tree | b55f79890aac42697ac765111a1cc041cd7ce50c /server/tests/src | |
parent | 1eba7f023bf534c8dd5d508ef5f9a5970af67885 (diff) | |
download | vaadin-framework-5ebfdecb1b4eb016b674f7a01257959a50047eb3.tar.gz vaadin-framework-5ebfdecb1b4eb016b674f7a01257959a50047eb3.zip |
Better handle exceptions when opening Grid editor (#17935)
Change-Id: I68103db75c422b042988c6662da268ff9d11a306
Diffstat (limited to 'server/tests/src')
-rw-r--r-- | server/tests/src/com/vaadin/tests/server/component/grid/GridEditorTest.java | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/server/tests/src/com/vaadin/tests/server/component/grid/GridEditorTest.java b/server/tests/src/com/vaadin/tests/server/component/grid/GridEditorTest.java index 135d7d398c..b70f17779a 100644 --- a/server/tests/src/com/vaadin/tests/server/component/grid/GridEditorTest.java +++ b/server/tests/src/com/vaadin/tests/server/component/grid/GridEditorTest.java @@ -184,12 +184,16 @@ public class GridEditorTest { .getEditorField(); field.setValue("New Name"); + Property<?> datasource = field.getPropertyDataSource(); + grid.cancelEditor(); assertFalse(grid.isEditorActive()); assertNull(grid.getEditedItemId()); assertFalse(field.isModified()); - assertEquals(DEFAULT_NAME, field.getValue()); - assertEquals(DEFAULT_NAME, field.getPropertyDataSource().getValue()); + assertEquals("", field.getValue()); + assertEquals(DEFAULT_NAME, datasource.getValue()); + assertNull(field.getPropertyDataSource()); + assertNull(grid.getEditorFieldGroup().getItemDataSource()); } @Test(expected = IllegalArgumentException.class) |