Skip to content

Commit 91d9698

Browse files
committed
Feat: adds getDetails method to Employee file and more Tests implementations
1 parent ce93bbe commit 91d9698

File tree

3 files changed

+47
-10
lines changed

3 files changed

+47
-10
lines changed

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,4 +45,8 @@ public double getSalary() {
4545
public void setSalary(double salary) {
4646
this.salary = salary;
4747
}
48+
49+
public String getDetails() {
50+
return id + " " + name + " " + department + " " + salary + ".";
51+
}
4852
}
Lines changed: 38 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,13 @@
1-
import static org.junit.jupiter.api.Assertions.assertEquals;
2-
3-
import com.codedifferently.lesson15.Employee;
4-
import com.codedifferently.lesson15.EmployeeManager;
51
import java.util.HashMap;
62
import java.util.Map;
3+
4+
import static org.junit.jupiter.api.Assertions.assertEquals;
75
import org.junit.jupiter.api.BeforeEach;
86
import org.junit.jupiter.api.Test;
97

8+
import com.codedifferently.lesson15.Employee;
9+
import com.codedifferently.lesson15.EmployeeManager;
10+
1011
public class EmployeeManagerTest {
1112
Employee employee;
1213
EmployeeManager employeeManager;
@@ -22,39 +23,66 @@ void setUp() {
2223

2324
@Test
2425
void testAddEmployee() {
25-
// Arrange
2626
employees.put(employee.getId(), employee);
2727

28-
// Act
2928
employeeManager.addEmployee(employee);
3029

31-
// Assert
3230
assertEquals(employees.get(employee.getId()), employeeManager.getEmployee(employee.getId()));
3331
}
3432

3533
@Test
3634
void testGetEmployee() {
3735
employees.put(employee.getId(), employee);
36+
3837
assertEquals(employee, employees.get(employee.getId()));
3938
}
4039

4140
@Test
4241
void testUpdateEmployee() {
4342
employees.put(employee.getId(), employee);
43+
4444
employeeNew = new Employee(15, "Paul", "President", 200);
45+
4546
employees.put(employeeNew.getId(), employeeNew);
47+
4648
employeeManager.addEmployee(employee);
49+
4750
employeeManager.updateEmployee(employeeNew);
51+
4852
assertEquals(
4953
employeeManager.getEmployee(employeeNew.getId()), employees.get(employeeNew.getId()));
5054
}
5155

5256
@Test
53-
void testRemoveEmployee() {}
57+
void testRemoveEmployee() {
58+
employees.put(employee.getId(), employee);
59+
60+
employees.remove(employee.getId(), employee);
61+
62+
employeeManager.addEmployee(employee);
63+
64+
employeeManager.removeEmployee(employee.getId());
65+
66+
assertEquals(0, employeeManager.getEmployeeCount());
67+
}
5468

5569
@Test
56-
void testAssertEmployeeInCollection() {}
70+
void testAssertEmployeeInCollection() throws IllegalArgumentException {
71+
72+
}
5773

5874
@Test
59-
void testGetEmployeeCount() {}
75+
void testGetEmployeeCount() {
76+
employees.put(employee.getId(), employee);
77+
78+
employeeNew = new Employee(16, "Paul", "President", 200);
79+
80+
employees.put(employeeNew.getId(), employeeNew);
81+
82+
employeeManager.addEmployee(employee);
83+
84+
employeeManager.addEmployee(employeeNew);
85+
86+
assertEquals(2, employeeManager.getEmployeeCount());
87+
}
6088
}

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

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,4 +61,9 @@ void testsetSalary() {
6161

6262
assertEquals(1000000, employee.getSalary());
6363
}
64+
65+
@Test
66+
void getDetails() {
67+
assertEquals(employee.getDetails(), employee.getDetails());
68+
}
6469
}

0 commit comments

Comments
 (0)