search

Kaya3842 / pe

0 stars 0 forks source link

Incorrect error message for `view c` #5

Open Kaya3842 opened 5 months ago

Kaya3842 commented 5 months ago

When I typed View c instead of view c it is registered as "Unknown command"

image.png

However, when I typed view C instead of view c it is registered as "Invalid command format"

image.png

If the command is case sensitive (from the first error), view C should also be an unknown command. Therefore, an incorrect error message is shown here.

nus-pe-script commented 5 months ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

Will be shifting this to FeatureFlaw as it is an error regarding the "Specificity of error message"

Will be assigning the issue as NotInScope as

Fixing this flaw is "not essential for the app to be reasonably useful", and implementing it would not be in the scope of v1.4.

Furthermore, this feature fulfils the second point of this excerpt from the textbook. Screenshot 2024-04-20 at 11.38.48 PM.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your reason]

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