Quick change to the access Modifiers for fields in Worker,Task and Assignmentmanager, refactored tests to use getWorkerMap
This commit is contained in:
@@ -69,9 +69,9 @@ public class GoodCasesTest {
|
||||
public void assertGettersWorkCorrectly() {
|
||||
int workerId = manager.createWorker("Alfred");
|
||||
int taskId = manager.addTask(workerId, "Run", "Jog 10 Miles");
|
||||
assertEquals("Alfred", manager.workerMap.get(workerId).getName());
|
||||
assertEquals("Alfred", manager.getWorkerMap().get(workerId).getName());
|
||||
//This one may be somewhat nonsensical, but it ensures the getId works so one may iterate over the workerMap
|
||||
assertEquals(manager.workerMap.get(workerId).getId(), workerId);
|
||||
assertEquals(manager.getWorkerMap().get(workerId).getId(), workerId);
|
||||
assertEquals(manager.getTask(taskId).getTaskId(), taskId);
|
||||
assertEquals(manager.getTask(taskId).getWorkerId(), workerId);
|
||||
|
||||
@@ -150,6 +150,6 @@ public class GoodCasesTest {
|
||||
public void assertRemoveWorkerActuallyRemoves() {
|
||||
int workerId = manager.createWorker("Alfred");
|
||||
manager.removeWorker(workerId);
|
||||
assertThrows(IllegalArgumentException.class, () -> manager.workerMap.containsKey(workerId));
|
||||
assertThrows(IllegalArgumentException.class, () -> manager.getWorkerMap().containsKey(workerId));
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user