Close #411 with a test case. See the previous commit [263b47e] .

Author: harawata

src/test/java/org/apache/ibatis/submitted/foreach/ForEachTest.java

@@ -18,21 +18,28 @@
 import java.io.Reader;
 import java.sql.Connection;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.List;
 
+import org.apache.ibatis.exceptions.PersistenceException;
 import org.apache.ibatis.io.Resources;
 import org.apache.ibatis.jdbc.ScriptRunner;
 import org.apache.ibatis.session.SqlSession;
 import org.apache.ibatis.session.SqlSessionFactory;
 import org.apache.ibatis.session.SqlSessionFactoryBuilder;
 import org.junit.Assert;
 import org.junit.BeforeClass;
+import org.junit.Rule;
 import org.junit.Test;
+import org.junit.rules.ExpectedException;
 
 public class ForEachTest {
 
   private static SqlSessionFactory sqlSessionFactory;
 
+  @Rule
+  public ExpectedException ex = ExpectedException.none();
+
   @BeforeClass
   public static void setUp() throws Exception {
     // create a SqlSessionFactory
@@ -124,4 +131,18 @@ public void nullItemInContext() {
     }
   }
 
+  @Test
+  public void shouldReportMissingPropertyName() {
+    ex.expect(PersistenceException.class);
+    ex.expectMessage("There is no getter for property named 'idd' in 'class org.apache.ibatis.submitted.foreach.User'");
+
+    SqlSession sqlSession = sqlSessionFactory.openSession();
+    try {
+      Mapper mapper = sqlSession.getMapper(Mapper.class);
+      mapper.typoInItemProperty(Arrays.asList(new User()));
+    } finally {
+      sqlSession.close();
+    }
+  }
+
 }

src/test/java/org/apache/ibatis/submitted/foreach/Mapper.java

@@ -27,4 +27,5 @@ public interface Mapper {
 
   String selectWithNullItemCheck(List<User> users);
 
+  int typoInItemProperty(List<User> users);
 }

src/test/java/org/apache/ibatis/submitted/foreach/Mapper.xml

@@ -62,4 +62,11 @@
       </where>
   </select>
 
+  <insert id="typoInItemProperty">
+    insert into users (id, name) values
+    <foreach item="item" collection="list" separator=",">
+        (#{item.idd}, #{item.name})
+    </foreach>
+  </insert>
+
 </mapper>