search

theodorekwok / pe

0 stars 0 forks source link

Wrong error message when not providing attribute value #7

Open theodorekwok opened 2 years ago

theodorekwok commented 2 years ago

error 5.PNG From the error message it states that I need 'RECIPE_NAME /ATTRIBUTE VALUE', but I provided the recipe name and '/price' as attribute value.

nus-pe-bot commented 2 years ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Error not caught properly

image.png

image.png

image.png

image.png

It was very very difficult to understand how find by attribute worked. (The dollar sign popped up in the UG in find but it clearly does not work that way.)

[original: nus-cs2113-AY2122S1/pe-interim#2686] [original labels: severity.High type.DocumentationBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

I agree, the user should have better info on dealing with a bad command.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: The error is for a different feature specifically the set functionality and its not about the find feature that the team claim its a duplicate of. The team did not provide any explanation for why it should be a duplicate to convince me.

:question: Issue type

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

Reason for disagreement: [replace this with your explanation]