History feature shows the wrong error message

[sdevih]

Description

Command history feature is said to re-run commands based on the specified index, but when a delete command is re-run, the index in the error message shown does not refer to that in the command history index but of the student list. This may be confusing to read as the error message does not match the error (given that the user cannot see the current student list) and thus, this behaviour is not specified and differs from normal expectations.

Steps to reproduce

1. list
2. delete 8
3. history
4. history 3

Expected

• An error message indicating that the delete command cannot be re run

Actual

• An error message referring to an index not specified in the command list

Screenshot

[nus-pe-script]

Team's Response

Thanks for raising this issue! While the output does seem to be giving out the correct output given that it outputs "student index", it probably will cause quite a bit of confusion for users. The output delete command in this case, wasn't made clear enough to distinguish between the student list and the command list.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

• [ ] I disagree

Reason for disagreement: [replace this with your explanation]

---

## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]

---