search

yongkotaro / pe

0 stars 0 forks source link

Difference in error message for edit command #4

Open yongkotaro opened 6 months ago

yongkotaro commented 6 months ago

image.png

image.png

There is difference between the error message when I put different values.

soc-pe-bot commented 6 months ago

Team's Response

Invalid command format! edit: Edits the details of the person identified by the index number used in the displayed person list. Existing values will be overwritten by the input values. Parameters: INDEX (must be a positive integer) [n/NAME] [p/PHONE] [e/EMAIL] [g/GROUP] Example: edit 1 p/91234567 e/johndoe@example.com

This might be the first error message you might received. The parameter specified was a "INDEX(must be a positive Integer)". As -1 is not a positive integer, the error message said that it is invalid command format. In contrast, 14 is a valid positive integer. It is a valid format, but person with that index does not exist. This is why two different error message was displayed. They are both accurate and work as expected.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]