From b5a898342d84a198afe5e6961df5fc401b0221f1 Mon Sep 17 00:00:00 2001 From: jcartign Date: Wed, 28 Sep 2016 14:44:06 +0200 Subject: [PATCH] In-class bug fixes --- README.md | 1 + src/test/java/fr/plil/sio/persistence/jdbc/GroupServiceTest.java | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index ca6f94e..de331a3 100644 --- a/README.md +++ b/README.md @@ -11,6 +11,7 @@ Documentation ------------- * add usage scenario +* complete documentation to precise one level dependencies for create and find methods * clarify tree of rights * clarify why the dependencies need to be managed in the service (to avoid to break the CRUD model, *i.e.* reppsitory methods are specialised to load the one-level diff --git a/src/test/java/fr/plil/sio/persistence/jdbc/GroupServiceTest.java b/src/test/java/fr/plil/sio/persistence/jdbc/GroupServiceTest.java index fe59d40..36bf728 100644 --- a/src/test/java/fr/plil/sio/persistence/jdbc/GroupServiceTest.java +++ b/src/test/java/fr/plil/sio/persistence/jdbc/GroupServiceTest.java @@ -127,8 +127,8 @@ public class GroupServiceTest extends AbstractServiceSupport { assertEquals(0, group.getRights().size()); } - @Test - public void testRemoveRightIfNotPresent() { + @Test(expected = IllegalArgumentException.class) + public void testRemoveRightIfNotInDatabase() { Right right = new Right(); right.setName("not-a-right"); assertFalse(groupService.removeRight("group", right)); -- libgit2 0.21.2