Skip to content

Commit ce865a1

Browse files
author
jjcapparell
committed
added tests to get 89% coverage
1 parent 4adb80b commit ce865a1

File tree

1 file changed

+91
-2
lines changed

1 file changed

+91
-2
lines changed

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

Lines changed: 91 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
package com.codedifferently.lesson15;
22

33
import static org.junit.jupiter.api.Assertions.assertEquals;
4-
54
import org.junit.jupiter.api.BeforeEach;
65
import org.junit.jupiter.api.Test;
76

@@ -12,43 +11,133 @@ class Lesson15Test {
1211

1312
@BeforeEach
1413
public void setUp() {
14+
// Arrange
1515
employeeManager = new EmployeeManager();
16-
employee = new Employee(1, "Wall E", "Inbound", 30000.00);
16+
employee = new Employee(1, "Wall-E", "Inbound", 30000.00);
17+
}
18+
19+
@Test
20+
public void testGetId() {
21+
employee.getId();
22+
// Assert
23+
assertEquals(employee.getId(), 1);
24+
}
25+
26+
@Test
27+
public void testSetId() {
28+
employee.setId(2);
29+
//Assert
30+
assertEquals(employee.getId(), 2);
31+
}
32+
33+
@Test
34+
public void testGetName() {
35+
employee.getName();
36+
//Assert
37+
assertEquals(employee.getName(), "Wall-E");
38+
}
39+
40+
@Test
41+
public void testSetName() {
42+
employee.setName("BURN-E");
43+
// Assert
44+
assertEquals(employee.getName(), "BURN-E");
45+
}
46+
47+
@Test
48+
public void testGetDepartment() {
49+
employee.setName("BURN-E");
50+
// Assert
51+
assertEquals(employee.getDepartment(), "Inbound");
52+
}
53+
54+
@Test
55+
public void testSetDepartment() {
56+
employee.setDepartment("NONE");
57+
// Assert
58+
assertEquals(employee.getDepartment(), "NONE");
59+
}
60+
61+
@Test
62+
public void testGetSalary() {
63+
employee.getSalary();
64+
// Assert
65+
assertEquals(employee.getSalary(), 30000.00);
66+
}
67+
68+
@Test
69+
public void testSetSalary() {
70+
employee.setSalary(100);
71+
// Assert
72+
assertEquals(employee.getSalary(), 100);
1773
}
1874

1975
@Test
2076
public void testAddEmployee() {
77+
// Act
2178
employeeManager.addEmployee(employee);
79+
// Assert
2280
assertEquals(1, employeeManager.getEmployeeCount());
2381
assertEquals(employee, employeeManager.getEmployee(1));
2482
}
2583

2684
@Test
2785
public void testGetEmployee() {
86+
// Act
2887
employeeManager.addEmployee(employee);
2988
Employee retrieved = employeeManager.getEmployee(1);
89+
// Assert
3090
assertEquals(employee, retrieved);
3191
}
3292

3393
@Test
3494
public void testUpdateEmployee() {
95+
// Act
3596
employeeManager.addEmployee(employee);
3697
Employee updatedEmployee = new Employee(1, "EVE", "Inbound", 80000.00);
3798
employeeManager.updateEmployee(updatedEmployee);
99+
// Assert
38100
assertEquals(updatedEmployee, employeeManager.getEmployee(1));
39101
}
40102

41103
@Test
42104
public void testRemoveEmployee() {
105+
// Act
43106
employeeManager.addEmployee(employee);
44107
employeeManager.removeEmployee(1);
108+
// Assert
45109
assertEquals(0, employeeManager.getEmployeeCount());
46110
}
47111

48112
@Test
49113
public void testGetEmployeeCount() {
114+
// Assert
50115
assertEquals(0, employeeManager.getEmployeeCount());
116+
// Act
51117
employeeManager.addEmployee(employee);
118+
// Assert
52119
assertEquals(1, employeeManager.getEmployeeCount());
53120
}
121+
122+
@Test
123+
public void testGetDetails() {
124+
// Assert
125+
assertEquals(
126+
"ID: 1, Name: Wall-E, Department: Inbound, Salary: 30000.0", employee.getDetails());
127+
}
128+
129+
@Test
130+
public void testMultipleEmployees() {
131+
// Arrange
132+
Employee emp2 = new Employee(2, "Captain B. McCrea", "Captain", 100000.00);
133+
Employee emp3 = new Employee(3, "AUTO", "Pilot", 95000.00);
134+
// Act
135+
employeeManager.addEmployee(employee);
136+
employeeManager.addEmployee(emp2);
137+
employeeManager.addEmployee(emp3);
138+
// Assert
139+
assertEquals(3, employeeManager.getEmployeeCount());
140+
assertEquals(emp2, employeeManager.getEmployee(2));
141+
assertEquals(emp3, employeeManager.getEmployee(3));
142+
}
54143
}

0 commit comments

Comments
 (0)