From 61ffac2c77c1f20878893898ce7e19cb11c99db3 Mon Sep 17 00:00:00 2001
From: Marcos Lopez Gonzalez <asturcon1234@gmail.com>
Date: Fri, 9 Nov 2018 20:54:10 +0100
Subject: [PATCH] jpa cast error

---
 .../baeldung/jpa/stringcast/DummyEntity.java  | 44 ++++++++++++++
 .../jpa/stringcast/QueryExecutor.java         | 41 +++++++++++++
 .../main/resources/META-INF/persistence.xml   | 15 +++++
 .../jpa/stringcast/SpringCastTest.java        | 58 +++++++++++++++++++
 .../src/test/resources/persistence.xml        | 35 +++++++++++
 5 files changed, 193 insertions(+)
 create mode 100644 persistence-modules/java-jpa/src/main/java/com/baeldung/jpa/stringcast/DummyEntity.java
 create mode 100644 persistence-modules/java-jpa/src/main/java/com/baeldung/jpa/stringcast/QueryExecutor.java
 create mode 100644 persistence-modules/java-jpa/src/test/java/com/baeldung/jpa/stringcast/SpringCastTest.java

diff --git a/persistence-modules/java-jpa/src/main/java/com/baeldung/jpa/stringcast/DummyEntity.java b/persistence-modules/java-jpa/src/main/java/com/baeldung/jpa/stringcast/DummyEntity.java
new file mode 100644
index 0000000000..12b57ff112
--- /dev/null
+++ b/persistence-modules/java-jpa/src/main/java/com/baeldung/jpa/stringcast/DummyEntity.java
@@ -0,0 +1,44 @@
+package com.baeldung.jpa.stringcast;
+
+import javax.persistence.*;
+
+@SqlResultSetMapping(name = "textQueryMapping", classes = {
+  @ConstructorResult(targetClass = DummyEntity.class, columns = {
+    @ColumnResult(name = "text")
+  })
+})
+@Entity
+@Table(name = "dummy")
+public class DummyEntity {
+
+    @Id
+    @GeneratedValue(strategy = GenerationType.IDENTITY)
+    private Long id;
+
+    private String text;
+
+    public DummyEntity() {
+
+    }
+
+    public DummyEntity(String text) {
+        this.text = text;
+    }
+
+    public String getText() {
+        return text;
+    }
+
+    public void setText(String text) {
+        this.text = text;
+    }
+
+    public Long getId() {
+        return id;
+    }
+
+    public void setId(Long id) {
+        this.id = id;
+    }
+
+}
diff --git a/persistence-modules/java-jpa/src/main/java/com/baeldung/jpa/stringcast/QueryExecutor.java b/persistence-modules/java-jpa/src/main/java/com/baeldung/jpa/stringcast/QueryExecutor.java
new file mode 100644
index 0000000000..6f1e2ee5ca
--- /dev/null
+++ b/persistence-modules/java-jpa/src/main/java/com/baeldung/jpa/stringcast/QueryExecutor.java
@@ -0,0 +1,41 @@
+package com.baeldung.jpa.stringcast;
+
+import com.sun.istack.internal.Nullable;
+
+import javax.persistence.EntityManager;
+import javax.persistence.Query;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.stream.Collectors;
+
+public class QueryExecutor {
+
+    public static List<String[]> executeNativeQueryNoCastCheck(String statement, EntityManager em) {
+        Query query = em.createNativeQuery(statement);
+        return query.getResultList();
+    }
+
+    public static List<String[]> executeNativeQueryWithCastCheck(String statement, EntityManager em) {
+        Query query = em.createNativeQuery(statement);
+        List results = query.getResultList();
+
+        if (results.isEmpty()) {
+            return new ArrayList<>();
+        }
+
+        if (results.get(0) instanceof String) {
+            return ((List<String>) results)
+              .stream()
+              .map(s -> new String[] { s })
+              .collect(Collectors.toList());
+        } else {
+            return (List<String[]>) results;
+        }
+    }
+
+    public static <T> List<T> executeNativeQueryGeneric(String statement, String mapping, EntityManager em) {
+        Query query = em.createNativeQuery(statement, mapping);
+        return query.getResultList();
+    }
+
+}
diff --git a/persistence-modules/java-jpa/src/main/resources/META-INF/persistence.xml b/persistence-modules/java-jpa/src/main/resources/META-INF/persistence.xml
index 3d881673b2..6345e7f364 100644
--- a/persistence-modules/java-jpa/src/main/resources/META-INF/persistence.xml
+++ b/persistence-modules/java-jpa/src/main/resources/META-INF/persistence.xml
@@ -20,6 +20,21 @@
             <property name="hibernate.temp.use_jdbc_metadata_defaults" value="false"/>
         </properties>
     </persistence-unit>
