cczhouqi / pe

0 stars 0 forks source link

Specificity of error message #2

Open cczhouqi opened 2 years ago

cczhouqi commented 2 years ago

Steps to reproduce: Type command: schedule /update 1,2 3

Expected output: Error message with reason specified (e.g."You can only update one day at a time.")

Actual output: Error message only saying input is invalid.

Screen Shot 2022-04-16 at 4.45.02 PM.png

nus-pe-bot commented 2 years ago

Team's Response

Similar issue as issue #154. We believe that the error message "... user argument supplied is invalid" is specific enough for this scenario.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Specificity of error message

Steps to reproduce:

  1. run the jar file
  2. type "plan /"

Expected output: Error message with specific reason (e.g. "format error: / should be followed with actions").

Actual output: Only says "Invalid".

Screen Shot 2022-04-16 at 4.21.21 PM.png


[original: nus-cs2113-AY2122S2/pe-interim#180] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Thank you for your feedback. However, we feel that the error message is specific enough for this scenario since an invalid user action was indeed entered. The application takes any words that are after the / as a user action. If no characters were placed or entered after the /, it will be deemed as an invalid user action, since no user action was specified in the first place.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: This is under "Feature Flaws" in "Guidelines for bug triaging". The error message should tell the user that the it's the format that goes wrong.

Screen Shot 2022-04-22 at 3.27.33 PM.png