search

thanhduc2000 / pe

0 stars 0 forks source link

Wrong message output #3

Open thanhduc2000 opened 4 years ago

thanhduc2000 commented 4 years ago

image.png. Should print invalid date instead because users followed the format

nus-pe-bot commented 4 years ago

Team's Response

Thank you for your comment. We feel that this is a very low severity because of these two points.

  1. The program did not crash or accept the invalid input given.
  2. Highly unlikely that the user, unless to game the system, will input a date that does not exist - in the issue posted was '32'.

However, as with point 2, it might have been a user typo and as you've recommened, we should provide a more value added reply instead of just rejecting.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Medium]

Reason for disagreement: The severity should be set as low as this is not the expected message (should be invalid date time). Moreover, like #3 , it is hard to follow for users since the error message is printed out wrongly.

image.png