homuntan02 / pe

0 stars 0 forks source link

Wrong Error thrown for index issues using edit #4

Open homuntan02 opened 1 year ago

homuntan02 commented 1 year ago

image.png

When trying to execute the edit command with an index outside the current list. I received an Invalid command format error instead of an index out of bounds error. This can cause users to think that the issue with their command is the formating of their command.

This is a medium severity bug as it can cause occasional inconvenience because the user will always have to check whether the format is actually wrong or the index is out of bounds.

nus-se-bot commented 1 year ago

Team's Response

Hello, while we do agree that this is an issue and it would benefit users to give a more specific error message, we have flagged this as an issue in our developers guide. As a result, we are rejecting this bug report.

image.png

Even if somehow this is not covered by the above, this should be a featureflaw and low severity according to the textbook.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: image.png

As per the abstract from the website, known bugs are still considered bugs in this PE as it affects usage of the application.

image.png

And according to the website, response.Rejected is only for expected and correct behaviour.


## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** If the error message was indeed not specific enough, this bug would be a severity.Low. However, as this is an incorrect error message, it warrants a higher severity(severity.Medium) as it will lead to users mistake an index out of bound error for an invalid format error.