Skip to content

Commit 4a93ee2

Browse files
committed
chore: fixed one of two failing tests
1 parent 8073d85 commit 4a93ee2

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ private void assertEmployeeInCollection(int id) {
3434
if (this.employees.containsKey(id)) {
3535
return;
3636
}
37-
throw new IllegalArgumentException("Employee does not in collection with id " + id);
37+
throw new IllegalArgumentException("Employee does not exist in collection with Id " + id);
3838
}
3939

4040
public int getEmployeeCount() {

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

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,8 @@ void testUpdateEmployee() {
5050

5151
@Test
5252
void testRemoveEmployee() {
53+
// Arrange
54+
employeeManager.addEmployee(new Employee(1, "Frankenstein", "Development", 0.0));
5355
// Act
5456
employeeManager.removeEmployee(1);
5557
Employee removedEmployee = employeeManager.getEmployee(1);
@@ -63,7 +65,7 @@ void testAssertEmployeeInCollection() {
6365
// Arrange & Act & Assert
6466
assertThatThrownBy(() -> employeeManager.getEmployee(52))
6567
.isInstanceOf(IllegalArgumentException.class)
66-
.hasMessage("Employee does not exist in collection with Id 1");
68+
.hasMessage("Employee does not exist in collection with Id 52");
6769
}
6870

6971
@Test

0 commit comments

Comments
 (0)