winston-lim / pe

1 stars 0 forks source link

No format errors are handled #2

Open winston-lim opened 1 year ago

winston-lim commented 1 year ago

cs2113 docs states that a valid functionalitybug is having too low a specificity of error message

fav 1291291 shows Some values were invalid... but firstly, fav only supports 1 possible argument, and 2 the message not describe which parameter it refers to.

nus-se-bot commented 1 year ago

Team's Response

Hi, thank you for your bug report!

The team would like to reject this bug as intended input. It is untrue that every command only takes in one parameter (or argument): Refer to the favourite documentation in the UG below:

image.png

As such, "Some values" is an acceptable error message due to fav accepting more than one argument.

Even if this bug were to be accepted, by the rule you are quoting below:

image.png

It should be a type.FeatureFlaw bug with severity.Low.

We would contest that telling the user exactly which argument is invalid is not within the scope of the program.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]


:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]