jiexiong-zeng / pe

0 stars 0 forks source link

Stock alert is wrong #12

Open jiexiong-zeng opened 2 years ago

jiexiong-zeng commented 2 years ago

image.png

alerts stock should show only ingredients with stock lower than the set value. This ingredient has the same amount as the threshhold, but is instead shown.

nus-pe-script commented 2 years ago

Team's Response

Less than or equal

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: The command prompt clearly states "There are 1 ingredients with stock less than 6.678".

The UG states lower as well. image.png

The team is dishonest in their response.


:question: Issue severity

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

Reason for disagreement: A key function is implemented different from being stated in the UG (less than or equal vs less than).