feat: adds JosephCaballero folder containing EmployeeManagetTest.java and EmployeeTest.java. Modifies Employee.java and EmployeeManager.java to ensure any errors are thrown which we tested for in the prior files mentioned. Lesson_15 - Joseph Caballero

Author: Josephcabs

lesson_15/tdd/tdd_app/src/main/java/com/codedifferently/lesson15/Employee.java

@@ -45,4 +45,8 @@ public double getSalary() {
   public void setSalary(double salary) {
     this.salary = salary;
   }
+
+  public String getDetails() {
+    return "Employee: " + id + " " + name + " Department: " + department + " Salary: " + salary;
+  }
 }

lesson_15/tdd/tdd_app/src/main/java/com/codedifferently/lesson15/EmployeeManager.java

@@ -14,15 +14,21 @@ public void addEmployee(Employee employee) {
     employees.put(employee.getId(), employee);
   }
 
-  public Employee getEmployee(int id) {
+  public Employee getEmployee(int id) throws RuntimeException {
+    if (employees.get(id) == null) {
+      throw new NullPointerException("Employee with id " + id + " not found");
+    }
     return employees.get(id);
   }
 
   public void updateEmployee(Employee employee) {
     employees.put(employee.getId(), employee);
   }
 
-  public void removeEmployee(int id) {
+  public void removeEmployee(int id) throws RuntimeException {
+    if (employees.get(id) == null) {
+      throw new NullPointerException("Employee with id " + id + " not found");
+    }
     employees.remove(id);
   }
 

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

@@ -0,0 +1,68 @@
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.assertThatThrownBy;
+
+import com.codedifferently.lesson15.Employee;
+import com.codedifferently.lesson15.EmployeeManager;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+
+public class EmployeeManagerTest {
+  EmployeeManager employeeManager;
+  Employee employee;
+
+  @BeforeEach
+  void setUp() {
+    employeeManager = new EmployeeManager();
+    employee = new Employee(999, "Joseph", "Boxing", 150000000.5);
+    employeeManager.addEmployee(employee);
+  }
+
+  @Test
+  void testAddEmployee() {
+    assertThat(employeeManager.getEmployee(999)).isEqualTo(employee);
+  }
+
+  @Test
+  void testGetEmployee() {
+    assertThat(employeeManager.getEmployee(999)).isEqualTo(employee);
+  }
+
+  @Test
+  void testGetEmployee_failedDueToNull() throws RuntimeException {
+    assertThatThrownBy(() -> employeeManager.getEmployee(9))
+        .isInstanceOf(NullPointerException.class)
+        .hasMessage("Employee with id 9 not found");
+  }
+
+  @Test
+  void testEmployeeUpdate() {
+    Employee employee2 = new Employee(99, "Angelica", "CodeDifferently", 900.55);
+    employeeManager.addEmployee(employee2);
+    employeeManager.updateEmployee(employee2);
+    assertThat(employeeManager.getEmployeeCount()).isEqualTo(2);
+  }
+
+  @Test
+  void testRemoveEmployee() throws RuntimeException {
+    employeeManager.removeEmployee(999);
+    assertThat(employeeManager.getEmployeeCount()).isEqualTo(0);
+  }
+
+  @Test
+  void testRemoveEmployee_errorFromNullEmployeeId() throws RuntimeException {
+    assertThatThrownBy(() -> employeeManager.removeEmployee(9))
+        .isInstanceOf(NullPointerException.class)
+        .hasMessage("Employee with id 9 not found");
+  }
+
+  @Test
+  void testGetEmployeeCount_whenNoEmployeesAreInMap() {
+    employeeManager.removeEmployee(999);
+    assertThat(employeeManager.getEmployeeCount()).isEqualTo(0);
+  }
+
+  @Test
+  void testGetEmployeeCount_whenOneEmployeeIsInTheMap() {
+    assertThat(employeeManager.getEmployeeCount()).isEqualTo(1);
+  }
+}

lesson_15/tdd/tdd_app/src/test/java/JosephCaballero/EmployeeTest.java

@@ -0,0 +1,65 @@
+import static org.assertj.core.api.AssertionsForClassTypes.assertThat;
+
+import com.codedifferently.lesson15.Employee;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+
+class EmployeeTest {
+
+  Employee employee;
+
+  @BeforeEach
+  void setUp() {
+    employee = new Employee(9, "joseph", "Boxing", 1000000.5);
+  }
+
+  @Test
+  void testGetId() {
+    assertThat(employee.getId()).isEqualTo(9);
+  }
+
+  @Test
+  void testSetId_testingIfNumberOneWorks() {
+    employee.setId(1);
+    assertThat(employee.getId()).isEqualTo(1);
+  }
+
+  @Test
+  void testGetName() {
+    assertThat(employee.getName()).isEqualTo("joseph");
+  }
+
+  @Test
+  void testSetName_testingIfPabloWorks() {
+    employee.setName("Pablo");
+    assertThat(employee.getName()).isEqualTo("Pablo");
+  }
+
+  @Test
+  void testGetDepartment() {
+    assertThat(employee.getDepartment()).isEqualTo("Boxing");
+  }
+
+  @Test
+  void testSetDepartment_testingSettingDepartmentToHome() {
+    employee.setDepartment("Home");
+    assertThat(employee.getDepartment()).isEqualTo("Home");
+  }
+
+  @Test
+  void testGetSalary() {
+    assertThat(employee.getSalary()).isEqualTo(1000000.5);
+  }
+
+  @Test
+  void testSetSalary_salaryToTwoMillionAndFiveCents() {
+    employee.setSalary(2000000.5);
+    assertThat(employee.getSalary()).isEqualTo(2000000.5);
+  }
+
+  @Test
+  void testGetDetails_testsIfWeGetSpecificDetails() {
+    assertThat(employee.getDetails())
+        .isEqualTo("Employee: 9 joseph Department: Boxing Salary: 1000000.5");
+  }
+}