+
+    <persistence-unit name="jpa-h2">
+        <provider>org.hibernate.jpa.HibernatePersistenceProvider</provider>
+        <class>com.baeldung.jpa.stringcast.DummyEntity</class>
+        <properties>
+            <property name="javax.persistence.jdbc.driver" value="org.h2.Driver" />
+            <property name="javax.persistence.jdbc.url" value="jdbc:h2:mem:test" />
+            <property name="javax.persistence.jdbc.user" value="sa" />
+            <property name="javax.persistence.jdbc.password" value="" />
+            <property name="hibernate.dialect" value="org.hibernate.dialect.H2Dialect" />
+            <property name="hibernate.hbm2ddl.auto" value="create-drop" />
+            <property name="show_sql" value="true"/>
+            <property name="hibernate.temp.use_jdbc_metadata_defaults" value="false"/>
+        </properties>
+    </persistence-unit>
     
     <persistence-unit name="jpa-db">
         <provider>org.hibernate.jpa.HibernatePersistenceProvider</provider>
diff --git a/persistence-modules/java-jpa/src/test/java/com/baeldung/jpa/stringcast/SpringCastTest.java b/persistence-modules/java-jpa/src/test/java/com/baeldung/jpa/stringcast/SpringCastTest.java
new file mode 100644
index 0000000000..f66bd5d4e4
--- /dev/null
+++ b/persistence-modules/java-jpa/src/test/java/com/baeldung/jpa/stringcast/SpringCastTest.java
@@ -0,0 +1,58 @@
+package com.baeldung.jpa.stringcast;
+
+import org.junit.BeforeClass;
+import org.junit.Test;
+
+import javax.persistence.EntityManager;
+import javax.persistence.EntityManagerFactory;
+import javax.persistence.EntityTransaction;
+import javax.persistence.Persistence;
+import java.util.List;
+
+import static org.junit.Assert.assertEquals;
+
+public class SpringCastTest {
+
+    private static EntityManager em;
+    private static EntityManagerFactory emFactory;
+
+    @BeforeClass
+    public static void setup() {
+        emFactory = Persistence.createEntityManagerFactory("jpa-h2");
+        em = emFactory.createEntityManager();
+
+        // insert an object into the db
+        DummyEntity dummyEntity = new DummyEntity();
+        dummyEntity.setText("text");
+
+        EntityTransaction tr = em.getTransaction();
+        tr.begin();
+        em.persist(dummyEntity);
+        tr.commit();
+    }
+
+    @Test(expected = ClassCastException.class)
+    public void givenExecutorNoCastCheck_whenQueryReturnsOneColumn_thenClassCastThrown() {
+        List<String[]> results = QueryExecutor.executeNativeQueryNoCastCheck("select text from dummy", em);
+
+        // fails
+        for (String[] row : results) {
+            // do nothing
+        }
+    }
+
+    @Test
+    public void givenExecutorWithCastCheck_whenQueryReturnsOneColumn_thenNoClassCastThrown() {
+        List<String[]> results = QueryExecutor.executeNativeQueryWithCastCheck("select text from dummy", em);
+        assertEquals("text", results.get(0)[0]);
+    }
+
+    @Test
+    public void givenExecutorGeneric_whenQueryReturnsOneColumn_thenNoClassCastThrown() {
+        List<DummyEntity> results = QueryExecutor.executeNativeQueryGeneric("select text from dummy", "textQueryMapping", em);
+        assertEquals("text", results
+          .get(0)
+          .getText());
+    }
+
+}
diff --git a/persistence-modules/java-jpa/src/test/resources/persistence.xml b/persistence-modules/java-jpa/src/test/resources/persistence.xml
index d94221b54f..f65f2c7c62 100644
--- a/persistence-modules/java-jpa/src/test/resources/persistence.xml
+++ b/persistence-modules/java-jpa/src/test/resources/persistence.xml
@@ -17,5 +17,40 @@
 			<property name="hibernate.show_sql" value="true" />
 		</properties>
 	</persistence-unit>
+
+	<persistence-unit name="jpa-h22">
+		<provider>org.hibernate.jpa.HibernatePersistenceProvider</provider>
+		<class>com.baeldung.jpa.stringcast.DummyEntity</class>
+		<properties>
+			<property name="javax.persistence.jdbc.driver" value="org.h2.Driver" />
+			<property name="javax.persistence.jdbc.url" value="jdbc:h2:mem:test" />
+			<property name="javax.persistence.jdbc.user" value="sa" />
+			<property name="javax.persistence.jdbc.password" value="" />
+			<property name="hibernate.dialect" value="org.hibernate.dialect.H2Dialect" />
+			<property name="hibernate.hbm2ddl.auto" value="create-drop" />
+			<property name="show_sql" value="true"/>
+			<property name="hibernate.temp.use_jdbc_metadata_defaults" value="false"/>
+		</properties>
+	</persistence-unit>
+
+	<persistence-unit name="h2" transaction-type="RESOURCE_LOCAL">
+
+		<!-- add classes -->
+		<class>com.baeldung.jpa.stringcast.DummyEntity</class>
+
+		<properties>
+			<!-- Configuring JDBC properties -->
+			<property name="javax.persistence.jdbc.url" value="jdbc:h2:mem:test;DB_CLOSE_DELAY=-1"/>
+			<property name="javax.persistence.jdbc.driver" value="org.h2.Driver"/>
+
+			<!-- Hibernate properties -->
+			<property name="hibernate.dialect" value="org.hibernate.dialect.H2Dialect"/>
+			<property name="hibernate.hbm2ddl.auto" value="validate"/>
+			<property name="hibernate.format_sql" value="false"/>
+			<property name="hibernate.show_sql" value="true"/>
+
+		</properties>
+	</persistence-unit>
+
 </persistence>