Skip to content

Commit 47e6270

Browse files
author
“Tezz03”
committed
fix: updated test to pass
1 parent ceb16b5 commit 47e6270

File tree

3 files changed

+73
-28
lines changed

3 files changed

+73
-28
lines changed

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

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,8 @@ public void setSalary(double salary) {
5151
public String getDetails() {
5252
return details;
5353
}
54+
5455
public void setDetail(String detail) {
55-
this.details = details;
56+
this.details = details;
5657
}
5758
}
Lines changed: 49 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,49 @@
1+
package com.codedifferently.lesson15;
2+
3+
import static org.assertj.core.api.Assertions.assertThat;
4+
5+
import org.junit.jupiter.api.BeforeEach;
6+
import org.junit.jupiter.api.Test;
7+
8+
public class EmployeeManagerTest {
9+
10+
EmployeeManager manager;
11+
12+
@BeforeEach
13+
void setup() {
14+
manager = new EmployeeManager();
15+
}
16+
17+
@Test
18+
public void addEmployeeToMap() {
19+
Employee employee = new Employee(1, "M", "Tech", 50000, "Non Hispanic Black");
20+
manager.addEmployee(employee);
21+
22+
// Assert that the employee was added correctly
23+
assertThat(manager.getEmployee(1)).isEqualTo(employee);
24+
}
25+
26+
@Test
27+
public void getEmployee() {
28+
Employee employee = new Employee(1, "M", "Tech", 50000, "Non Hispanic Black");
29+
manager.addEmployee(employee);
30+
assertThat(manager.getEmployee(1)).isEqualTo(employee);
31+
}
32+
33+
@Test
34+
public void updateEmployee() {
35+
Employee origin = new Employee(1, "John", "Amazon", 1, "Detail");
36+
manager.addEmployee(origin);
37+
Employee updated = new Employee(1, "Mike", "Netflix", 1, "Detail");
38+
manager.updateEmployee(updated);
39+
assertThat(manager.getEmployee(1)).isEqualTo(updated);
40+
}
41+
42+
@Test
43+
public void removeEmployee() {
44+
Employee employee1 = new Employee(3, "Jason", "Apple", 3, "Detail");
45+
manager.addEmployee(employee1);
46+
manager.removeEmployee(3);
47+
assertThat(manager.getEmployeeCount()).isEqualTo(0);
48+
}
49+
}
Lines changed: 22 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package com.codedifferently.lesson15;
22

33
import static org.junit.jupiter.api.Assertions.assertEquals;
4+
45
import org.junit.jupiter.api.Test;
56

67
class EmployeeTest {
@@ -15,97 +16,91 @@ public void getId() {
1516
assertEquals(expectedId, actual);
1617
}
1718

18-
19-
@Test
19+
@Test
2020
public void setId() {
2121
int expectedId = 7;
2222

2323
Employee employee1 = new Employee(7, null, null, 0, null);
2424

2525
employee1.setId(expectedId);
2626

27-
assertEquals( expectedId, employee1.getId());
27+
assertEquals(expectedId, employee1.getId());
2828
}
2929

30-
31-
@Test
30+
@Test
3231
public void getName() {
3332
var expectedNameString = "Montez";
3433

3534
Employee employee1 = new Employee(0, "Montez", "", 0, null);
3635

3736
var actual = employee1.getName();
3837

39-
assertEquals( expectedNameString, actual);
38+
assertEquals(expectedNameString, actual);
4039
}
4140

42-
43-
@Test
41+
@Test
4442
public void setName() {
4543
var expectedNameString = "Montez";
4644

4745
Employee employee1 = new Employee(0, "Montez", "", 0, null);
4846

4947
employee1.setName(expectedNameString);
5048

51-
assertEquals( expectedNameString, employee1.getName());
49+
assertEquals(expectedNameString, employee1.getName());
5250
}
5351

54-
@Test
52+
@Test
5553
public void getDepartment() {
5654
var expectedString = "Tech";
5755

58-
Employee employee1 = new Employee(0, null, "Tech", 0, null );
56+
Employee employee1 = new Employee(0, null, "Tech", 0, null);
5957

6058
var actual = employee1.getDepartment();
6159

62-
assertEquals( expectedString, actual);
60+
assertEquals(expectedString, actual);
6361
}
6462

65-
@Test
63+
@Test
6664
public void setDepartment() {
6765
var expectedDepartment = "Sales";
6866

6967
Employee employee1 = new Employee(0, null, "Tech", 0, null);
7068

7169
employee1.setDepartment("Sales");
7270

73-
assertEquals( expectedDepartment, employee1.getDepartment());
71+
assertEquals(expectedDepartment, employee1.getDepartment());
7472
}
7573

76-
@Test
74+
@Test
7775
public void getSalary() {
7876
var expectedSalary = 50000;
7977

80-
Employee employye2 = new Employee( 0, null, null, expectedSalary, null);
78+
Employee employye2 = new Employee(0, null, null, expectedSalary, null);
8179

8280
var actual = employye2.getSalary();
8381

84-
assertEquals( expectedSalary, actual);
82+
assertEquals(expectedSalary, actual);
8583
}
8684

87-
@Test
85+
@Test
8886
public void setSalary() {
8987
int expectedSalary = 4200;
9088

9189
Employee employee1 = new Employee(0, null, null, expectedSalary, null);
9290

9391
employee1.setSalary(expectedSalary);
9492

95-
assertEquals( expectedSalary, employee1.getSalary());
93+
assertEquals(expectedSalary, employee1.getSalary());
9694
}
9795

9896
@Test
99-
public void setDetails() {
100-
var expectedDetailString = "Non Hipanic Black";
97+
public void getDetails() {
98+
var expectedString = "Detail";
10199

102-
Employee employee = new Employee(0, null, null, 0, "Non Hispanic Black");
100+
Employee employee1 = new Employee(0, null, null, 0, "Detail");
103101

104-
var actual = employee.getDetails();
102+
var actual = employee1.getDetails();
105103

106-
assertEquals(expectedDetailString, actual);
104+
assertEquals(expectedString, actual);
107105
}
108106
}
109-
110-
111-

0 commit comments

Comments
 (0)