Move TimeUtil ParseException throwing to ArgsParser::parse method

[chan-j-d]

Currently, the methods verifyDatesRangeIsCorrect and verifySinceDateIsValid in the TimeUtil class are throwing ParseException which is a side effect that might not make sense when the methods are reused.

RepoSense/src/main/java/reposense/util/TimeUtil.java

Lines 176 to 193 in 53d5197

	public static void verifyDatesRangeIsCorrect(LocalDateTime sinceDate, LocalDateTime untilDate)
	throws ParseException {
	if (sinceDate.compareTo(untilDate) > 0) {
	throw new ParseException(MESSAGE_SINCE_DATE_LATER_THAN_UNTIL_DATE);
	}
	}
	/**
	* Verifies that {@code sinceDate} is no later than the date of report generation, given by {@code currentDate}.
	*
	* @throws ParseException if {@code sinceDate} supplied is later than date of report generation.
	*/
	public static void verifySinceDateIsValid(LocalDateTime sinceDate, LocalDateTime currentDate)
	throws ParseException {
	if (sinceDate.compareTo(currentDate) > 0) {
	throw new ParseException(MESSAGE_SINCE_DATE_LATER_THAN_TODAY_DATE);
	}
	}

The exception thrown should be in the ArgsParser::parse method itself where they are used. Since these methods have been changed since when they were written to be a simple compareTo call, we can consider removing these methods and replacing the equivalent lines in ArgsParser::parse with the appropriate compareTo check and throw a ParseException if necessary.