jialatteo / pe

0 stars 0 forks source link

Invalid index error message #10

Open jialatteo opened 1 year ago

jialatteo commented 1 year ago

image.png

image.png

Entering both commands should give the same error message as only the index field is invalid.

nus-pe-script commented 1 year ago

Team's Response

As mentioned in the UG, we stated that the editC command can only take in positive integers. Hence entering a non positive integer would be an invalid command format while entering a index which is not present is an invalid index but correct format.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: editC 0 p/91234567 e/johndoe@example.com a/John Street 123, #01-01 t/friend will give an incorrect format error.

However, the command words, prefixes, arguments all follow the correct command format, only the index integer value is wrong.

A more specific error message to indicate that the index integer value is wrong would be more appropriate.