From 66642522a988c225a42a863fca137d8ad32f9945 Mon Sep 17 00:00:00 2001
From: Stephane Gamard <stephane.gamard@searchbox.com>
Date: Tue, 12 Aug 2014 23:07:14 +0200
Subject: [PATCH] improved test performances for sq (reuse iConnection in
 DBUnits)

---
 .../core/persistence/AbstractDaoTestCase.java   | 17 ++++-------------
 1 file changed, 4 insertions(+), 13 deletions(-)

diff --git a/sonar-core/src/test/java/org/sonar/core/persistence/AbstractDaoTestCase.java b/sonar-core/src/test/java/org/sonar/core/persistence/AbstractDaoTestCase.java
index be78536a392..aac92fb380d 100644
--- a/sonar-core/src/test/java/org/sonar/core/persistence/AbstractDaoTestCase.java
+++ b/sonar-core/src/test/java/org/sonar/core/persistence/AbstractDaoTestCase.java
@@ -39,7 +39,6 @@ import org.dbunit.dataset.filter.DefaultColumnFilter;
 import org.dbunit.dataset.xml.FlatXmlDataSet;
 import org.dbunit.ext.mssql.InsertIdentityOperation;
 import org.dbunit.operation.DatabaseOperation;
-import org.junit.AfterClass;
 import org.junit.Assert;
 import org.junit.Before;
 import org.slf4j.Logger;
@@ -85,17 +84,6 @@ public abstract class AbstractDaoTestCase {
   private static IDatabaseConnection connection;
   private WorkQueue queue = new NullQueue();
 
-  @AfterClass
-  public static void cleanUp(){
-    try {
-      if (connection != null) {
-        connection.close();
-      }
-    } catch (SQLException e) {
-      // ignore
-    }
-  }
-
   @Before
   public void startDatabase() throws Exception {
     if (database == null) {
@@ -117,12 +105,15 @@ public abstract class AbstractDaoTestCase {
 
       databaseCommands = DatabaseCommands.forDialect(database.getDialect());
       databaseTester = new MyDBTester(database.getDataSource());
-      connection = createConnection();
 
       myBatis = new MyBatis(database, new Logback(), queue);
       myBatis.start();
     }
 
+    if(connection == null){
+      connection = createConnection();
+    }
+
     databaseCommands.truncateDatabase(database.getDataSource());
   }
 
-- 
2.39.5