Resolve Fallman2 review comments

Resolved all comments except for:
-ParserUtil lines 136-139
-ParserUtil lines 154-157

src/main/java/seedu/address/logic/LogicManager.java

@@ -39,7 +39,7 @@ public class LogicManager implements Logic {
     public LogicManager(Model model, Storage storage) {
         this.model = model;
         this.storage = storage;
-        this.uniMateParser = new UniMateParser();
+        uniMateParser = new UniMateParser();
     }
 
     @Override

src/main/java/seedu/address/logic/parser/ParserUtil.java

@@ -135,7 +135,7 @@ public static Set<Tag> parseTags(Collection<String> tags) throws ParseException
     public static EventDescription parseEventDescription(String description) throws ParseException {
         requireNonNull(description);
         String trimmedDescription = description.trim();
-        if (trimmedDescription.isEmpty()) {
+        if (!EventDescription.isValid(description)) {
             throw new ParseException(EventDescription.MESSAGE_CONSTRAINTS);
         }
         return new EventDescription(trimmedDescription);

src/main/java/seedu/address/logic/parser/UniMateParser.java

@@ -21,17 +21,23 @@
 import seedu.address.logic.parser.exceptions.ParseException;
 
 /**
- * Parses user input for address book component of application.
+ * Parses user input for UniMate.
  */
-public class UniMateParser extends ComponentParser {
+public class UniMateParser {
 
     /**
      * Used for initial separation of command word and args.
      */
     private static final Pattern BASIC_COMMAND_FORMAT = Pattern.compile("(?<commandWord>\\S+)(?<arguments>.*)");
     private static final Logger logger = LogsCenter.getLogger(UniMateParser.class);
 
-    @Override
+    /**
+     * Parses a user input string into the appropriate command object.
+     *
+     * @param userInput The user input string to be parsed.
+     * @return A command object corresponding to the user's command.
+     * @throws ParseException If the user input cannot be parsed successfully.
+     */
     public Command parseCommand(String userInput) throws ParseException {
         final Matcher matcher = BASIC_COMMAND_FORMAT.matcher(userInput.trim());
         if (!matcher.matches()) {
@@ -80,5 +86,4 @@ public Command parseCommand(String userInput) throws ParseException {
             throw new ParseException(MESSAGE_UNKNOWN_COMMAND);
         }
     }
-
 }

src/main/java/seedu/address/model/ModelManager.java

@@ -38,7 +38,7 @@ public ModelManager(ReadOnlyAddressBook addressBook, ReadOnlyUserPrefs userPrefs
         this.addressBook = new AddressBook(addressBook);
         this.userPrefs = new UserPrefs(userPrefs);
         filteredPersons = new FilteredList<>(this.addressBook.getPersonList());
-        this.calendar = new Calendar();
+        calendar = new Calendar();
     }
 
     public ModelManager() {
@@ -121,14 +121,14 @@ public void setPerson(Person target, Person editedPerson) {
     //=========== Calendar ===================================================================================
     @Override
    public boolean canAddEvent(Event event) {
-        return this.calendar.canAddEvent(event);
+        return calendar.canAddEvent(event);
     }
 
     @Override
     public void addEvent(Event event) {
         requireAllNonNull(event);
 
-        this.calendar.addEvent(event);
+        calendar.addEvent(event);
     }
 
     //=========== Filtered Person List Accessors =============================================================

src/main/java/seedu/address/model/event/EventDescription.java

@@ -33,6 +33,17 @@ public static EventDescription createUnusedDescription() {
         return new EventDescription("THIS IS A PLACEHOLDER");
     }
 
+    /**
+     * Checks if the given string is a valid description for creating a EventDescription object.
+     *
+     * @param description description String to be checked.
+     * @return true if description is non-empty, false if it is empty.
+     */
+    public static boolean isValid(String description) {
+        requireNonNull(description);
+        return !description.isEmpty();
+    }
+
     /**
      * Retrieve the underlying String of the description.
      *
@@ -41,6 +52,7 @@ public static EventDescription createUnusedDescription() {
     public String getDescription() {
         return this.description;
     }
+
     @Override
     public boolean equals(Object other) {
         if (other == this) {

src/main/java/seedu/address/model/event/EventPeriod.java

@@ -14,7 +14,7 @@
 /**
  * Represents a period in time when an event will occur.
  */
-public class EventPeriod {
+public class EventPeriod implements Comparable<EventPeriod> {
     public static final String MESSAGE_CONSTRAINTS = "The start date and time and end date and time should "
             + "be in the format 'yyyy-MM-dd HH:mm' where:\n"
             + "    -'yyyy' is the year.\n"
@@ -91,6 +91,7 @@ public boolean isOverlapping(EventPeriod other) {
      * @param other The EventPeriod to compare with.
      * @return 1 if this EventPeriod is after the other, -1 if it's before, 0 if they are the same.
      */
+    @Override
     public int compareTo(EventPeriod other) {
         requireNonNull(other);
         if (this.start.isBefore(other.start)) {

src/test/java/seedu/address/model/event/AllDaysEventListManagerTest.java

@@ -0,0 +1,10 @@
+package seedu.address.model.event;
+
+import org.junit.jupiter.api.Test;
+
+public class AllDaysEventListManagerTest {
+    @Test
+    public void constructorTest() {
+
+    }
+}

src/test/java/seedu/address/model/event/EventDescriptionTest.java

@@ -22,6 +22,17 @@ public void constructorTest() {
         assertThrows(InvalidDescriptionException.class, () -> new EventDescription(INVALID_DESCRIPTION));
     }
 
+    @Test
+    public void isValidTest() {
+        assertThrows(NullPointerException.class, () -> EventDescription.isValid(null));
+
+        assertFalse(EventDescription.isValid(INVALID_DESCRIPTION));
+
+        assertTrue(EventDescription.isValid(VALID_DESCRIPTION));
+
+        assertTrue(EventDescription.isValid(VALID_UNUSED_DESCRIPTION));
+    }
+
     @Test
     public void createUnusedDescriptionTest() {
         assertEquals(EventDescription.createUnusedDescription().getDescription(), VALID_UNUSED_DESCRIPTION);