junyi73 / pe

0 stars 0 forks source link

Parameter ordering does not matter in user guide but matters in addIngredient command #9

Open junyi73 opened 1 week ago

junyi73 commented 1 week ago

The parameter ordering is stated to not matter in the user guide. However, the ordering matters when using the addIngredient command. The addIngredient command works by addIngredient NAME COST in the user guide. However, switching the NAME and COST parameter when using the command in the program leads to an error. Hence there is a discrepancy between user guide behaviour and the actual behaviour of the program.

How to recreate the bug: 1)Input the command addIngredient 1.50 Flour 2)Notice that the program shows an error

image.png

image.png

nus-se-script commented 1 week 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

Thank you for raising that! However, we think that it is of low severity as described "A flaw that is unlikely to affect normal operations of the product. Appears only in very rare situations and causes a minor inconvenience only." as it is pretty clear from the user guide that the parameters are those with the prefixes. We still change the phrasing of that! And I do not think that it will cause inconvenience to users.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your reason]