feat: added getDetails method to Employee.java; created corresponding test for getDetails; added appropriate import statements to EmployeeManagerTest; added @beforeeach set up for EmployeeManagerTest; added test for each EmployeeManager method; added throw assert to testRemoveEmployee checking for an IllegalArgumentException and corresponding message; fix: corrected naming convention for EmployeeTest tests;

Author: “A1-4U2T1NN”

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

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

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

@@ -1,52 +1,73 @@
 package com.codedifferently.lesson15;
 
+import java.util.ArrayList;
+import java.util.List;
+
+import static org.assertj.core.api.Assertions.assertThatThrownBy;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
 public class EmployeeManagerTest {
 
-    EmployeeTest employeeTest;
-
-    @Test
-    public void testaddEmployee() {
-        // Arrange
-        
-        //Act
-
-        //Assert
-        
-    }
+    EmployeeManager employeeManager;
+    List<Employee> employees;
 
-    @Test
-    public void test() {
-        // tests if changed id matches new id
+    //Set up
+    @BeforeEach
+    void setUp() {
+        employeeManager = new EmployeeManager();
+        Employee employee = new Employee(0, "Nitsua", "Sales", 65000.00);
+        employeeManager.addEmployee(employee);
+        employees = new ArrayList();
+        employees.add(employee);
     }
-
     @Test
-    public void getNameTest() {
-        // tests if changed name matches name
-    }
+    public void testAddEmployee() {
+        //Arrange
+        Employee employee1 = new Employee(1, "Penny", "Sales", 65000.00);
+        //Act & Assert
+        employeeManager.addEmployee(employee1);
+    }   
 
     @Test
-    public void setNameTest() {
-        // tests if changed id matches new id
-    }
-    @Test
-    public void getDepartmentTest() {
-        
+    public void testGetEmployee() {
+        //Arrange
+        Employee employee = new Employee(0, "Nitsua", "Sales", 65000.00);
+        Employee expectedEmployee = employeeManager.getEmployee(employee.getId());
+        //Act
+        Employee actualEmployee = employeeManager.getEmployee(0);
+        // Assert
+        assertEquals(expectedEmployee, actualEmployee);
     }
 
     @Test
-    public void setDepartmentTest() {
-        
+    public void testUpdateEmployee() {
+        //Arrange
+        Employee expectedEmployeeUpdate = new Employee(0, "Nitsua", "Tech", 100000.00); 
+        //Act
+        employeeManager.updateEmployee(expectedEmployeeUpdate);
+        //Act
+        assertEquals(expectedEmployeeUpdate, employeeManager.getEmployee(0));
     }
 
     @Test
-    public void getSalaryTest() {
-        
+    public void testRemoveEmployee() {
+        //Arange
+        Employee employee = new Employee(0, "Nitsua", "Sales", 65000.00);
+        employeeManager.addEmployee(employee);
+        //Act
+        employeeManager.removeEmployee(0);
+        //Assert
+        assertThatThrownBy(() -> employeeManager.removeEmployee(0))
+        .isInstanceOf(IllegalArgumentException.class)
+        .hasMessage("Employee does not in collection with id " + employee.getId());
     }
-
     @Test
-    public void setSalaryTest() {
-        
+    public void testGetEmployeeCount() {
+        //Act
+        int actualEmployeeCount = employeeManager.getEmployeeCount();
+        //Assert(recieved help from Vicente on employees array)
+        assertEquals(employees.size(), actualEmployeeCount);
     }
 }

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

@@ -15,7 +15,7 @@ void setUp() {
     }
 
     @Test
-    //Method beign tested
+    //Method being tested
     public void testGetId() {
         //Execersise
         int actualId = employee.getId();
@@ -37,10 +37,10 @@ public void testSetID() {
 
     @Test
     public void testGetName() {
-        //Act
-        String actualName = employee.getName();
         //Arrange
         String expectedName = "Chigazo";
+        //Act
+        String actualName = employee.getName();
         //Assert
         assertEquals(expectedName, actualName);
     }
@@ -56,18 +56,18 @@ public void testSetName() {
     }
 
     @Test
-    public void getDepartmentTest() {
-        //Act
-        String expectedDepartment = employee.getDepartment();
+    public void testGetDepartment() {
         //Arrange
         String actualDepartment = "Finance";
+        //Act
+        String expectedDepartment = employee.getDepartment();
         //Assert
         assertEquals(expectedDepartment, actualDepartment);
     }
 
     @Test
-    public void setDepartmentTest() {
-         //Arrange
+    public void testSetDepartment() {
+        //Arrange
         String expectedSetDepartment = "Tech";
         //Act
         employee.setDepartment(expectedSetDepartment);
@@ -76,7 +76,7 @@ public void setDepartmentTest() {
     }
 
     @Test
-    public void getSalaryTest() {
+    public void testGetSalary() {
         //Arrange
         double actualSalary = 50000.00;
         //Act
@@ -86,13 +86,27 @@ public void getSalaryTest() {
     }
 
     @Test
-    public void setSalaryTest() {
+    public void testSetSalary() {
         //Arrange
         double expectedSetSalary = 100000.00;
         //Act
         employee.setSalary(expectedSetSalary);
         //Assert
         assertEquals(expectedSetSalary, employee.getSalary());
     }
+
+    @Test
+    public void testGetDetails() {
+        //Arrange
+        String actualDetails = 
+        "id: " + employee.getId()
+         + "name: " + employee.getName()
+         + "department: " + employee.getDepartment()
+         + "salary: " + employee.getSalary();
+        //Act
+        String expectedDetails = employee.getDetails();
+        //Assert
+        assertEquals(expectedDetails, actualDetails);
+    }
 }