Fixed NPE when converter is null (#10043)

Change-Id: I1d753a4da16feae0c83d755e9dcceae86f3dfbdc

1 files changed, 20 insertions, 0 deletions

diff --git a/server/tests/src/com/vaadin/tests/server/component/abstractfield/AbstractFieldValueConversions.java b/server/tests/src/com/vaadin/tests/server/component/abstractfield/AbstractFieldValueConversions.java
index 73f6063fdd..83224861b6 100644
--- a/server/tests/src/com/vaadin/tests/server/component/abstractfield/AbstractFieldValueConversions.java
+++ b/server/tests/src/com/vaadin/tests/server/component/abstractfield/AbstractFieldValueConversions.java
@@ -2,10 +2,15 @@ package com.vaadin.tests.server.component.abstractfield;
 
 import java.util.Locale;
 
+import junit.framework.Assert;
 import junit.framework.TestCase;
 
+import org.junit.Test;
+
 import com.vaadin.data.util.MethodProperty;
+import com.vaadin.data.util.ObjectProperty;
 import com.vaadin.data.util.converter.Converter;
+import com.vaadin.data.util.converter.Converter.ConversionException;
 import com.vaadin.data.util.converter.StringToIntegerConverter;
 import com.vaadin.server.VaadinServiceSession;
 import com.vaadin.tests.data.bean.Address;
@@ -179,4 +184,19 @@ public class AbstractFieldValueConversions extends TestCase {
 
     }
 
+    @Test
+    public void testNullConverter() {
+        TextField tf = new TextField("foo");
+        tf.setPropertyDataSource(new ObjectProperty<Integer>(12));
+        tf.setConverter((Converter) null);
+        try {
+            Object v = tf.getConvertedValue();
+            System.out.println(v);
+            Assert.fail("Trying to convert String -> Integer should fail when there is no converter");
+        } catch (ConversionException e) {
+            // ok, should happen when there is no converter but conversion is
+            // needed
+        }
+    }
+
 }