Skip to content

Commit 50f806f

Browse files
committed
Feat: fix EmployeeManager test
1 parent 91d9698 commit 50f806f

File tree

1 file changed

+18
-21
lines changed

1 file changed

+18
-21
lines changed

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

Lines changed: 18 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,11 @@
1-
import java.util.HashMap;
2-
import java.util.Map;
3-
41
import static org.junit.jupiter.api.Assertions.assertEquals;
5-
import org.junit.jupiter.api.BeforeEach;
6-
import org.junit.jupiter.api.Test;
72

83
import com.codedifferently.lesson15.Employee;
94
import com.codedifferently.lesson15.EmployeeManager;
5+
import java.util.HashMap;
6+
import java.util.Map;
7+
import org.junit.jupiter.api.BeforeEach;
8+
import org.junit.jupiter.api.Test;
109

1110
public class EmployeeManagerTest {
1211
Employee employee;
@@ -42,47 +41,45 @@ void testUpdateEmployee() {
4241
employees.put(employee.getId(), employee);
4342

4443
employeeNew = new Employee(15, "Paul", "President", 200);
45-
44+
4645
employees.put(employeeNew.getId(), employeeNew);
47-
46+
4847
employeeManager.addEmployee(employee);
49-
48+
5049
employeeManager.updateEmployee(employeeNew);
51-
50+
5251
assertEquals(
5352
employeeManager.getEmployee(employeeNew.getId()), employees.get(employeeNew.getId()));
5453
}
5554

5655
@Test
5756
void testRemoveEmployee() {
5857
employees.put(employee.getId(), employee);
59-
58+
6059
employees.remove(employee.getId(), employee);
61-
60+
6261
employeeManager.addEmployee(employee);
63-
62+
6463
employeeManager.removeEmployee(employee.getId());
65-
64+
6665
assertEquals(0, employeeManager.getEmployeeCount());
6766
}
6867

6968
@Test
70-
void testAssertEmployeeInCollection() throws IllegalArgumentException {
71-
72-
}
69+
void testAssertEmployeeInCollection() throws IllegalArgumentException {}
7370

7471
@Test
7572
void testGetEmployeeCount() {
7673
employees.put(employee.getId(), employee);
77-
74+
7875
employeeNew = new Employee(16, "Paul", "President", 200);
79-
76+
8077
employees.put(employeeNew.getId(), employeeNew);
81-
78+
8279
employeeManager.addEmployee(employee);
83-
80+
8481
employeeManager.addEmployee(employeeNew);
85-
82+
8683
assertEquals(2, employeeManager.getEmployeeCount());
8784
}
8885
}

0 commit comments

Comments
 (0)