diff --git a/libreplan-webapp/src/test/java/org/libreplan/web/test/ws/orders/OrderElementServiceTest.java b/libreplan-webapp/src/test/java/org/libreplan/web/test/ws/orders/OrderElementServiceTest.java index 0b23b9a15..3d9b610b4 100644 --- a/libreplan-webapp/src/test/java/org/libreplan/web/test/ws/orders/OrderElementServiceTest.java +++ b/libreplan-webapp/src/test/java/org/libreplan/web/test/ws/orders/OrderElementServiceTest.java @@ -444,12 +444,7 @@ public class OrderElementServiceTest { .addOrders(orderListDTO).instanceConstraintViolationsList; assertThat(instanceConstraintViolationsList.size(), equalTo(0)); - try { - orderElementDAO.findByCode(code); - assertTrue(true); - } catch (InstanceNotFoundException e) { - fail(); - } + checkIfExistsByCodeInAnotherTransaction(code); } @Test @@ -569,12 +564,7 @@ public class OrderElementServiceTest { assertTrue(instanceConstraintViolationsList.toString(), instanceConstraintViolationsList.size() == 0); - try { - orderElementDAO.findByCode(code); - assertTrue(true); - } catch (InstanceNotFoundException e) { - fail(); - } + checkIfExistsByCodeInAnotherTransaction(code); } @Test @@ -657,12 +647,7 @@ public class OrderElementServiceTest { assertTrue(instanceConstraintViolationsList.toString(), instanceConstraintViolationsList.size() == 0); - try { - orderElementDAO.findByCode(code); - assertTrue(true); - } catch (InstanceNotFoundException e) { - fail(); - } + checkIfExistsByCodeInAnotherTransaction(code); } @Test @@ -716,7 +701,11 @@ public class OrderElementServiceTest { .addOrders(orderListDTO).instanceConstraintViolationsList; assertThat(instanceConstraintViolationsList.size(), equalTo(0)); - transactionService.runOnTransaction(new IOnTransaction() { + checkIfExistsByCodeInAnotherTransaction(code); + } + + private void checkIfExistsByCodeInAnotherTransaction(final String code) { + transactionService.runOnAnotherTransaction(new IOnTransaction() { @Override public Void execute() { try {