feat: add more tests to for the input command, Also adds comments to the code for easier reading

Author: Dlafferty251

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

@@ -22,6 +22,7 @@ public void setId(int id) {
     this.id = id;
   }
 
+  // Method to get all details of an employee
   public String getDetails() {
     return "The Employee's ID: "
         + id
@@ -33,6 +34,7 @@ public String getDetails() {
         + salary;
   }
 
+  // Method to set all employee details at once
   public void setDetails(String name, String department, double salary) {
     this.name = name;
     this.department = department;

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

@@ -1,6 +1,7 @@
 package com.codedifferently.lesson15;
 
 import static org.assertj.core.api.Assertions.assertThat;
+
 import org.junit.jupiter.api.Test;
 
 class Lesson15Test {
@@ -11,19 +12,19 @@ public void testLesson15() {
   }
 
   @Test
-  public void testGetGreeting() {
-    Lesson15 lesson15 = new Lesson15();
-    String result = lesson15.getGreeting();
-    String expected = "Hello, World!";
-    assertThat(result).isEqualTo(expected);
-  }
+  public void testGetGreeting() {}
 
+  // ------------
+  // Tests for Employee class
   @Test
   public void testGetDetails() {
+    // Create an instance of Employee by adding in temporary values
     Employee employee = new Employee(1, "John Doe", "Engineering", 50000);
-
+    // This will call the getDetails method and store that in a variable called result
     String result = employee.getDetails();
+    // This will create a string that we expect the getDetails method to return
     String expected = "The Employee's ID: 1 Name: John Doe Department: Engineering Salary: 50000.0";
+    // This will compare the result of the getDetails method to the expected string
     assertThat(result).isEqualTo(expected);
   }
 
@@ -52,12 +53,13 @@ public void testSetId() {
     assertThat(result).isEqualTo(2);
   }
 
-  @Test  
+  @Test
   public void testGetName() {
     Employee employee = new Employee(1, "John Doe", "Engineering", 50000);
     String result = employee.getName();
     assertThat(result).isEqualTo("John Doe");
   }
+
   @Test
   public void testSetName() {
     Employee employee = new Employee(1, "John Doe", "Engineering", 50000);
@@ -80,26 +82,32 @@ public void testSetDepartment() {
     String result = employee.getDepartment();
     assertThat(result).isEqualTo("Marketing");
   }
+
   @Test
   public void testGetSalary() {
     Employee employee = new Employee(1, "John Doe", "Engineering", 50000);
     double result = employee.getSalary();
     assertThat(result).isEqualTo(50000);
   }
+
   @Test
   public void testSetSalary() {
     Employee employee = new Employee(1, "John Doe", "Engineering", 50000);
     employee.setSalary(60000);
     double result = employee.getSalary();
     assertThat(result).isEqualTo(60000);
   }
+
+  // -------
+  // Tests for EmployeeManager class
   @Test
   public void testAddEmployee() {
     EmployeeManager employeeManager = new EmployeeManager();
     Employee employee = new Employee(1, "John Doe", "Engineering", 50000);
     employeeManager.addEmployee(employee);
     assertThat(employeeManager.getEmployeeCount()).isEqualTo(1);
   }
+
   @Test
   public void testGetEmployee() {
     EmployeeManager employeeManager = new EmployeeManager();
@@ -108,6 +116,7 @@ public void testGetEmployee() {
     Employee result = employeeManager.getEmployee(1);
     assertThat(result).isEqualTo(employee);
   }
+
   @Test
   public void testUpdateEmployee() {
     EmployeeManager employeeManager = new EmployeeManager();
@@ -122,12 +131,33 @@ public void testUpdateEmployee() {
   @Test
   public void testRemoveEmployee() {
     EmployeeManager employeeManager = new EmployeeManager();
-    Employee employee = new  Employee(1, "John Doe", "Engineering", 50000);
+    Employee employee = new Employee(1, "John Doe", "Engineering", 50000);
     employeeManager.addEmployee(employee);
     employeeManager.removeEmployee(1);
     assertThat(employeeManager.getEmployeeCount()).isEqualTo(0);
   }
 
   @Test
-  public void test
+  public void testEmployeeCount() {
+    EmployeeManager employeeManager = new EmployeeManager();
+    Employee employee1 = new Employee(1, "John Doe", "Engineering", 50000);
+    Employee employee2 = new Employee(2, "Jane Doe", "Marketing", 60000);
+    employeeManager.addEmployee(employee1);
+    employeeManager.addEmployee(employee2);
+    int result = employeeManager.getEmployeeCount();
+    assertThat(result).isEqualTo(2);
+  }
+
+  @Test
+  public void assertEmployeeInCollection() {
+    EmployeeManager employeeManager = new EmployeeManager();
+    Employee employee = new Employee(1, "John Doe", "Engineering", 50000);
+    employeeManager.addEmployee(employee);
+    try {
+      employeeManager.getEmployee(1);
+    } catch (IllegalArgumentException e) {
+      // This should not happen
+      assertThat(e.getMessage()).isEqualTo("Employee does not in collection with id 1");
+    }
+  }
 }