search

tirameshu / pe

0 stars 0 forks source link

Irrelevant prefixes not checked specifically #2

Open tirameshu opened 4 years ago

tirameshu commented 4 years ago

Entered "add rt/BMI r/MBI dt/1234-23-49". Expected: message to ask user not to key in irrelevant input. Message shown: "DateTime should be in the format: yyyy-mm-dd hh:mm and it should contain valid number"

System assumes that "r/MBI dt/1234-23-49" is a datetime input, which is not true since it begins with a non datetime prefix. Should perhaps check for irrelevant prefixes instead of assuming.

nus-pe-bot commented 4 years ago

Team's Response

Hi there, thank you for taking the time to test this feature. dt/1234-23-49 is not a valid datetime because there is no 23rd month or 49th day. The hh:mm is also missing. Hence, this is correctly rejected by SugarMummy app.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]