msahmane / spring-persistence

Commit b5a898342d84a198afe5e6961df5fc401b0221f1

Authored by jcartign
1 parent d52b440e

In-class bug fixes 

- update README
- clarify method removeRightNotInDatabase
Showing 2 changed files with 3 additions and 2 deletions   Show diff stats
Inline Side-by-side
@@ -11,6 +11,7 @@ Documentation @@ -11,6 +11,7 @@ Documentation
11 ------------- 11 -------------
12 12
13 * add usage scenario 13 * add usage scenario
  14 +* complete documentation to precise one level dependencies for create and find methods
14 * clarify tree of rights 15 * clarify tree of rights
15 * clarify why the dependencies need to be managed in the service (to avoid to break the 16 * clarify why the dependencies need to be managed in the service (to avoid to break the
16 CRUD model, *i.e.* reppsitory methods are specialised to load the one-level 17 CRUD model, *i.e.* reppsitory methods are specialised to load the one-level
src/test/java/fr/plil/sio/persistence/jdbc/GroupServiceTest.java
  Show/Hide comments   View file @b5a8983
@@ -127,8 +127,8 @@ public class GroupServiceTest extends AbstractServiceSupport { @@ -127,8 +127,8 @@ public class GroupServiceTest extends AbstractServiceSupport {
127 assertEquals(0, group.getRights().size()); 127 assertEquals(0, group.getRights().size());
128 } 128 }
129 129
130 - @Test  
131 - public void testRemoveRightIfNotPresent() { 130 + @Test(expected = IllegalArgumentException.class)
  131 + public void testRemoveRightIfNotInDatabase() {
132 Right right = new Right(); 132 Right right = new Right();
133 right.setName("not-a-right"); 133 right.setName("not-a-right");
134 assertFalse(groupService.removeRight("group", right)); 134 assertFalse(groupService.removeRight("group", right));