search

gsyndromes / pe

0 stars 0 forks source link

wrong error message for edit #8

Open gsyndromes opened 4 years ago

gsyndromes commented 4 years ago

wrong error message for edit

image.png

nus-pe-bot commented 4 years ago

Team's Response

This is a correct message. The parser parses everything after the command in search for an index, and “5 testing” is clearly an invalid index. We know that intuitively, we should be providing a warning that something else is wrong. But with such a flawed input, something else can refer to so many possible things and there is no way to objectively classify this error. Thus, we chose to say that the index is invalid.

Severity has been downgraded as the error messages are purely cosmetic, not affecting the main usage of the application, but more for the purposes of informing users that their input was not understood by the application. In this case, the program is simply repeating the user input back to the user

Rejected.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Medium]

Reason for disagreement: 1) Error messages are not purely for cosmetic usages. They are used to let the user know what to input and have a great impact on the usage of the application. 2) Without the error messages, how would a user know what to input? And how would a user use the program? 3) This definitely warrants a higher severity than very Low.