Fixed close leakage of SqlSession

Author: kazuki43zoo

src/test/java/org/apache/ibatis/submitted/batch_keys/BatchKeysTest.java

@@ -104,10 +104,13 @@ public void testInsert() throws Exception {
     } finally {
       sqlSession.close();
     }
-
-    sqlSession = sqlSessionFactory.openSession();
-    List<User> users = sqlSession.selectList("select");
-    Assert.assertTrue(users.size() == 2);
+    try {
+      sqlSession = sqlSessionFactory.openSession();
+      List<User> users = sqlSession.selectList("select");
+      Assert.assertTrue(users.size() == 2);
+    } finally {
+      sqlSession.close();
+    }
   }
 
 
@@ -127,9 +130,13 @@ public void testInsertJdbc3() throws Exception {
       sqlSession.close();
     }
 
-    sqlSession = sqlSessionFactory.openSession();
-    List<User> users = sqlSession.selectList("selectIdentity");
-    Assert.assertTrue(users.size() == 2);
+    try {
+      sqlSession = sqlSessionFactory.openSession();
+      List<User> users = sqlSession.selectList("selectIdentity");
+      Assert.assertTrue(users.size() == 2);
+    } finally {
+      sqlSession.close();
+    }
   }
 
   @Test
@@ -149,9 +156,13 @@ public void testInsertWithMapper() throws Exception {
       sqlSession.close();
     }
 
-    sqlSession = sqlSessionFactory.openSession();
-    List<User> users = sqlSession.selectList("select");
-    Assert.assertTrue(users.size() == 2);
+    try {
+      sqlSession = sqlSessionFactory.openSession();
+      List<User> users = sqlSession.selectList("select");
+      Assert.assertTrue(users.size() == 2);
+    } finally {
+      sqlSession.close();
+    }
   }
 
   @Test
@@ -171,9 +182,13 @@ public void testInsertMapperJdbc3() throws Exception {
       sqlSession.close();
     }
 
-    sqlSession = sqlSessionFactory.openSession();
-    List<User> users = sqlSession.selectList("selectIdentity");
-    Assert.assertTrue(users.size() == 2);
+    try {
+      sqlSession = sqlSessionFactory.openSession();
+      List<User> users = sqlSession.selectList("selectIdentity");
+      Assert.assertTrue(users.size() == 2);
+    } finally {
+      sqlSession.close();
+    }
   }
 
   @Test
@@ -189,9 +204,13 @@ public void testInsertMapperNoBatchJdbc3() throws Exception {
       sqlSession.close();
     }
 
-    sqlSession = sqlSessionFactory.openSession();
-    List<User> users = sqlSession.selectList("selectIdentity");
-    Assert.assertTrue(users.size() == 1);
+    try {
+      sqlSession = sqlSessionFactory.openSession();
+      List<User> users = sqlSession.selectList("selectIdentity");
+      Assert.assertTrue(users.size() == 1);
+    } finally {
+      sqlSession.close();
+    }
   }
 
 }