Incorrect error message for time that exceeds 60 minutes

[tanjiaxian99]

Steps to reproduce:

1. add 1 -tnSell Book -td2021-02-28 -tt00:90

Expected: Error message: Task time should not exceed 60 minutes

Actual: Error message: Task time should be of format HH:MM

[nus-pe-bot]

Team's Response

The image above is extracted from Wikipedia.

A HH:MM format is only considered valid when :

• when HH is numerical, and in the range of 0-11
• when MM is numerical, and in the range of 0-59 (60 for leap second)

So it's fair to say that when MM is 90, it is not following the HH:MM format.

Furthermore, the command text is not cleared when it is invalid. And with the result message stating invalid HH:MM format, it will redirect users to see the HH:MM portion of the command text and let them realise that they did not use a valid MM value. Therefore no inconvenience is caused as a result.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

• [ ] I disagree

Reason for disagreement: [replace this with your explanation]

---