YongbinWang / pe

0 stars 0 forks source link

Error message is not very useful and does exactly not match the error #5

Open YongbinWang opened 1 year ago

YongbinWang commented 1 year ago

Describe the bug

Error message provided is not very useful to help the user determine which area requires fixing.

To Reproduce

Steps to reproduce the behavior:

  1. Create the following test files test3 and test1 txt
  2. adD n/test1 t/SORTING
  3. add n/test3 t/

Expected

Error message should let the user know that the first error is due to invalid command while the second error should be due to missing payload input from user to "t/".

Actual

Both error messages are the exact same and it is not very informative to the user.

Screenshots

image.png

image.png

nus-se-script commented 1 year ago

Team's Response

This is the intended behavior but we acknowledge that we could update the error messages in future iterations. However, the error message also states that the user can type help for additional assistance.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I disagree with the team's response. Firstly, they have already acknowledged that the error messages can be updated in future iterations, this should make it at least a NotInScope but the team is still trying to reject this bug.

Secondly, typing help will still not solve the issue of the error messages being not informative for the user and it seems like a one size fits all error message pertaining to command.

image.png

Thirdly, the module website also supports that the error message not addressing the error which is proven in the issue's screenshots and the below screenshot.

image.png

image.png


## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** I disagree that this has a severity of VeryLow, this is clearly not a cosmetic bug and it does affect the users usage when they have troubles identifying which part of their input is wrong. However, I update that this should have a severity of **Low** according to the module website on specificity of error message. Below is a screenshot of the bug severity to help dispute against **VeryLow**. ![image.png](https://raw.githubusercontent.com/YongbinWang/pe/main/files/31859510-5469-427e-b7c1-187f5404846f.png)