Skip to content

Commit 6c813fa

Browse files
committed
refactor: fixed getDetails method in Employee class
1 parent a50d1c1 commit 6c813fa

File tree

2 files changed

+12
-2
lines changed

2 files changed

+12
-2
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,6 @@ public void setSalary(double salary) {
4747
}
4848

4949
public String getDetails() {
50-
return "ID: " + this.id + " Name: " + this.name + " Department: " + this.department + " Salary " + this.salary;
50+
return "ID: " + id + " Name: " + name + " Department: " + department + " Salary " + salary;
5151
}
5252
}

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

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
import static org.assertj.core.api.Assertions.assertThat;
44
import static org.junit.jupiter.api.Assertions.assertEquals;
5+
56
import org.junit.jupiter.api.Test;
67

78
class Lesson15Test {
@@ -123,6 +124,15 @@ public void testGetEmplopyeeCount() {
123124
public void testGetDetails() {
124125
Employee employee1 = new Employee(2004, "Edmar", "Tech", 60000.0);
125126

126-
assertEquals("ID: " + employee1.getId() + " Name: " + employee1.getName() + " Department: " + employee1.getDepartment() + " Salary " + employee1.getSalary(), employee1.getDetails());
127+
assertEquals(
128+
"ID: "
129+
+ employee1.getId()
130+
+ " Name: "
131+
+ employee1.getName()
132+
+ " Department: "
133+
+ employee1.getDepartment()
134+
+ " Salary "
135+
+ employee1.getSalary(),
136+
employee1.getDetails());
127137
}
128138
}

0 commit comments

Comments
 (0)