From 0b94647d38074d9cdc6c8f021949ea7db524a24d Mon Sep 17 00:00:00 2001 From: mallim Date: Tue, 16 Jul 2013 10:34:52 +0800 Subject: [PATCH] Failed test case submission for issue #62 --- .../CallSettersOnNullsTest.java | 24 +++++++++++++++---- .../call_setters_on_nulls/CreateDB.sql | 13 ++++++++++ .../call_setters_on_nulls/Mapper.java | 2 ++ .../call_setters_on_nulls/Mapper.xml | 4 ++++ 4 files changed, 39 insertions(+), 4 deletions(-) diff --git a/src/test/java/org/apache/ibatis/submitted/call_setters_on_nulls/CallSettersOnNullsTest.java b/src/test/java/org/apache/ibatis/submitted/call_setters_on_nulls/CallSettersOnNullsTest.java index 063b89c4f3e..259a59cdf69 100644 --- a/src/test/java/org/apache/ibatis/submitted/call_setters_on_nulls/CallSettersOnNullsTest.java +++ b/src/test/java/org/apache/ibatis/submitted/call_setters_on_nulls/CallSettersOnNullsTest.java @@ -15,10 +15,6 @@ */ package org.apache.ibatis.submitted.call_setters_on_nulls; -import java.io.Reader; -import java.sql.Connection; -import java.util.Map; - import org.apache.ibatis.io.Resources; import org.apache.ibatis.jdbc.ScriptRunner; import org.apache.ibatis.session.SqlSession; @@ -28,6 +24,11 @@ import org.junit.BeforeClass; import org.junit.Test; +import java.io.Reader; +import java.sql.Connection; +import java.util.List; +import java.util.Map; + public class CallSettersOnNullsTest { private static SqlSessionFactory sqlSessionFactory; @@ -86,4 +87,19 @@ public void shouldCallNullOnMap() { } } + @Test + public void shouldCallNullOnMapForSingleColumn() { + SqlSession sqlSession = sqlSessionFactory.openSession(); + try { + Mapper mapper = sqlSession.getMapper(Mapper.class); + List> oneColumns = mapper.getNameOnly(); + for( Map onecol: oneColumns ) + { + Assert.assertTrue( onecol.containsKey( "NAME" ) ); + } + } finally { + sqlSession.close(); + } + } + } diff --git a/src/test/java/org/apache/ibatis/submitted/call_setters_on_nulls/CreateDB.sql b/src/test/java/org/apache/ibatis/submitted/call_setters_on_nulls/CreateDB.sql index df4c35f2592..ab2a9914c40 100644 --- a/src/test/java/org/apache/ibatis/submitted/call_setters_on_nulls/CreateDB.sql +++ b/src/test/java/org/apache/ibatis/submitted/call_setters_on_nulls/CreateDB.sql @@ -16,9 +16,22 @@ drop table users if exists; +drop table users2 if exists; + create table users ( id int, name varchar(20) ); +create table users2( + id int, + name varchar(20) +) + insert into users (id, name) values(1, NULL); + +insert into users2 (id, name) values( 2, 'mary' ); +insert into users2 (id, name) values( 3, 'angel' ); +insert into users2 (id, name) values( 1, NULL ); + + diff --git a/src/test/java/org/apache/ibatis/submitted/call_setters_on_nulls/Mapper.java b/src/test/java/org/apache/ibatis/submitted/call_setters_on_nulls/Mapper.java index b6c563f73b6..388cdc140ff 100644 --- a/src/test/java/org/apache/ibatis/submitted/call_setters_on_nulls/Mapper.java +++ b/src/test/java/org/apache/ibatis/submitted/call_setters_on_nulls/Mapper.java @@ -15,6 +15,7 @@ */ package org.apache.ibatis.submitted.call_setters_on_nulls; +import java.util.List; import java.util.Map; public interface Mapper { @@ -22,5 +23,6 @@ public interface Mapper { User getUserMapped(Integer id); User getUserUnmapped(Integer id); Map getUserInMap(Integer id); + List> getNameOnly(); } diff --git a/src/test/java/org/apache/ibatis/submitted/call_setters_on_nulls/Mapper.xml b/src/test/java/org/apache/ibatis/submitted/call_setters_on_nulls/Mapper.xml index 912e560eca4..4df11acc213 100644 --- a/src/test/java/org/apache/ibatis/submitted/call_setters_on_nulls/Mapper.xml +++ b/src/test/java/org/apache/ibatis/submitted/call_setters_on_nulls/Mapper.xml @@ -37,4 +37,8 @@ + +