Merge pull request database-rider#44 from empyrosx/master

Transactional interceptor doesn't close transaction when test falls down by assert
jenskreidler · Oct 2, 2017 · 464d622 · 464d622
2 parents 2e737f6 + 083992d
commit 464d622
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/rider-cdi/src/main/java/com/github/database/rider/cdi/DBUnitInterceptorImpl.java b/rider-cdi/src/main/java/com/github/database/rider/cdi/DBUnitInterceptorImpl.java
@@ -75,13 +75,11 @@ public Object intercept(InvocationContext invocationContext)
                 if(expectedDataSet != null){
                     dataSetProcessor.compareCurrentDataSetWith(new DataSetConfig(expectedDataSet.value()).disableConstraints(true),expectedDataSet.ignoreCols());
                 }
-
-            }catch (Exception e){
+            } finally {
                 if(isTransactionalTest && em.getTransaction().isActive()){
                     em.getTransaction().rollback();
                 }
-                throw e;
-            } finally {
+
                 int openConnectionsAfter = 0;
                 if(leakHunter != null){
                     openConnectionsAfter = leakHunter.openConnections();