Inconsistency in error message for invalid index

[JiaYi-Gallium369]

When a large number is being input as index, the actual error message differs from the expected error message.

Input: delete 12345678901 Expected Message: The person index provided is invalid Actual Message:

Invalid command format! 
delete: Deletes the person identified by the index number used in the displayed person list.
Parameters: INDEX (must be a positive integer)
Example: delete 1

[soc-se-bot]

Team's Response

We did mention in the UG that the index should be a number from 1 to the total number of existing contacts

Why It’s a FeatureFlaw:

• Correct Functionality: The delete command works as intended, rejecting invalid indices without crashing or performing unintended actions. The issue lies in the error message, which is too generic and does not align with user expectations.

• Design Issue, Not Functional: Error messaging is part of UX design. While the current message lacks clarity, the underlying functionality of the command remains intact.

  Planned Resolution: We will improve error handling to differentiate between:

• Malformed inputs (e.g., non-integer indices), which will retain the "Invalid command format!" message.

• Validly formatted but out-of-range indices, which will show "The person index provided is invalid."

  Items for the Tester to Verify

  :question: Issue type

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

• [ ] I disagree

Reason for disagreement: [replace this with your explanation]

---