Changed Tests so they don't permanently insert workers and tasks into the database
This commit is contained in:
@@ -19,6 +19,7 @@ public class BadCasesTest {
|
||||
public void assertNewTasksAreNotNull() {
|
||||
int workerId = manager.createWorker("Alfred");
|
||||
assertThrows(IllegalArgumentException.class, () -> manager.addTask(workerId, null, null));
|
||||
manager.removeWorker(workerId);
|
||||
}
|
||||
@Test
|
||||
@DisplayName("Assert List isn't empty after adding a task")
|
||||
@@ -26,6 +27,8 @@ public class BadCasesTest {
|
||||
int workerId = manager.createWorker("Alfred");
|
||||
int taskId = manager.addTask(workerId, "Run", "Jog 10 Miles");
|
||||
assertFalse(manager.getTaskMap().isEmpty());
|
||||
manager.removeWorker(workerId);
|
||||
manager.removeTask(taskId);
|
||||
}
|
||||
@Test
|
||||
@DisplayName("Assert only existing tasks can be edited")
|
||||
@@ -33,6 +36,7 @@ public class BadCasesTest {
|
||||
int workerId = manager.createWorker("Alfred");
|
||||
assertThrows(IllegalArgumentException.class, () -> manager.editTask(workerId, 99969, "I", "am Illegal"));
|
||||
assertThrows(IllegalArgumentException.class, () -> manager.editTask(22200, 99969, "I", "am Illegal"));
|
||||
manager.removeWorker(workerId);
|
||||
}
|
||||
@Test
|
||||
@DisplayName("Assert Add Task is programmed defensively")
|
||||
@@ -40,6 +44,7 @@ public class BadCasesTest {
|
||||
int workerId = manager.createWorker("Alfred");
|
||||
assertThrows(IllegalArgumentException.class, () -> manager.addTask( 20203,"I", "am Illegal"));
|
||||
assertThrows(IllegalArgumentException.class, () -> manager.addTask( workerId,null, null));
|
||||
manager.removeWorker(workerId);
|
||||
}
|
||||
@Test
|
||||
@DisplayName("Assert that Remove Task is programmed defensively")
|
||||
@@ -67,5 +72,7 @@ public class BadCasesTest {
|
||||
} catch (Exception e) {
|
||||
fail("Should not crash just because a command is invalid");
|
||||
}
|
||||
manager.removeWorker(workerId);
|
||||
manager.removeTask(taskId);
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user