improved testing, all tests pass

src/test/java/io/zipcoder/crudapp/PersonControllerTest.java

@@ -6,12 +6,15 @@ import org.junit.Test;
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
 import org.mockito.MockitoAnnotations;
+import org.mockito.invocation.InvocationOnMock;
+import org.mockito.stubbing.Answer;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
 
 import java.util.ArrayList;
 
 import static org.mockito.Matchers.any;
+import static org.mockito.Mockito.doAnswer;
 import static org.mockito.Mockito.when;
 
 public class PersonControllerTest {
@@ -54,7 +57,7 @@ public class PersonControllerTest {
     }
 
     @Test
-    public void getPersonList(){
+    public void getPersonListTest(){
         Iterable<Person> personList = new ArrayList<>();
         when(repository.findAll()).thenReturn(personList);
 
@@ -65,7 +68,7 @@ public class PersonControllerTest {
     }
 
     @Test
-    public void updatePerson(){
+    public void updatePersonTest(){
         Person p = new Person("a", "b");
         Person update = new Person("a","b");
         when(repository.save(p)).thenReturn(update);
@@ -75,26 +78,28 @@ public class PersonControllerTest {
         Assert.assertEquals(actual.getStatusCode(), HttpStatus.OK);
         Assert.assertEquals(actual.getBody(), update);
     }
-    
-//    @Test
-//    public void updatePersonTest(){
-//        when(controller.updatePerson(any(Person.class))).thenReturn(new ResponseEntity<>( HttpStatus.OK));
-//
-//        Person p = new Person();
-//        p.setId(1L);
-//        ResponseEntity<Person> actual = controller.updatePerson(any(Person.class));
-//        ResponseEntity<Person> expected = new ResponseEntity<>(HttpStatus.OK);
-//
-//        Assert.assertEquals(expected, actual);
-//    }
-//
-//    @Test
-//    public void deletePerson(){
-//        when(controller.deletePerson(1)).thenReturn(new ResponseEntity<>( HttpStatus.NO_CONTENT));
-//
-//        ResponseEntity<Person> actual = controller.deletePerson(1);
-//        ResponseEntity<Person> expected = new ResponseEntity<>( HttpStatus.NO_CONTENT);
-//
-//        Assert.assertEquals(expected, actual);
-//    }
+
+    @Test
+    public void deletePersonTest(){
+        Person p = new Person("a", "b");
+        p.setId(8783L);
+        doAnswer(new Answer<Void>() {
+
+            @Override
+            public Void answer(InvocationOnMock invocation) throws Throwable {
+                Object[] arguments = invocation.getArguments();
+                if (arguments != null && arguments.length > 1 && arguments[0] != null && arguments[1] != null) {
+
+                    controller.deletePerson(8783);
+
+                }
+                return null;
+            }
+        }).when(repository).delete(8783L);
+
+        ResponseEntity<Person> actual = controller.deletePerson(8783);
+
+        Assert.assertEquals(actual.getStatusCode(), HttpStatus.NO_CONTENT);
+        Assert.assertEquals(actual.getBody(), null);
+    }
 }