Deunitato / pe

0 stars 0 forks source link

Wrong error message output #3

Open Deunitato opened 4 years ago

Deunitato commented 4 years ago

No details provided. image.png

How to replicate:

  1. using the index 0
  2. Expected: Index does not exist but received invalid command
nus-pe-bot commented 4 years ago

Team's Response

Thank you for your feedback. However, this is not a bug and is the expected response of our app. The full error message in the result box if you scroll down would be

"Invalid command format! edit: Edits the details of the recipe identified by the index number used in the displayed recipe list. Existing values will be overwritten by the input values. Parameters: [recipe index] (must be a positive integer) [<n/name> <t/time> <ig/grain>... <iv/vegetable>... <ip/protein>... <if/fruit>... <io/other>... <s/step>...] Example: edit 1 t/10 iv/40g, Kailan ip/20g, Chicken thigh s/Wash the vegetables"

It is clearly stated in the error message that the recipe index must be a positive integer. Hope this clarifies!

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]