feat: updated lesson15 unit test HW to handle exception case by Yemi

Author: jimoye244

lesson_15/tdd/tdd_app/src/test/java/EmployeeManagerTest.java

@@ -1,5 +1,5 @@
+import static org.assertj.core.api.Assertions.assertThatThrownBy;
 import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertNull;
 
 import com.codedifferently.lesson15.Employee;
 import com.codedifferently.lesson15.EmployeeManager;
@@ -30,40 +30,44 @@ public void testAddEmployee() {
   }
 
   @Test
-  public void testgetEmployee() {
+  public void testUpdateEmployee() {
     // Arrange
-    Employee employee = new Employee(2, "Washington", "Finance", 29.23);
+    Employee employee = new Employee(5, "John", "Safety", 50);
+    cut.addEmployee(employee);
 
     // Act
-    Employee returnedEmployee = cut.getEmployee(employee.getId());
+    cut.updateEmployee(employee);
 
     // Assert
-    assertNull(returnedEmployee);
+    int expectedId = 5;
+    assertEquals(expectedId, employee.getId());
   }
 
   @Test
   public void testRemoveEmployee() {
     // Arrange
-    Employee employee = new Employee(3, "Belvedere", "Finance", 100.1);
-
+    Employee employee = new Employee(2, "Belvedere", "Finance", 100.1);
+    cut.addEmployee(employee);
     // Act
     cut.removeEmployee(employee.getId());
 
     // Assert
-    int removedId = 3;
-    assertNull(cut.getEmployee(removedId));
+    int removedId = 2;
+    assertThatThrownBy(() -> cut.removeEmployee(removedId))
+        .isInstanceOf(IllegalArgumentException.class)
+        .hasMessage("Employee does not in collection with id " + removedId);
   }
 
   @Test
-  public void testUpdateEmployee() {
+  public void testGetEmployee() {
     // Arrange
-    Employee employee = new Employee(5, "John", "Safety", 50);
-
+    Employee employee = new Employee(2, "Washington", "Finance", 29.23);
     // Act
-    cut.updateEmployee(employee);
+    cut.addEmployee(employee);
 
+    int employeeId = employee.getId();
+    Employee returnedEmployee = cut.getEmployee(employeeId);
     // Assert
-    int expectedId = 5;
-    assertEquals(expectedId, employee.getId());
+    assertEquals(employee, returnedEmployee);
   }
-}
+}

lesson_15/tdd/tdd_app/src/test/java/com/codedifferently/lesson15/EmployeeTest.java

@@ -61,7 +61,7 @@ public void testGetSalary() {
     double expectedSalary = cut.getSalary();
 
     // Assert
-    double actualSalary = cut.getSalary();
+    double actualSalary = 200.00;
     assertEquals(expectedSalary, actualSalary, "The Salary should be '200.00'");
   }