Skip to content

Commit

Permalink
Fix checkstyle violations
Browse files Browse the repository at this point in the history
  • Loading branch information
Fallman2 committed Nov 2, 2023
1 parent 88c21a2 commit 962b40a
Show file tree
Hide file tree
Showing 9 changed files with 34 additions and 26 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@

import seedu.address.logic.commands.AddTaskCommand;
import seedu.address.logic.parser.exceptions.ParseException;
import seedu.address.model.event.exceptions.InvalidDescriptionException;
import seedu.address.model.task.Task;

/**
Expand Down
1 change: 0 additions & 1 deletion src/main/java/seedu/address/model/task/Task.java
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@
import java.util.Comparator;

import seedu.address.commons.util.ToStringBuilder;
import seedu.address.model.event.exceptions.InvalidDescriptionException;

/**
* Represents a task in the task list.
Expand Down
3 changes: 3 additions & 0 deletions src/main/java/seedu/address/model/task/TaskList.java
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,9 @@ public void remove(Task toRemove) {
}
}

/**
* Removes a task at specified index from the list and returns the removed task.
*/
public Task remove(int index) {
requireNonNull(index);

Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,9 @@
package seedu.address.logic.commands;

import static java.util.Objects.requireNonNull;
import static org.junit.jupiter.api.Assertions.*;
import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertTrue;
import static seedu.address.logic.commands.CommandTestUtil.VALID_END_DATE_LATER;
import static seedu.address.testutil.Assert.assertThrows;

Expand All @@ -13,8 +15,9 @@
import java.util.List;
import java.util.function.Predicate;

import javafx.collections.ObservableList;
import org.junit.jupiter.api.Test;

import javafx.collections.ObservableList;
import seedu.address.commons.core.GuiSettings;
import seedu.address.logic.Messages;
import seedu.address.logic.commands.exceptions.CommandException;
Expand Down Expand Up @@ -270,4 +273,4 @@ public void addTask(Task task) {
tasksAdded.add(task);
}
}
}
}
1 change: 1 addition & 0 deletions src/test/java/seedu/address/model/task/DeadlineTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
import java.time.format.DateTimeFormatter;

import org.junit.jupiter.api.Test;

import seedu.address.model.event.exceptions.InvalidDeadlineException;

class DeadlineTest {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -58,4 +58,4 @@ public void compareToTest() {
assertEquals(-1, descriptionA.compareTo(descriptionB));
assertEquals(0, descriptionA.compareTo(descriptionA));
}
}
}
19 changes: 9 additions & 10 deletions src/test/java/seedu/address/model/task/TaskListTest.java
Original file line number Diff line number Diff line change
@@ -1,22 +1,21 @@
package seedu.address.model.task;


import org.junit.jupiter.api.Test;
import seedu.address.model.person.UniquePersonList;
import seedu.address.model.task.exceptions.DuplicateTaskException;
import seedu.address.model.task.exceptions.TaskNotFoundException;

import java.util.Arrays;
import java.util.Collections;
import java.util.List;

import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertTrue;
import static seedu.address.testutil.Assert.assertThrows;
import static seedu.address.testutil.TypicalTasks.ASSIGNMENT;
import static seedu.address.testutil.TypicalTasks.HYDRATION;

import java.util.Arrays;
import java.util.Collections;
import java.util.List;

import org.junit.jupiter.api.Test;

import seedu.address.model.task.exceptions.DuplicateTaskException;
import seedu.address.model.task.exceptions.TaskNotFoundException;

class TaskListTest {
private final TaskList taskList = new TaskList();

Expand Down
14 changes: 9 additions & 5 deletions src/test/java/seedu/address/model/task/TaskManagerTest.java
Original file line number Diff line number Diff line change
@@ -1,12 +1,16 @@
package seedu.address.model.task;

import org.junit.jupiter.api.Test;
import seedu.address.model.task.exceptions.DuplicateTaskException;

import static org.junit.jupiter.api.Assertions.*;
import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertTrue;
import static seedu.address.testutil.Assert.assertThrows;
import static seedu.address.testutil.TypicalTasks.ASSIGNMENT;
import static seedu.address.testutil.TypicalTasks.HYDRATION;

import org.junit.jupiter.api.Test;

import seedu.address.model.task.exceptions.DuplicateTaskException;

class TaskManagerTest {
private final TaskManager taskManager = new TaskManager();

Expand Down Expand Up @@ -56,4 +60,4 @@ public void equalsMethod() {

assertFalse(taskManager.equals(notTaskManager));
}
}
}
10 changes: 5 additions & 5 deletions src/test/java/seedu/address/model/task/TaskTest.java
Original file line number Diff line number Diff line change
@@ -1,17 +1,17 @@
package seedu.address.model.task;

import org.junit.jupiter.api.Test;

import java.time.LocalDateTime;
import java.util.Comparator;

import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertNotEquals;
import static org.junit.jupiter.api.Assertions.assertTrue;
import static seedu.address.model.task.Task.DATE_TIME_STRING_FORMATTER;
import static seedu.address.testutil.Assert.assertThrows;

import java.time.LocalDateTime;
import java.util.Comparator;

import org.junit.jupiter.api.Test;

class TaskTest {
private static final String VALID_DESCRIPTION_STRING = "Eat Nuts";
private static final TaskDescription VALID_TASK_DESCRIPTION = new TaskDescription(VALID_DESCRIPTION_STRING);
Expand Down

0 comments on commit 962b40a

Please sign in to comment.