search

Shruthi0310 / pe

0 stars 0 forks source link

Del command error message for invalid index #3

Open Shruthi0310 opened 2 years ago

Shruthi0310 commented 2 years ago

Steps to reproduce: del -1

Expected Output: Error message saying invalid index expected

Actual Output: Error message saying command format is incorrect

image.png

nus-pe-bot commented 2 years ago

Team's Response

We specified that index should be a positive integer, so in this case, it should be an invalid command. However for very large indexes the command is correct, but the index is invalid. We also mentioned in the screenshot that index must be positive.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: As a user, invalid command format would imply that I missed out a prefix, or added too many parameters, etc (so things that are related to the layout of the command input). However, in the screenshot above, the format should still considered valid as I entered the required parameters, which is the index. But only the value of the index, which is unrelated to the format, is wrong, hence as a user, I would be confused why it shows invalid command format error.

:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]