From fd5794f57b43b0f1794de7d914f366f494c95ceb Mon Sep 17 00:00:00 2001 From: Manuel Rego Casasnovas Date: Thu, 10 May 2012 08:58:15 +0200 Subject: [PATCH] Improve UserDAOTest adding a new check for getUnboundUsers method FEA: ItEr76S27ResourceBinding --- .../business/test/users/daos/UserDAOTest.java | 34 ++++++++++++++----- 1 file changed, 26 insertions(+), 8 deletions(-) diff --git a/libreplan-business/src/test/java/org/libreplan/business/test/users/daos/UserDAOTest.java b/libreplan-business/src/test/java/org/libreplan/business/test/users/daos/UserDAOTest.java index 3212d92f9..af25d774c 100644 --- a/libreplan-business/src/test/java/org/libreplan/business/test/users/daos/UserDAOTest.java +++ b/libreplan-business/src/test/java/org/libreplan/business/test/users/daos/UserDAOTest.java @@ -299,15 +299,23 @@ public class UserDAOTest { assertEquals(previous + 1, unboundUsers.size()); } - private Worker givenStoredWorkerRelatedTo(User user) { - Worker worker = Worker.create(); - worker.setFirstName("Name " + UUID.randomUUID()); - worker.setSurname("Surname " + UUID.randomUUID()); - worker.setNif("ID " + UUID.randomUUID()); - worker.setUser(user); - workerDAO.save(worker); + private Worker givenStoredWorkerRelatedTo(final User user) { + return transactionService + .runOnAnotherTransaction(new IOnTransaction() { - return worker; + @Override + public Worker execute() { + Worker worker = Worker.create(); + worker.setFirstName("Name " + UUID.randomUUID()); + worker.setSurname("Surname " + UUID.randomUUID()); + worker.setNif("ID " + UUID.randomUUID()); + worker.setUser(user); + workerDAO.save(worker); + worker.dontPoseAsTransientObjectAnymore(); + + return worker; + } + }); } @Test @@ -320,4 +328,14 @@ public class UserDAOTest { assertEquals(previous, unboundUsers.size()); } + @Test + public void testUnoundUsers3() { + int previous = userDAO.getUnboundUsers(null).size(); + User user = createUser(getUniqueName()); + user.setWorker(givenStoredWorkerRelatedTo(user)); + + List unboundUsers = userDAO.getUnboundUsers(user.getWorker()); + assertEquals(previous + 1, unboundUsers.size()); + } + }