Jean-Francois Leveque

Clarification @Displayname

...@@ -201,7 +201,6 @@ public class PublisherVersionViewTest { ...@@ -201,7 +201,6 @@ public class PublisherVersionViewTest {
201 public boolean updateCalled = false; 201 public boolean updateCalled = false;
202 202
203 @BeforeEach 203 @BeforeEach
204 - @DisplayName("Given some test reimplementation")
205 public void given() { 204 public void given() {
206 publisherVersionViewForSave = new PublisherVersionView() { 205 publisherVersionViewForSave = new PublisherVersionView() {
207 206
...@@ -219,7 +218,7 @@ public class PublisherVersionViewTest { ...@@ -219,7 +218,7 @@ public class PublisherVersionViewTest {
219 } 218 }
220 219
221 @Test 220 @Test
222 - @DisplayName("Test should call add on new publisher") 221 + @DisplayName("when view is for a new publisher, should delegate to add() method")
223 public void testSaveAdd() { 222 public void testSaveAdd() {
224 publisherVersionViewForSave.setNewPublisher(true); 223 publisherVersionViewForSave.setNewPublisher(true);
225 publisherVersionViewForSave.save(); 224 publisherVersionViewForSave.save();
...@@ -227,7 +226,7 @@ public class PublisherVersionViewTest { ...@@ -227,7 +226,7 @@ public class PublisherVersionViewTest {
227 } 226 }
228 227
229 @Test 228 @Test
230 - @DisplayName("Test should call update on old publisher") 229 + @DisplayName("when view is not for a new publisher, should delegate to update() method")
231 public void testSaveUpdate() { 230 public void testSaveUpdate() {
232 publisherVersionViewForSave.setNewPublisher(false); 231 publisherVersionViewForSave.setNewPublisher(false);
233 publisherVersionViewForSave.save(); 232 publisherVersionViewForSave.save();
......